Stacks doc page

With the recent redesigned Getting Started page, we now have an awesome quick start tutorial that lets users pack build one of our Sample applications using the builders.
As a followup page, we should create a Stacks page that explains the following:

"In the getting started tutorial, you used the Paketo builder to build a Node.js app. One of the core pieces to Buildpacks and Builders are Stack Images. Stacks are...". Provide simple build/run image explanation and link to https://buildpacks.io/docs/concepts/components/stack/
Paketo project stacks. Possible content:

Paketo Stacks
The Paketo project releases several stack images. These are:
Tiny:
Build Images

index.docker.io/paketobuildpacks/build:tiny-cnb

Run Images

index.docker.io/paketobuildpacks/run:tiny-cnb

Contains:

Build: ubuntu:bionic + openssl + CA certs + compilers + shell utilities
Run: distroless-like bionic + glibc + openssl + CA certs

Base:
Build Images

index.docker.io/paketobuildpacks/build:base-cnb

Run Images

index.docker.io/paketobuildpacks/run:base-cnb

Contains:

Build: ubuntu:bionic + openssl + CA certs + compilers + shell utilities
Run: ubuntu:bionic + openssl + CA certs

Full:
Build Images

index.docker.io/paketobuildpacks/build:full-cnb

Run Images

index.docker.io/paketobuildpacks/run:full-cnb

Contains:

Build: ubuntu:bionic + many common C libraries and utilities
Run: ubuntu:bionic + many common libraries and utilities

Link the stacks repo https://github.com/paketo-buildpacks/stacks

This should be in the top level nav under "getting started" as specified in https://github.com/paketo-buildpacks/paketo-website/issues/24

Changes for this are in the redesign branch, and the page will be live once the redesign branch is merged.


Message of unavailable temeprature appears every minute when AC is switched off

Hi,
Actually, when AC is switched off (or at least when homebridge starts with AC switched off) two messages appears on homebridge's log every minute:
[17/1/2022 12:54:46] [Homebridge Panasonic AC Platform] Temperature state is not available
[17/1/2022 12:54:46] [Homebridge Panasonic AC Platform] Outdoor temperature is not available
[17/1/2022 12:55:46] [Homebridge Panasonic AC Platform] Temperature state is not available
[17/1/2022 12:55:46] [Homebridge Panasonic AC Platform] Outdoor temperature is not available
[17/1/2022 12:56:47] [Homebridge Panasonic AC Platform] Temperature state is not available
[17/1/2022 12:56:47] [Homebridge Panasonic AC Platform] Outdoor temperature is not available
[17/1/2022 12:57:46] [Homebridge Panasonic AC Platform] Temperature state is not available
[17/1/2022 12:57:46] [Homebridge Panasonic AC Platform] Outdoor temperature is not available
[17/1/2022 12:58:46] [Homebridge Panasonic AC Platform] Temperature state is not available
[17/1/2022 12:58:46] [Homebridge Panasonic AC Platform] Outdoor temperature is not available
[17/1/2022 12:59:46] [Homebridge Panasonic AC Platform] Temperature state is not available
[17/1/2022 12:59:46] [Homebridge Panasonic AC Platform] Outdoor temperature is not available
It's like.. OK man, I know it, AC is off! Keep calm!
Can these two messages be showed only on debug mode? I don't need a log full of these messages that can't let me see other messages from other pluggins because log is full of these messages
I've tried to change this.platform.log.error by this.platform.log.debug on my installation for these two messages, but won't work (or I don't doing it well...)
Thanks!!

Thanks for reporting this - I will look into it. We should definitely make this less noisy.
I'm wondering what equipment you are using in your home? If you want to be extra helpful, consider switching on Debug Mode and sharing the API response for your setup (after removing any device identifiers).

Yeah, I want to help as much as I can't, but I don't know what to do... I've activated debug mode on plugin and restarted homebridge, but I won't see any extra messages on startup, just these (I've deleted messages from homebridge and other plugins)
[17/1/2022 19:13:54] [Homebridge Panasonic AC Platform] Initializing Panasonic AC Platform platform...
...
[17/1/2022 19:13:54] [Homebridge Panasonic AC Platform] Loading accessory 'Panasonic AC Outdoor Unit' from cache.
[17/1/2022 19:13:54] [Homebridge Panasonic AC Platform] Loading accessory 'Comedor' from cache.
...
[17/1/2022 19:13:54] [Homebridge Panasonic AC Platform] Attempting to log into Comfort Cloud.
...
[17/1/2022 19:13:57] [Homebridge Panasonic AC Platform] Successfully logged in.
[17/1/2022 19:13:57] [Homebridge Panasonic AC Platform] Configuring outdoor unit.
[17/1/2022 19:13:57] [Homebridge Panasonic AC Platform] Restoring accessory 'Panasonic AC Outdoor Unit' (0x000x00-000x-0x00-x000-0xx000x00xx0) from cache.
[17/1/2022 19:13:57] [Homebridge Panasonic AC Platform] Discovering devices on Comfort Cloud.
...
[17/1/2022 19:13:58] [Homebridge Panasonic AC Platform] Restoring accessory 'Comedor' (X000000000) from cache.
[17/1/2022 19:13:59] [homebridge-panasonic-ac-platform] This plugin generated a warning from the characteristic 'Model': characteristic value expected string and received undefined. See https://git.io/JtMGR for more info.
[17/1/2022 19:14:00] [Homebridge Panasonic AC Platform] Temperature state is not available
[17/1/2022 19:14:00] [Homebridge Panasonic AC Platform] Outdoor temperature is not available
...
[17/1/2022 19:15:02] [Homebridge Panasonic AC Platform] Temperature state is not available
[17/1/2022 19:15:02] [Homebridge Panasonic AC Platform] Outdoor temperature is not available
What I need to do exactly?

That's a good starting point, thanks. I can see that the model number is also not populated in your case.
Besides that, there should be an entry in your log every minute or so which shows the device status (in debug mode). Check if you can locate that as well.

Sorry but the only messages I see on log are the two mentioned before.
I think I don't debugging it well... If I can get anything more I'll tell you

The API response for your AC model seems to be different. In the logs above you removed some parts and replaced them with "...". To allow further debugging, we'll need to have a look at the API response.
I added some error handling in b6c87ab but that will only remove the characteristic warning for you.

It's like.. OK man, I know it, AC is off! Keep calm!

Your AC communicates with Comfort Cloud even when it's off. The error "Temperature state is not available" doesn't mean that the device is switched off, but the API payload isn't supported by this plugin yet. At least that's how my installation behaves.

The API response for your AC model seems to be different. In the logs above you removed some parts and replaced them with "...". To allow further debugging, we'll need to have a look at the API response.

The ... means that I deleted messages from homebridge itself or other plugins, I can put a full log with all messages from all plugins, but I think is quite irrelevant.
I've updated to new version but, again, I don't see any new messages apart from what I've posted before.
Actually, I can see same messages with debug ON and OFF, this is why I think that I'm not debugging it well, but I don't know what to do to solve this...

I added some error handling in b6c87ab but that will only remove the characteristic warning for you.

It's like.. OK man, I know it, AC is off! Keep calm!

Your AC communicates with Comfort Cloud even when it's off. The error "Temperature state is not available" doesn't mean that the device is switched off, but the API payload isn't supported by this plugin yet. At least that's how my installation behaves.

Yeah, I know that those messages don't mean that devide is off, but I know that device is off and this messages, in this case, are showed because that
If I can get any new messages, from API or whatever, I post it here to let you know

Sounds good. If you turn on debug mode, you should see "Comfort Cloud - getDeviceStatus()" in your log, and the text under it is the API response which would allow further debugging. Consider putting your log into a Gist for increased clarity and organisation.

Yeah is what I expect to see, but I don't... I have to look at it with time and patience

Removing pull request reference because it won't fix all the issues brought up here.

Finally I've learned how to correctly debug a homebridge instance...
There you have the log: https://gist.github.com/efipso/ac6e9c9ed9c07fb8ccbc626c9ee71945
I've just removed all lines not refering this plugin and deleted IDs from lines of this plugin
Hope this log can help you

From https://github.com/embee8/homebridge-panasonic-ac-platform/issues/17#issue-1112387051

Related with closed issue #9 (wich I cannot re-open), sorry if a new issue is not necessary
Finally I've learned how to correctly debug a homebridge instance...
There you have the log: https://gist.github.com/efipso/ac6e9c9ed9c07fb8ccbc626c9ee71945 I've just removed all lines not refering this plugin and deleted IDs from my device/connection
Hope this log can help you

Thanks for posting the detailed log, @efipso.
I compared it to my instance and conclude that your installation does indeed not report the insideTemperature and outTemperature. To be precise, they are included in the response object, but carry the value 126, and it was previously established that this value can be interpreted as undefined/unsupported.
When the plugin fetches the device list, there is also a field deviceModuleNumber in my setup which is not returned for you. This is not relevant for the functionality of the plugin, but I am documenting it anyway.
In your device list, check the field deviceGuid. While I am not interested in your exact serial number, it would be great to know the value before the + symbol, which indicates the series/model of your AC. If we know that, we can start to maintain a list of unsupported devices and, where possible, modify the plugin to support them.

Closing this issue since we cannot report a temperature through the plugin that is not returned via the API.
Consider sharing the device model if you have it at hand, so that we can maintain a supported/unsupported list in the docs.

Hi,
My indoor unit device model is CS-TZ35TKEW using CZ-TACG1 adapter for wifi.
I don't know exactly the device model for outdoor unit as it's on the roof and I can't look at it, but it has to be CU-TZ35TKE or CU-TE35TKE (if I have to bet, I think is the firt one)


Make up to date with Devanagari fix commits

https://github.com/tabulapdf/tabula-extractor/commit/0743e96cf190311aa60c2b34106408b271ec36c9
https://github.com/tabulapdf/tabula-extractor/commit/9e69136957207c1b2f58921866706625ad2af8ca
https://github.com/tabulapdf/tabula-extractor/commit/c7fd88ccc51546e10c1496b6fdbe0c389bbe97d0
https://github.com/tabulapdf/tabula-extractor/commit/da2ab4ff4360bfb0aabf22adffac6981559afcaf
https://github.com/tabulapdf/tabula-extractor/commit/0b47c4f6bbccc7281bd66d2278cf9958cf5bac9f
Together, these commits do the following

order characters in a TextElement by their horizontal center point, not their left.
take true averages of character width and space-between-words rather than running pseudo-averages (memo + new) / 2
test the above

Hey, can you take a look at a164831d9ac19bd005f782b3c1be941b28b184b1 and check if the tests I added match up with those in tabula-extractor?

Close when PR #19 is merged

Hmm. What I don't get is that tabula-java seems to work perfectly before the fix. I think these fixes are still good practice... but I'm confused in any case. Maybe it has something to do with a different PDFBox version?

On Apr 25, 2015, at 7:11 PM, Jeremy B. Merrill<EMAIL_ADDRESS>wrote:
Maybe it has something to do with a different PDFBox version?

Probably. The glyph-handling code of the snapshot version we’re stuck with in tabula-extractor is substantially different than 1.8.x

Yep, that's it:
Compare
technology.tabula.TextElement[x=549.49,y=55.42,w=6.76,h=4.58,bottom=60.000000,right=556.250000,text="N"]
technology.tabula.TextElement[x=556.22,y=55.42,w=5.75,h=4.58,bottom=60.000000,right=561.969971,text="^"]
technology.tabula.TextElement[x=561.97,y=55.42,w=0.06,h=4.58,bottom=60.000000,right=562.029968,text="n"]
technology.tabula.TextElement[x=561.97,y=55.42,w=6.78,h=4.58,bottom=60.000000,right=568.750000,text="L"]
technology.tabula.TextElement[x=568.81,y=55.42,w=2.86,h=4.58,bottom=60.000000,right=571.669983,text="l"]

to
#<TextElement: 55.42,549.49,60.0,556.25 'N'>
#<TextElement: 55.42,556.22,60.0,561.97 '^'>
#<TextElement: 55.42,561.96,60.0,568.74 'L'>
#<TextElement: 55.42,561.97,60.0,562.02 'n'>
#<TextElement: 55.42,568.8,60.0,571.66 'l'>


Nothing works yet

I'm experimenting to see whether I can produce anything usable, very frustrating trying to replicate the expressivity of Python in Java believe me. Maybe I should learn some newer language. Even javascript would be better!

Nested Bitstruct, some Macros and some Adapters working :-)


Subgraph console fails for two or more named queries

Do you want to request a feature or report a bug?
Report a bug.
What is the current behavior?
The subgraph console with the following error:
{
  "errors": [
    {
      "message": "Operation name required"
    }
  ]
}

If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem.

Both queries should be successfully executed.

I'm having the same issue. Any updates?

still seems broken for me...almost 2 years later...

Hey @Rjrunner44, thanks for re-reporting. Sorry about that. We were working on a better overall better dev-experience and will unveil it soon. Until then, the recommended work-around is to use https://graphiql-online.com.

Actually, looking at this again it seems like this is against the GraphQL spec.  Feel free to reopen it if that does not solve the problem for you.


How to store current page on localstorage

How to store current page on localstorage

if you using ReactViewer, using onPageChange method.
If you using EpubViewer, using pageChanged method.
Just pass it as props.

@altmshfkgudtjr if I close the reader and open it again it gives me page one rather than the last location and saving the location on localstorage does not help either

https://github.com/altmshfkgudtjr/react-epub-viewer/blob/0eca3d003f8a830ecffb513dd7d38b630687a91d/src/containers/Reader.tsx#L78-L85
How about executing the following code through useEffect ?
useEffect(() => {
  if(!viewerRef.current) return;

  const targetCFI = window.localStorage.getItem('savedCFI')
  viewerRef.current.setLocation(loc);
}, [viewerRef]);

or using rendition
const App = () => {
  const ref = useRef(null);
  const [rendition, setRendition] = useState(null);

  const onRenditionChanged = (rendition) => setRendition(rendition);

  useEffect(() => {
    if (!rendition) return;
    const targetCFI = window.localStorage.getItem("savedCFI");
    rendition.display(targetCFI);
  }, [rendition]);

  return (

      <EpubViewer
        ref={ref}
        url="https://altmshfkgudtjr.github.io/react-epub-viewer/files/Alices%20Adventures%20in%20Wonderland.epub"
        rendtionChanged={onRenditionChanged}
      />

  );
};

@altmshfkgudtjr
where do you set this  window.localStorage.getItem("savedCFI");

const onPageChange = (pageInfo) => {
    setChapterName(pageInfo.chapterName);
    setPage(pageInfo.currentPage);
    setTotalPage(pageInfo.totalPage);

    // like this?
    window.localStorage.setItem('savedCFI', pageInfo.startCfi);
};

return (

      <ReactEpubViewer
        ref={ref}
        url="https://altmshfkgudtjr.github.io/react-epub-viewer/files/Alices%20Adventures%20in%20Wonderland.epub"
        onPageChange={onPageChange}
      />

  );
);

Yes,exactly thats my question if Page is reloaded onpagechange tiggered and the current cfi or saveCfi is page 0 not last located page.

oh,
Has confirmed. I'll check it out.

and one more thing, Is it possible to add reading time and percentage? If possible,how?

Yes,exactly thats my question if Page is reloaded onpagechange tiggered and the current cfi or saveCfi is page 0 not last located page. Try reloading browser

As a result of testing in sandbox, it was confirmed that it works well.
Can you check it out?
const App = () => {
  const ref = useRef(null);
  const [rendition, setRendition] = useState(null);
  const [chapterName, setChapterName] = useState("");
  const [page, setPage] = useState(0);
  const [totalPage, setTotalPage] = useState(0);

  const onRenditionChanged = (rendition) => setRendition(rendition);

  const onPageChange = (pageInfo) => {
    setChapterName(pageInfo.chapterName);
    setPage(pageInfo.currentPage);
    setTotalPage(pageInfo.totalPage);
    window.localStorage.setItem("savedCFI", pageInfo.startCfi);
  };

  useEffect(() => {
    if (!rendition) return;
    const targetCFI = window.localStorage.getItem("savedCFI");
    rendition.display(targetCFI);
  }, [rendition]);

  return (

        Page: {page} / {totalPage} [{chapterName}]

      <EpubViewer
        ref={ref}
        url="https://altmshfkgudtjr.github.io/react-epub-viewer/files/Alices%20Adventures%20in%20Wonderland.epub"
        pageChanged={onPageChange}
        rendtionChanged={onRenditionChanged}
      />

  );
};

and one more thing, Is it possible to add reading time and percentage? If possible,how?

percentage is simply the current number of pages divided by the total number of pages.
but what is reading time?

total time user spend reading a book

Yes,exactly thats my question if Page is reloaded onpagechange tiggered and the current cfi or saveCfi is page 0 not last located page. Try reloading browser

As a result of testing in sandbox, it was confirmed that it works well.
Can you check it out?
const App = () => {
  const ref = useRef(null);
  const [rendition, setRendition] = useState(null);
  const [chapterName, setChapterName] = useState("");
  const [page, setPage] = useState(0);
  const [totalPage, setTotalPage] = useState(0);

  const onRenditionChanged = (rendition) => setRendition(rendition);

  const onPageChange = (pageInfo) => {
    setChapterName(pageInfo.chapterName);
    setPage(pageInfo.currentPage);
    setTotalPage(pageInfo.totalPage);
    window.localStorage.setItem("savedCFI", pageInfo.startCfi);
  };

  useEffect(() => {
    if (!rendition) return;
    const targetCFI = window.localStorage.getItem("savedCFI");
    rendition.display(targetCFI);
  }, [rendition]);

  return (

        Page: {page} / {totalPage} [{chapterName}]

      <EpubViewer
        ref={ref}
        url="https://altmshfkgudtjr.github.io/react-epub-viewer/files/Alices%20Adventures%20in%20Wonderland.epub"
        pageChanged={onPageChange}
        rendtionChanged={onRenditionChanged}
      />

  );
};

This works.Thanks a lot

Could you please provide an example with ReactEpubReader everything works fine except setting the epub CFI on page load, using the useEffect provided above results in an error saying: viewerRef.current.setLocation is not a function, any help would be appreciated.
Thanks

set it in location.
first do it
location = {location}
use useState [location, setLocation] = useState(Cookies.get("currentlocation")) or  useState('epubcfi(....)')


Using of java.time generates exceptions

Hi,
Could you help? Why when I'm using java.time classes as indexed fields this generates a conversion error like this.
java.lang.ClassCastException: java.util.Date cannot be cast to com.mongodb.DBObject

My pojo class below:
@Entity(value = "customers")
public class Customer {
    @Id
    private ObjectId id;
    @Indexed(unique = true)
    private long customerId;
    @Indexed
    private String firstName;
    @Indexed
    private String lastName;
    @Indexed
    private Gender gender;
    @Indexed
    private LocalDate birthDate;

// getters and setters
}

You need to write a converter similar to #404

depends on #940

closing this since this is tracked by #940


Summary is not added when pushing changes

I have a PR in draft mode that I've been pushing changes to.
I'd expect that it would run this action and add a comment after a successful CI run but it's failing since the pr variable isn't being set here - https://github.com/daun/playwright-report-summary/blob/main/src/index.ts#L73-L77
I'm not sure if just adding pr = issueNumber would fix this or not but ideally I should have control over when this action runs and updates the comment -- not the action itself (does that make sense?).
I'm going to try to fork this to see if I can get things working. If it does, I'll open a PR.

I personally would have expected the action to not run at all on draft PRs. If you find a way to do this, a PR sounds great!

@jkalberer I think I misunderstood your question.
When you open a PR, the action first adds a comment containing the test result. Pushing additional commits to the PR branch will update the existing comment to show the new test results. It won't create new comments. If you're not seeing that, it's probably a configuration issue. From a bit of research, the PR being a draft shouldn't make a difference here.
I'll close this for now, but feel free to get back if you're still seeing the issue.


Clean JUnit5 tests visibility

See https://rules.sonarsource.com/java/RSPEC-5786

Build failed, following https://github.com/jhipster/generator-jhipster/pull/16431 as some config have changed
cc @swarajsaaj


add check to ensure a lookup value is populated.

This request provides check to ensure a lookup value is populated during adding or editing of a lookup entry.

Looks like banjo app is validating the lookup values before passing the value to amberdb, so I'll close this request.


Wiring pass broken after annotation changes

I put the fix here https://github.com/ucb-bar/firrtl/tree/fix-wiring-pins
I wanted to use Jack's PR so I made the branch based off that rather than master which makes the PR look strange.
Also, I'm not sure if thats the right way to do it, but it seems like the fewest changes.

closed by #416


CB-17557: Update blueprints for cdp 7.2.16 Streams Messaging clusters

The StorageOperations role will now be in StubDfs service, so existing templates that contain CoreSettings need to be updated. This commit updates the streams messaging templates.
Test:

Manually on provisioned cluster (local)

could you copy the pr description into the commit message @patrik-marton ?


.exe Issue/Connection Issue

Upon following the instructions and compiling the code into a .exe I have received the following message:

There have been no changes to the pre-existing code aside from editing the config.ini
The window pops up for half of a second and then closes itself automatically. The bot doesn't launch or even make an attempt to connect, assuming because of one of these issues or error related messages.
Have ran the all-in-one pre-compiled and it functions as intended with my current config.ini however I wanted to make a few alterations such as extending the spam-ban criteria. Clearly something you can not do with an already compiled .exe
Any solutions or suggestions to help get this thing up and running would be greatly appreciated. Marvelous piece of work and I hope you guys keep it up!

Should be fixed, please confirm and close this issue @gloriousone

Confirmed the fix. Much appreciated!
Side note:
UserWarning: The soupsieve package is not installed. CSS selectors cannot be used.
I have been receiving this warning and confirm soupsieve is installed and up to date. Any thoughts? Doesn't seem to affect functionality in any way I've found so far. Present in both the precompiled and the one I just compiled myself.

Yeah this can safely be ignored.


Adds alternative LI generator to read profiles from db

LI generator which reads in injector parameters by injector ID from json db file for collimator, diffuser and laser ball simulation.
Placeholder light injectors (esp. OD collimators and diffusers) in data/LightInjectors.json
Example usage: macros/LIGen.mac
Example for reading output sample-root-scripts/read_LIGen_OD_output.C
For OD light injectors, it's currently recommended to comment out #define ACTIVATE_IDPMTS in src/WCSimConstructCylinder.cc (line 71), pending a change to the existing geometry.

Thanks @ekneale Just a couple of small things left

Just to explain to everyone why the validation is breaking for OD hits (It scared me at first)
It's your change to WCSimEventAction to apply the RelativeHitTime to OD PMTs (in the same way it is applied to ID 20" & mPMT PMTs)
This is a bug fix, so is working as expected

Hopefully all sorted now @tdealtry

Sorry @ekneale some conflicts to resolve after #379...

Yeah that's right
Thanks for the contribution @ekneale


Debug: Logpoint: Intellisense

Please allow for intellisense when entering in the Log Message Template Section {}, currently is says No Suggestions. Thanks!

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

This feature will make debugging in vscode a lot better experience & improve productivity.
We appreciate this feature.

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

:slightly_frowning_face: In the last 60 days, this feature request has received less than 20 community upvotes and we closed it. Still a big Thank You to you for taking the time to create this issue! To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

It may be worth adding here that this was not a request for new functionality. This functionality is already implemented but is broken - I suspect due to changes in the bracket system.
https://github.com/microsoft/vscode/blob/61532aeb5643822cf46cb8ccf0301024ad4af483/src/vs/workbench/contrib/debug/browser/breakpointWidget.ts#L247-L277
That code clearly indicates that if the context is a log message and isPositionInCurlyBracketBlock(this.input), completions should be provided. That they are not being provided is a bug.

Thanks for pointing that out, I agree this actually looks broken. @hediet it looks like you changed the bracket detection logic, could you take a look?

This actually works for symbols:

Member completion never worked here and will probably never be supported.
The reason for this is that typescript cannot offer contextual member completion for strings outside of source files.

@hediet, what you're seeing is not what I'm seeing in 1.72.2. For me, no suggestions are ever provided for a log point in a TS file, whether it's a member reference or not:

@roblourens @hediet like Colin mentioned I'm not getting any completions whatsoever for logpoints when using v1.72.2.

I can reproduce now after clearing my settings.


Cannot implicitly witness a parameter

Welcome to the Ammonite Repl 1.0.1
(Scala 2.12.3 Java 1.8.0_131)
If you like Ammonite, please support our development at www.patreon.com/lihaoyi
@ import shapeless.Witness
import shapeless.Witness

@ Seq("xxx").map { s =>
    implicitly[Witness.Aux[s.type]]
  }
cmd1.sc:2: could not find implicit value for parameter e: shapeless.Witness.Aux[s.type]
  implicitly[Witness.Aux[s.type]]
            ^
Compilation Failed

I'm pretty sure this is because s isn't a stable path. If you add an intermediate val you'll get the result you're after,
@ Seq("xxx").map { s =>
      val s0 = s
      implicitly[Witness.Aux[s0.type]]
    }
res0: Seq[Witness{type T = s0.type} forSome { val s0: String }] = List(shapeless.Witness$$anon$1@15c4b1a4)

Having the compiler treat s as stable might be possible, but it'd be a scalac change, not a shapeless change.

Then why Witness.apply works?
@ Seq("xxx").map { s =>
    Witness(s)
  }
res1: Seq[Witness{type T = s.type} forSome { val s: String }] = List(shapeless.Witness$$anon$1@687e6293)

More information:
@ repl.compiler.settings.XlogImplicits.value = true

@ Seq("xxx").map { s =>
    implicitly[Witness.Aux[s.type]]
  }
cmd4.sc:2: shapeless.this.Witness.apply is not a valid implicit value for shapeless.Witness.Aux[s.type] because:
hasMatchingSymbol reported error: Type argument s.type is not a singleton type
  implicitly[Witness.Aux[s.type]]
            ^
cmd4.sc:2: could not find implicit value for parameter e: shapeless.Witness.Aux[s.type]
  implicitly[Witness.Aux[s.type]]
            ^
cmd4.sc:10: materializing requested scala.reflect.type.ClassTag[Seq[Nothing]] using scala.reflect.`package`.materializeClassTag[Seq[Nothing]]()
            .print(res4, "res4", _root_.scala.None)
                  ^
Compilation Failed

Looks like an issue in the macro materializer

Any idea @milessabin ?

This bug does not exist in TypeLevel Scala's ValueOf.

Fixed in https://github.com/milessabin/shapeless/commit/e70f01ecdcca9c2a991fb045b002dc32e9d88388.


Unlocked cutscene FPS doesn't really correspond to actual frame rate increase

Even though the fps counter reads the unlocked value the visual appearance of the cutscenes appears to be 30 fps only

Are you using frame generation?

Not using framegen

Not sure what is cause there. Maybe could be a composite cutscene which they talked about in gdc where they add pre-rendered stuff to dynamically rendered scene (ex-> the opening battle scene between two armies with shiva and titan)

Various cutscenes are prerendered with a very high bitrate. The videos for these cutscenes are 30 fps.

That being said; is it possible to also include the ability to select a framerate cap for cutscenes specifically? Something like this;

[Cutscene Framerate]
; Select Framerate for Cutscenes or Photo Mode (default: 60)
CutsceneFramerate = 60

the reason is that the Dynamic Resolution system seems to be based on the targeted in-game framerate, so having a better way to adjust that would be nice. Not only that; it's similar to how Gears 5 approaches its Cinematic framerate cap while giving PC players the option to either set a specific cap or disable it completely.

Not sure what is cause there. Maybe could be due to it being a composite cutscene which they talked about in gdc where they add pre-rendered stuff to dynamically rendered scene (ex-> the opening battle scene between two armies with shiva and titan)

You're likely just seeing pre-rendered video as mentioned by Drahsid.

That being said; is it possible to also include the ability to select a framerate cap for cutscenes specifically? Something like this;

[Cutscene Framerate]
; Select Framerate for Cutscenes or Photo Mode (default: 60)
CutsceneFramerate = 60

the reason is that the Dynamic Resolution system seems to be based on the targeted in-game framerate. Ideally; prioritizing visual fidelity in cutscene would help-- knowing how demanding the cutscenes can be on low-spec/mid-spec PCs and Steam Deck.
Not only that; it's similar to how Gears 5 approaches its Cinematic framerate cap while giving PC players the option to either set a specific cap or disable it completely.
edit: and a perfect excuse to run cutscenes at 48fps while making a IMAX-style Aspect ratio for the true cinematic experience. :P

I'd be somewhat hesitant about adding that as an option given most in-game frame limiters tend to be awful. The utility, however, of a per cutscene framerate limit might offset that though. Especially for lower end devices/PCs. I'll look into it and see if I can add it.

Added in v0.7.7


[mac] obtain 7za from stevedore

Instead of including a 7za binary here, let's fetch it from stevedore.

Quoting @KasparasKralikas on slack:

Anyways, if it works on Bokken VM, it should be good, approving it

(Double-checked that it works by spawning a bokken VM and running the commands manually.)


Same profile avatar displayed for different accounts

Checklist

[X] I have used the search function to see if someone else has already submitted the same bug report.
[X] I will describe the problem with as much detail as possible.

App version
6.201
Where did you get the app from?
Google Play
Android version
Lineage OS unofficial (Android 10)
Device model
Samsung Galaxy A3 (2017)
Steps to reproduce

Open the side menu by sliding from the left border of the screen
Click on the arrow to open the accounts dropdown menu

Expected behavior
I have 5 different accounts :

3 from Gmail
1 from Google Workspace (pro account)
1 from Outlook

I expect every account to display its own profile avatar if set.
Actual behavior
Th 3 Gmail accounts display the same profile avatar. It's the correct avatar for one of them (the one crossed out in green on the screenshot), but the 2 others don't have their correct avatar.
The Google Workspace account (orange profile avatar on the screenshot) is supposed to have an avatar as well, but it isn't displayed on K9.
The Outlook account doesn't have any avatar, so it's normal to see the default avatar.

Logs
I activated the debug mode and reproduced the steps at 10:03
k9mail-logs.txt

This is not a bug (but annoying/confusing). See #5509


Support for multitask classifigression is lacking

This is not very flexible. But I will pass it for now since we need results.
Originally posted by @DomInvivo in https://github.com/datamol-io/goli/pull/58#discussion_r1215940433

Fixed by @zhiyil-graphcore


Password Match

Password Match
User Story
AS AN employee with access to sensitive data
I WANT to randomly generate a password that meets certain criteria
SO THAT I can create a strong password that provides greater security
Acceptance Criteria
GIVEN I need a new, secure password
WHEN all prompts are answered
THEN a password is generated that matches the selected criteria

Feature/password generation


Version database changes with migrations

As of version 0.2.3, the database is setup through an sql script this will get complex as the app evolves. Additionally, new version setup deletes and creates the database which means all the data is lost.  With versioned migrations, this will not be the case.

Added migrations in commit 3df1200e63753875c29eed52b2c3aad5cffb0d5f


Company culture discussions

We should agree on:

[ ] Vision
[ ] Mission
[ ] Purpose
[ ] Values
[ ] Principals

Examples of company values
https://www.cascade.app/blog/examples-of-great-company-values

Vision vs mission, + examples https://www.businessnewsdaily.com/3882-vision-statement.html
plus lecture slides 20221027-PUB-Planspiel-02.pdf

Purpose:
Our goal is to deliver the best possible solution, that guarantees security, the data availability and transparency for our Clients and increases the productivity of manufacturing enterprises.
Vision:
Become the customer's first thought when it come to Industrial Data Management. [check if the first idea exists as a fixed/regular phrase]
To become the leading and most reliable data management company.

Values:
Passion
Privacy
Dignity
Peace
Support
Loyalty
Teamwork
Agile
Flexibility
Trustworthiness
Reliability
Security
Simplicity
Communication
Commitment to Customers
Innovation
Quality
Equality
Openness
Respect
Optimism


Advanced CSS (make your own bootstrap)

Not JS, but I guess your projects also have styles!
I could give that talk if you are interested :)

+1 for this one, I really think that @BcnJS should have more CSS talks

@Elecash @rnowm could you go into more detail what you would cover/expect from the talk?

There is a lot of tooling written in JS for CSS stuff
@jgnatch talked about CSS Modules which are quite good approach to have better CSS management for writing web components to keep encapsulations and isolation.
The most of the tooling for that kind of stuff is, today, written in node; for example postcss have so many plugins to do tons of new things.
So I believe that we can have good talks of CSS.
I hope that this comments could help to give a description of the talk which gather the interests of the community.

Sorry for not answering before, my gh notifications are muted :)
I could do something like this:
http://rnowm.github.io/bootstrapU/1.html
Talking about pros and cons of css frameworks.

Could you add, if you have experience on, in how styles and layouts impact in painting performance?
Some people like me, we spend our main time in JS like so we don't have that much knowledge about; we have sometimes to build HTML and CSS and we are quite clear in what to worry in terms of JS execution and browser API calls to avoid bad performance, however we usually wonder (or at least I do) if the HTML has the right structure and CSS is well structure to avoid painting bottle necks obtaining similar results. I totally understand that it depends of the dynamism of the site (animations, etc) but I don't know if there are things to avoid as much as we can to get a performance boost.

Sure I could :)


Fix : allow some url to be allowed in script-src CSP

Closes #651
Description
add this url, to be allowed in script-src CSP

*.netlify.com
unpkg.com
analytics.google.com
*.google-analytics.com

after see the preview
'unsafe-eval' must be allowed.
it's this okay?

Can we pass different config for different domain?

Can we pass different config for different domain?
The error on the Homepage are Gone for now.
But for the CMS still exist.

right now I don't know how we can pass different configs for different domains.
but for now, if we want to quick fix, I will allow 'unsafe-eval'
then we will figure out later

  for = "/admin*"
  [headers.values]
    // Set different value
  for = "/*"
  [headers.values]
    X-Frame-Options = "DENY"
    X-XSS-Protection = "1; mode=block"
    X-Content-Type-Options = "nosniff"
    Referrer-Policy = "same-origin"
    Content-Security-Policy = "default-src 'self'; style-src 'self' 'unsafe-inline'; script-src 'self' 'unsafe-inline' *.googletagmanager.com *.netlify.com unpkg.com analytics.google.com *.google-analytics.com; img-src 'self' blob: data: https:; frame-ancestors 'none'; "
    Permissions-Policy = "camera=(), microphone=(), geolocation=(), interest-cohort=()"

How about differentiate based on path?
  for = "/admin*"
  [headers.values]
    // Set different value
  for = "/*"
  [headers.values]
    X-Frame-Options = "DENY"
    X-XSS-Protection = "1; mode=block"
    X-Content-Type-Options = "nosniff"
    Referrer-Policy = "same-origin"
    Content-Security-Policy = "default-src 'self'; style-src 'self' 'unsafe-inline'; script-src 'self' 'unsafe-inline' *.googletagmanager.com *.netlify.com unpkg.com analytics.google.com *.google-analytics.com; img-src 'self' blob: data: https:; frame-ancestors 'none'; "
    Permissions-Policy = "camera=(), microphone=(), geolocation=(), interest-cohort=()"

ah I see, let's give it a shot

Still not working, so let's try to allow on all pages for now @asaadam

Still not working, so let's try to allow on all pages for now @asaadam

okay, working on it

Works in my end.
let's discuss later with @resir014 ya to tackle this issue properly.

@all-contributors please add @asaadam for code


Allow iOS to connect to servers without SSL

Summary
It appears to have a conflict with NSAllowLocalNetworking and NSAllowsArbitraryLoads, by removing NSAllowLocalNetworking we are now allowing connections without SSL.
Also some changes around pinging the server to make the connection faster.
Ticket Link
https://mattermost.atlassian.net/browse/MM-57387
Fixes: #7844
Release Note
Added back the ability to connect the iOS app to servers without SSL

@mvitale1989 this is failing to build due to
 Already downloaded<EMAIL_ADDRESS>  ==> Pouring<EMAIL_ADDRESS>  Error: The `brew link` step did not complete successfully
  The formula built, but is not symlinked into /usr/local
  Could not symlink bin/2to3-3.11
  Target /usr/local/bin/2to3-3.11
  already exists. You may want to remove it:
    rm '/usr/local/bin/2to3-3.11'

  To force the link and overwrite all conflicting files:
    brew link --overwrite<EMAIL_ADDRESS>
  To list all files that would be deleted:
    brew link --overwrite<EMAIL_ADDRESS>--dry-run

  Possible conflicting files are:

now the question here is why did we add python? I don't see how is this needed.. in fact I would consider removing watchman as is really needed for development.
I'm going to give it a go in this PR

@mvitale1989 as you can see removing watchman and python from the build works just fine.

/cherry-pick release-2.15

Cherry pick is scheduled.


rabbitmq headers

Is it possible to read headers from the rabbitmq input plugin?
There's an old jira ticket it mentions such a feature. And from the ticket it looks like it should work. But I can't find anything in the docs to show how to extract headers from rabbitmq.
https://logstash.jira.com/browse/LOGSTASH-926
In the docs for an older version of logstash it has a headers_field as a config option. This option is no longer available in the newer versions.
Also looked at "@metadata" and it seems to be empty.
Thanks

Delivery info and properties are excluded from event decoding: https://github.com/logstash-plugins/logstash-input-rabbitmq/blob/master/lib/logstash/inputs/rabbitmq/bunny.rb#L108-112.
Not sure if Logstash core allows events to have metadata these days.

michaelklishin, thanks for the explanation.  The metadata wouldn't really have to pass to logstash core.  It could be copied to a field within in the input plugin and that way it could get passed to the filter. (not sure if add_field works that way).
But according to the source mentioned above (bunny.rb) it looks like everything except for data is discarded.  Probably for performance reasons.
thanks again.

@fsarwari not for performance reasons. Nobody thought it would be particularly useful when the plugin was re-written about a year ago. We can add it. The question is, is there required infrastructure for this change in the plugin system these days, and how many people want this.

This is a duplicate of #7 which I'm working on addressing since I'm very interested in getting the routing key and timestamp from the consumed messages. I'll close that issue in favor of this one.

@michaelklishin nowadays ( LS 1.5 ) we have a @metadata object that can contain information but won't be indexed.
Perhaps that can used. See https://github.com/elastic/logstash/pull/1836 for some details.

what should be in that metadata object? I passed some properties to the queue, but I see no way to retrieve them in Logstash. I can't find both: headers and properties in metadata field. It's just empty. I'm particularly interested in user_id property. I can send data to RMQ with property user-id which will be validated against client's certificate CommonName so nobody can spoof the message, but I'd like also to pass that msg to logstash so I can use it in filters and output...

Yes, that's exactly the kind of scenario that solving this issue will address (do note that the issue is still open). I'm done with the implementation but got stalled when I reached the unit tests. I might have time to revisit this in the upcoming couple of weeks.

Any updates about this?

I haven't been able to prioritize the completion of this. The fact that the Logstash team doesn't seem to be reviewing and merging even trivial typo-correction pull requests isn't really motivating me to complete the rspec work. I can upload what I've got so far though.

In my project, I am pulling RabbitMQ messages into Elasticsearch using this plugin. Routing keys are required so that I know where the messages come from. I am willing to contribute to this if someone can help me get stared.

@jingzhaoou If you could test my patch and see if it gives you want you need I would be grateful to hear your feedback [@metadata][routingKey] should contain what you need.

I'd love to have this available as well. Especially exchange, queue name and routing key would be useful, but everything we can get out of RabbitMQ should be made available in @metadata, imho.

This has just been merged into master and should be released soon!


Update invitation link to Kubeflow Slack channel

The old invitation link had expired. Thanks to @jlewi who provided this new link.

This change is

@jlewi

/assign @nickchase

Thanks!
/ok-to-test
/lgtm
/approve

thanks @sarahmaddox

FYI: PR to change the Slack invite link in kubeflow/examples docs: https://github.com/kubeflow/examples/pull/769


Cannot start server

I'm trying to start this server so I can mark the associated listing as tested.
Current Behavior
This is my Dockerfile:
FROM python:3.10

RUN pip install uv

WORKDIR /app

RUN git clone https://github.com/szeider/mcp-solver .

RUN uv sync

CMD ["uv", "run", "mcp-solver"]

When I run it, I get the following error:
/app/.venv/lib/python3.10/site-packages/minizinc/__init__.py:26: RuntimeWarning: MiniZinc was not found on the system. No default driver could be initialised.
  warnings.warn(
Traceback (most recent call last):
  File "/app/.venv/bin/mcp-solver", line 8, in <module>
    sys.exit(main())
  File "/app/src/mcp_solver/server.py", line 277, in main
    asyncio.run(serve())
  File "/usr/local/lib/python3.10/asyncio/runners.py", line 44, in run
    return loop.run_until_complete(main)
  File "/usr/local/lib/python3.10/asyncio/base_events.py", line 649, in run_until_complete
    return future.result()
  File "/app/src/mcp_solver/server.py", line 19, in serve
    model_mgr = ModelManager()
  File "/app/src/mcp_solver/model_manager.py", line 22, in __init__
    self.solver = Solver.lookup(solver_name)
  File "/app/.venv/lib/python3.10/site-packages/minizinc/solver.py", line 135, in lookup
    assert driver is not None
AssertionError

Is there a way to install all dependencies using uv sync here?
Expected Behavior
The server should start correctly when using uv run.
Suggested Improvement
Could you provide a complete example in the README showing how to start the server using uv run?
This would improve usability for devel

Thanks for reaching out! From the error message, it looks like the issue is that MiniZinc, a required external dependency for the MCP Solver, is not installed or accessible in your environment. While uv sync sets up Python dependencies, MiniZinc needs to be installed separately.
Here are some suggestions to resolve the issue:

Make sure MiniZinc is installed and available in the environment where you're running the solver.
If you're using Docker, you might need to add a step in your Dockerfile to install MiniZinc.
On macOS or Linux systems, you can use a package manager like Homebrew or apt to install MiniZinc directly.
Once MiniZinc is installed, try re-syncing the environment with uv sync and starting the server again with uv run mcp-solver.

Let me know how it goes or if you need further help!


[Bug] ScatterChart brush selection with large amount of data is invalid

Version
5.3
Link to Minimal Reproduction
https://echarts.apache.org/examples/zh/editor.html?c=scatter-large
Steps to Reproduce

Create a scatter chart of a large amount of data
Add brush configuration items to scatter chart
Brush selected data

The specific code is as follows:
function genData(len, offset) {
  let arr = new Float32Array(len * 2);
  let off = 0;
  for (let i = 0; i < len; i++) {
    let x = +Math.random() * 10;
    let y =
      +Math.sin(x) -
      x * (len % 2 ? 0.1 : -0.1) * Math.random() +
      (offset || 0) / 10;
    arr[off++] = x;
    arr[off++] = y;
  }
  return arr;
}
const data1 = genData(5e5);
const data2 = genData(5e5, 10);
option = {
  title: {
    text:
      echarts.format.addCommas(data1.length / 2 + data2.length / 2) + ' Points'
  },
  tooltip: {},
  toolbox: {
    left: 'center',
    feature: {
      dataZoom: {}
    }
  },
  legend: {
    orient: 'vertical',
    right: 10
  },
  xAxis: [{}],
  yAxis: [{}],
  dataZoom: [
    {
      type: 'inside',
      xAxisIndex: [0]
    },
    {
      type: 'inside',
      yAxisIndex: [0]
    }
  ],
  animation: false,
  brush: {
    brushLink: 'all',
    toolbox: ['rect', 'polygon' , 'clear'],
    seriesIndex: 'all',
    xAxisIndex: 0,
    inBrush: {
      opacity: 1
    },
    throttleType: 'debounce',
    throttleDelay: 1000
  },
  series: [
    {
      name: 'A',
      type: 'scatter',
      data: data1,
      dimensions: ['x', 'y'],
      symbolSize: 3,
      itemStyle: {
        opacity: 0.4
      },
      large: true,
    },
    {
      name: 'B',
      type: 'scatter',
      data: data2,
      dimensions: ['x', 'y'],
      symbolSize: 3,
      itemStyle: {
        opacity: 0.4
      },
      large: true
    }
  ]
};

The renderings are as follows:

Current Behavior
Does not get the value of the brush selection and has no visual effects
Expected Behavior
Can get the brush selected data and have a visual effect
Environment
- OS:
- Browser:
- Framework:

Any additional comments?
No response

Same issue here. Is there anybody on this ? :)

Yes, they are the same, because no reply, thought not to be seen

Brush selection doesnt seem to work with large data optimization which is enabled for both series (large: true). If large data optimization is disabled, the selection works as intended, though very slow compared to the zoom.
I would appreciate getting a zoom tool that either works with large data optimization or is better optimized to begin with.
Here is the working code:
function genData(len, offset) {
  let arr = new Float32Array(len * 2);
  let off = 0;
  for (let i = 0; i < len; i++) {
    let x = +Math.random() * 10;
    let y =
      +Math.sin(x) -
      x * (len % 2 ? 0.1 : -0.1) * Math.random() +
      (offset || 0) / 10;
    arr[off++] = x;
    arr[off++] = y;
  }
  return arr;
}
const data1 = genData(5e5);
const data2 = genData(5e5, 10);
option = {
  title: {
    text:
      echarts.format.addCommas(data1.length / 2 + data2.length / 2) + ' Points'
  },
  tooltip: {},
  toolbox: {
    left: 'center',
    feature: {
      dataZoom: {}
    }
  },
  legend: {
    orient: 'vertical',
    right: 10
  },
  xAxis: [{}],
  yAxis: [{}],
  dataZoom: [
    {
      type: 'inside'
    },
    {
      type: 'slider'
    }
  ],
  animation: false,
  brush: {
    toolbox: ['rect', 'polygon', 'keep', 'clear'],
  },
  series: [
    {
      name: 'A',
      type: 'scatter',
      data: data1,
      dimensions: ['x', 'y'],
      symbolSize: 3,
      itemStyle: {
        opacity: 0.4
      },
    },
    {
      name: 'B',
      type: 'scatter',
      data: data2,
      dimensions: ['x', 'y'],
      symbolSize: 3,
      itemStyle: {
        opacity: 0.4
      },
    }
  ]
};

But the display of large amounts of data would be very slow

Are you planning to improve the brush/selection tool in the near future?

I hope to solve this problem, some business sometimes need to use, it is best to add circular box selection, triangle box selection and so on.

I think the brush is slower than zoom, because the points in the selected region have to be rerendered/highlighted on every mousemove while the selection box is drawn, whereas the zoom is only calculated once on brushEnd. I would suggest to implement a brush behaviour similar to zoom if the largeThreshold is exceeded.
So it would be still useable & performant with large scale data.
Should I open a new feature request for that?

I think it is OK, depending on how much work you do. My business is mainly used for the Mapping of the wafer.

I have the same problem here

The following small change in the source code seems to work:
In BrushView.ts exchange the _onBrush method:
private _onBrush(eventParam: BrushControllerEvents['brush']): void {
    const modelId = this.model.id;

    const areas = this.model.brushTargetManager.setOutputRanges(eventParam.areas, this.ecModel);

    // Action is not dispatched on drag end, because the drag end
    // emits the same params with the last drag move event, and
    // may have some delay when using touch pad, which makes
    // animation not smooth (when using debounce).
    (!eventParam.isEnd || eventParam.removeOnClick) && this.api.dispatchAction({
        type: 'brush',
        brushId: modelId,
        areas: zrUtil.clone(areas),
        $from: modelId
    });
    eventParam.isEnd && this.api.dispatchAction({
        type: 'brushEnd',
        brushId: modelId,
        areas: zrUtil.clone(areas),
        $from: modelId
    });
}

by the following code:
private _onBrush(eventParam: BrushControllerEvents['brush']): void {
    const throttleType = this.model.option.throttleType;
    if (throttleType === 'debounce' && !eventParam.isEnd) {
        return;
    }

    const modelId = this.model.id;

    const areas = this.model.brushTargetManager.setOutputRanges(eventParam.areas, this.ecModel);

    // Action is not dispatched on drag end, because the drag end
    // emits the same params with the last drag move event, and
    // may have some delay when using touch pad, which makes
    // animation not smooth (when using debounce).
    (throttleType === 'debounce' || !eventParam.isEnd || eventParam.removeOnClick) && this.api.dispatchAction({
        type: 'brush',
        brushId: modelId,
        areas: zrUtil.clone(areas),
        $from: modelId
    });
    eventParam.isEnd && this.api.dispatchAction({
        type: 'brushEnd',
        brushId: modelId,
        areas: zrUtil.clone(areas),
        $from: modelId
    });
}

using the throttleType property (as it is intended if i understand correctly) to enable an "only render on release" mode.
Now use the brush component with debounce mode:
brush: {
    throttleType: 'debounce'
}

And enable large data optimization mode for your series and set the largeThreshold to a value larger than the number of items in your series (dont ask me why but it works this way):
series: [
    {
        type: 'scatter'.
        ...,
        large: true,
        largeThreshold: 99999999,
    },
    ...
]


Fix duplicate ENV statement in several dockerfiles

Several of the Dockerfiles contained the following line in duplicate:
ENV DEBIAN_FRONTEND=noninteractive

As such, I removed the second instance of each line, including the preceding comment.
I think this occurred because the mistake was included in the template Dockerfile.

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

The reason the first line is present is to eliminate errors during the docker build.  However if you leave this on, then if you manually install anything once in the container, you won't be prompted, so it reverts it back to dialog when done.

Given my explanation above,  I don't think we'll end up being able to take this PR.  Thanks so much for the contribution though and let me know if you don't think I've understood what you were trying to say here.  Thanks!


InvalidTable,  Unknown table name 'Comments'

having a test case file like
*** Setting ***
Library           SeleniumLibrary

*** Comments ***
    blabla bla

*** Keywords ***
kw blabla bla
    blabla bla...

*** Test Cases ***
blabla bla

When is executed:
rflint --recursive --ignore all --argumentfile .\rflint.args testdirectory
was verified:

❌  wrong validation InvalidTable, Unknown table name 'Comments'

notes
rflint --version
1.0
❌  rflint --version returns version 1.0 but pip returns robotframework-lint   1.1
pip list
Package                        Version
------------------------------ -------
astroid                        2.4.1
autopep8                       1.5.3
colorama                       0.4.3
isort                          4.3.21
lazy-object-proxy              1.4.3
mccabe                         0.6.1
pip                            20.1.1
pycodestyle                    2.6.0
pylint                         2.5.2
PyYAML                         5.3.1
robotframework                 3.1.2
robotframework-lint            1.1
robotframework-pabot           1.8.0
robotframework-seleniumlibrary 4.3.0
robotremoteserver              1.1
selenium                       3.141.0
setuptools                     47.1.1
six                            1.15.0
toml                           0.10.1
typed-ast                      1.4.1
urllib3                        1.25.9
wrapt                          1.12.1

Hi @JoaoEnesGoncalves
rflint version 1.1 knows about the Comments table.
If rflint --version and pip list give two different versions, it means you have 2 different installations and you're not running the correct one when executing rflint.
You use a backslash in one of the commands, does that mean you're using Windows? I'm not familiar with Windows, but apparently you can use where rflint to find what command is run when you type rflint. Compare with the output from pip show --files robotframework-lint.

thank you for feedback
yes, was a problem on differents users installation versions on Windows10, resolved!


Room plotting issues with scheduling sessions in rooms.

Nothing prevented you from putting multiple sessions into one room like it did in Drupal 6.

There are two ways to go about adding a session to a room for a particular time.  When I attempted to use the Scheduler page (admin/conference/1/schedule), it wouldn't allow me to add it... BUT when I edited the Session directly and selected the same time and room as a booked room, it allowed me to..
Problem is.. when I return back to the scheduler page, I don't see the second session that was added.  Where did it go?
Need to check COD's issue queue to see if this has been addressed.   Could be a potential problem.


Error handling consistency/conventions

I ran into this a while back and never raised it, but since I'm here and contributing to the error handling code (#128 #126 #118) I thought I'd bring it up now.
Errors should be instanceof Error
I would have expected the errors provided in the catch(err) callback to have been instanceof Error, as is suggested by the MDN docs for Promise.reject(reason):

For debugging purposes and selective error catching, it is useful to make reason an instanceof Error.

Indeed the convention for the continuation-style callback passing cb(err) is widely recognised, supported and provided by module authors (supporting material: 1,2).
Error handling should be consistent
In debugging and fixing #126, I discovered different error handling strategies for:

download request
RPC request
upload request

The difference being that the error response json is parsed in the RPC request, but not the others. I believe that this inconsistency was introduced by accident rather than design (in the fix to #108).
Solution
I think the solution should be to extract the error construction into a common function that is used by each request type. This will ensure consistency between request type.
The error constructed should be an Error object, and I see no reason why the information that is attached to the current non-Error object format could not be attached.
For example (just a quick sketch, not an exhaustive solution), something like:
var buildCustomError = function (error, response) {
  var msg;
  if (response) {
    try {
      msg = JSON.parse(response.text).error_summary;
    } catch (e) {
      msg = error.message;
    }
  }
  var err = new Error(msg);
  err.status = error.status;
  err.response = response;
  return err;
};

Let me know what you think. I'd be happy to put together a PR that implements this solution, or leave it with you to implement. Cheers.

Thanks! We'll take a look.

would be very nice to see error handling in some /example

Any improvements regarding consistent errors are appreciated. They should be marked as a breaking change. The error structure is so inconsistent that a refactor and testing will be needed after the proposed structural changes
-=Dan=-

Thanks for the feedback!

I definitely agree that every error thrown should be an instanceof Error.
In our case, we are using koa, and using an app.on('error'... callback.  Because the error thrown by Dropbox is not an instanceof Error, koa actually creates a new error message: https://github.com/koajs/koa/blob/162a5b3e78bf0838cee67119804f066d6e38bf02/lib/context.js#L112 which has us lose the actual JS object info for status, error, etc.
If there is sufficient interest in a fix for this, I could put in a PR.


Lower Accuracy Even with Batch norm.

Hi thanks for the amazing explanation of the implementation. However I failed to get the reported accuracy reported in the BLOG. With batch Norm I get an accuracy of around 36% which is also visible in the Notebook you have posted.

But on the second block. The accuracy becomes 70% somehow. Please Let me know if I am missing something.

Hello,
Actually it seems that if you fix incorrect ordering of loss, as commented in issue " Incorrect Specification of loss function. #1 ", it will work like in the blog and will get correct accuracy.
It worked for me.


[native_assets_cli] Disallow any other keys in the output file

In a discussion with @jonasfj earlier, we discussed to disallow any other keys in the output file in order to ensure package authors implementing the protocol themselves don't prevent us from adding keys later.
Implementing this would mean that end users always need the newest Dart / Flutter SDK if the native_assets_cli used in their build.dart is new, otherwise it might output a key that is not yet accepted by the Dart / Flutter SDK. This might be an issue while we develop the package because we need to release new versions of native_assets_cli and native_assets_builder on pub before they can roll into flutter_tools (it doesn't use DEPS, but a normal pubspec.yaml). And it takes a couple of months for commits to reach Flutter and Dart stable.
Essentially, disallowing other keys in the output file makes adding keys to the output file a breaking change. I believe it's better to avoid causing breaking changes. So maybe it's better to not disallow other keys in the output file. wdyt @jonasfj

See comment https://github.com/dart-lang/native/issues/93#issuecomment-1642393710


Variables are not being updated in tool scripts after they were edited

Godot version: Godot Engine v4.0.dev de3747fc3

OS/device including version: Windows 10 64-bit

Issue description:

Scripts marked with @tool remember the state of variables and refuse to update them when script was edited.

Restarting editor doesn't help. What helps:

removing @tool and adding it back
marking variables with @onready

Also tested on stable v3.3, couldn't reproduce the bug there
UPD: Its seems that Godot writes variables of tool scripts into .tscn file, even though there is no @export

[ext_resource path="res://Main.gd" type="Script" id=1]

[node name="Node" type="Node"]
script = ExtResource( 1 )
I = 42
D = {
"key4": [ 1, 2, 3 ],
"key5": [ 1, 2, 3 ]
}

Steps to reproduce:

Create a script
Declare and initialize some variables, print their values
Run scene with this script
Mark script with @tool
Change variables value
Run script once more
Observe that values haven't changed

OR

Open Minimal reproduction project
Try changing variable values in Main.gd and run the project

Minimal reproduction project: ToolUpdateBug.zip

UPD: Its seems that Godot writes variables of tool scripts into .tscn file, even though there is no @export

This is a known issue, see https://github.com/godotengine/godot/issues/48224.

This is likely the same as #48224

Closing per @TechnoPorg's comment.


METRON-1695: Expose pcap properties through Ambari

Contributor Comments
This change makes the PCAP config properties available via the Metron service -> Configs tab in Ambari. It also takes care of updating the pcap.properties file with relevant values depending on the deploy environment (single node vs. multi node vs. kerberized cluster).
Here's a screenshot of the PCAP Config:

Testing Steps

Launch full dev
Metron PCAP service should come up successfully.
Go to Ambari -> Metron services -> Configs -> PCAP tab; change any config parameter;
Restart the required services and the config change take effect.

Testing Done
I ran the following different tests to validate the fix.
A. Full Dev

Verified Metron PCAP service comes up successfully
Changed some of the PCAP config values from the Ambari UI, validated that this is reflected in the pcap.properties file upon service restart.

Note: I ran into METRON-1786 during testing. I was able to resolve the issue by applying the fix in PR #1206

B. Multi-node Cluster

Validated that the kafka.zk property in pcap.properties is updated with the correct Broker details
Changed a PCAP service-related property (E.g. kafka_pcap_numpackets) and validated that the pcap.properties file is updated.
Changed a PCAP REST-related property (E.g. pcap_page_size) and validated that the /etc/default/metron file is updated.

C. Kerberized Cluster

Kerberized the multi-node cluster and setup pycapa
Validated that topology.worker.childopts and topology.auto-credentials are
Ingested PCAP into the cluster
Ran queries through the PCAP panel and validated that the modified config settings are in effect (E.g. pcap_page_size set to 50).

Pull Request Checklist
Thank you for submitting a contribution to Apache Metron.
Please refer to our Development Guidelines for the complete guide to follow for contributions.
Please refer also to our Build Verification Guidelines for complete smoke testing guides.
In order to streamline the review of the contribution we ask you follow these guidelines and ask you to double check the following:
For all changes:

[x] Is there a JIRA ticket associated with this PR? If not one needs to be created at Metron Jira.
[x] Does your PR title start with METRON-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.
[x] Has your PR been rebased against the latest commit within the target branch (typically master)?

For code changes:

[x] Have you included steps to reproduce the behavior or problem that is being changed or addressed?

[x] Have you included steps or a guide to how the change may be verified and tested manually?

[NA] Have you ensured that the full suite of tests and checks have been executed in the root metron folder via:
mvn -q clean integration-test install && dev-utilities/build-utils/verify_licenses.sh

[ ] Have you written or updated unit tests and or integration tests to verify your changes?

[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

[ ] Have you verified the basic functionality of the build by building and running locally with Vagrant full-dev environment or the equivalent?

For documentation related changes:

[x] Have you ensured that format looks appropriate for the output in which it is rendered by building and verifying the site-book? If not then run the following commands and the verify changes via site-book/target/site/index.html:
cd site-book
mvn site

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.
It is also recommended that travis-ci is set up for your personal repository such that your branches are built there before submitting a pull request.

This is causing one problem on Full Dev. We have 6 slots in Full Dev, but since adding the Pcap Topology, we now require 7 slots for all of the topologies to run.
What ends up happening is that the 'random_access_indexing' often gets starved.  When spinning up Full Dev, no alerts are visible in the Alerts UI.  We want alerts to be immediately visible as this is a standard validation step.
We need to figure out a solution around this, which I am not sure what that looks-like right now.  Add a slot?  Do not immediately start 'pcap' topology?  Other?

We have an overall deficiency here I believe.  The Slot management is a oft forgotten and manual task, for adding new parsers from the ui etc, you ( I believe ) still have to remember to have already added a slot before starting your parser topology for example.  We may want to consider an overall underlying approach to this that is usable where we fall short.

@ottobackwards The Slot management is a oft forgotten and manual task, for adding new parsers from the ...

Sure, worthy of a discuss.
@anandsubbu I would just drop "snort" as a default for the parser configuration in Ambari.  Then we save a slot and get around this issue for now.  As a follow-on, we could explore other alternatives.

Might be worth looking at how we set what spins up by default. For instance, we might not want Pcap to start  by default in any installation, let alone full dev. Is there an option in the mpack config for one, or two, is there a way to disable it via our blueprints? This way we don't alter the existing set of topologies. One reason for this is that it's at least mildly useful to have 2+ sensors worth of data writing out to ES that can be viewed in Kibana. And while pcap doesn't go to indexing anyhow, we don't have anything generating data into the pcap topology currently at all. It would just be taking up a slot and doing nothing.

@anandsubbu I am also running into a separate issue.  After some period of time Ambari will start reporting the status of the pcap topology as stopped, even though it remains running in Storm.  Are you able to replicate this?  I am digging through the logs trying to find some sort of stack trace, but have not found anything yet.

@mmiklavc I agree with that, but I also think that we should separate the default sensors from the regular metron install, and have them be their own optional thing.  IE> you would choose to install the snort , yaf, bro things we install automatically now.

Latest commit modifies full dev to start only the bro parser topology by default. The PCAP topology now gets the worker slot which was freed up. Note that on a regular deploy, all of bro, snort and yaf topologies will be deployed by default.
I am also looking for the issue @nickwallen mentioned about the PCAP topology appearing stopped. Nothing so far. Will continue to monitor.

PCAP related properties needs to be taken out from the rest_application. Also the dependency of PCAP on rest settings to be removes as Changing a REST Setting triggers Ambari to Restart PCAP Topology .

@nickwallen suggested a great idea of making use of parser aggregation and thus starting the default parsers on full-dev as an aggregated parser topology. The latest commit makes use of PR #1215 which provides that ability.
Now, on full-dev we start a single aggregated topology for the default configured sensors viz. "bro,snort,yaf".
Testing Done

Spun up full-dev and noticed that a single topology named bro__snort__yaf is started
Verified Alerts UI has data from all three sensors:

+1 Thanks @anandsubbu !

Thanks much @nickwallen for giving it another spin!
Hey @MohanDV , let me know if my below explanation makes sense. @nickwallen , would you have any thoughts on this?

@MohanDV , as a part of my commit, I have moved all the UI related elements from REST to the PCAP config tab.
In my understanding, the following parameters are not directly related to the PCAP core service, but are functionally related to REST (used by the PCAP query panel UI):

pdml.script.path
base.path
base.interim.result.path
final.output.path
page.size
yarn.queue
finalizer.threadpool.size

For this reason, I did not move these properties out of the rest_application.yml.

@MohanDV Changing a REST Setting triggers Ambari to Restart PCAP Topology

With my current changes, modifying any config setting in the PCAP tab will require restarting both Metron PCAP and Metron REST services--which is necessary since user can change a PCAP-service related property or a PCAP-REST related property (or both). The converse, which you noted, i.e. changing any REST config does not require restarting the PCAP topology. Here's a screenshot after making changes to REST config...

Cool, thanks for your opinion @nickwallen . I will wait for @MohanDV 's thoughts as well before merging.

Cool, thanks for your opinion @nickwallen . I will wait for @MohanDV 's thoughts as well before merging.

+1 Thanks @anandsubbu

Thanks @MohanDV !


RTCView is falling behind other RTCView?

I encountered this problem, on Android only. When I create a container over the rtcView, it's ok, but when I put some RTCViews in that container, they're falling behind the other RTCView. Did anyone experienced this to? And does anyone have a solution for this? It's strange, because this only happens when i'm running my app on Android. On iOS it is all ok. You can see at the left of the picture, some stream is shown partially, the rest is falling behind the other RTCView.

I think I solved the order by adding a zIndex property on RTCView and, if I remember correctly, that was needed because of Android. Please read the source code of RTCView for details.

Thanks! This solved my problem! :)
Needed to add a zOrder property to my RTCViews.


Add empty swagger indy-rest-api.json/yaml files

Indy build fails if with -DskipTests
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-install-plugin:2.4:install (default-install) on project indy-rest-api: Failed to install artifact org.commonjava.indy.rest:indy-rest-api:yaml:1.3.0-SNAPSHOT: /home/ruhan/github/commonjava/indy/rest/api/target/indy-rest-api.yaml (No such file or directory)
I add two empty files for those. These files with be overridden when -Prun.it is enabled.

NCL devel builds use -DskipTests so they won't have swagger. Stage/prod use Indy formal release and the swagger should be Okay.


Fix multinode CI

Subj

There is a problem that virsh --all somehow does not work after the session is over; probably have to mount some other volume for the host for it to work

Here is a test result, https://github.com/aregm/icl/actions/runs/9915590474/job/27396680921


Add border radius to left/right panel

Adds curvature to top-right corner of left-panel, and top-left corner of right-panel which highlights separation more to show the collapsible nature with new hide panel button.

removes unnecessary padding to better align left and right panel top edges visually.

@thadguidry is there a corresponding issue that we could link to this PR?

No sir


[SSR] SyntaxError: Unexpected token export， babel-loader 已经编译 node_modules 中的 es6 文件

Version
2.6.10
Reproduction link
https://github.com/JackFGreen/vue-ssr-demo
Steps to reproduce

yarn build && yarn prod
http://localhost:3000/
logs/app.log

What is expected?
调用 vendors 中编译后的 demo-lib
What is actually happening?
调用了 node_modules/demo-lib 中的 es6 源码

Hi, thanks for your interest but Github issues are for bug reports and feature requests only. You can ask questions on the forum, the Discord server or StackOverflow.


Lmfit model for xrf

The fitting model for xrf is included. It is still in process, and some file structure might need to be re-organized.

Issue solved due to the update of lmfit to 0.8.3rc1.

@licode State of this?

This is ready to review.

Thanks for the comments. This PR does stay here too long...

@tacaswell , more corrections are added. Ready for review again, or it may be ready to go? thanks.

Coverage improves a bit. But more tests are needed.

Remove all "Advanced Mode" from this PR and submit a new PR with the advanced features.

The idea of using fitting strategy as separate dict is adopted, but more correction is needed on my side.

I think this one is ready to review again. The total coverage is increased by almost 10% :)

Our original idea is to remove ParamController class. Since @tacaswell has a good idea on parameter controller, I merged his PR. So the whole thing still stays together. But the structure is much clear now. Hope that everyone would agree with that, and we can get this one merged soon. So I can move on.

This is getting closer, for sure. I left a few line comments, and I see other from @tacaswell and @ericdill that still need to be addressed.
Would you write a quick example script illustrating how the methods like set_default and the various set_* and get_* methods are meant to be used? It doesn't need to be formal code -- just post it as a comment in this PR to start.

thanks, will do.

I added test_set_param_hint function in https://github.com/licode/scikit-xray/blob/lmfit_model/skxray/tests/test_xrf_fit.py. Other set_* or get_* are basically included in the same test file.

Any more comments on this PR? I hope this is ready to go?

Any more comments on this PR? I already merged all the corrections.

Looks great. @tacaswell concurs, so I'm pulling the trigger. Hurray!

:beers:


add copy of data to binary output folders for running all tests

When I ran "rtneural_tests all" it failed because the data was not on the relative path of the executing test.  this copies it to both /tests and to the root of the binary output in case the tests are run from that dir as well

Thanks for the contribution!
That said, I was actually planning to rework the way that the tests access the relevant data. Basically, the idea would be to define the RTNeural root directory as a compile definition:
target_compile_definitions(${example_name}
    PRIVATE
        RTNEURAL_ROOT_DIR="${CMAKE_SOURCE_DIR}"
)

Then the tests (and examples and benchmarks) could always find the files they need relative to the provided path.
Anyway, I haven't gotten around to implementing this yet, but I think that might be the most robust solution.
What do you think? I believe that solution would fix the problem that you ran into, but maybe there's something that I'm not seeing.

I would have done it exactly the way you have suggested, however, it was unclear why the final binary was also copied for the test as well and so I chose this approach instead to service both binaries and to do something quickly.
Also, I think would also do a different initial setup and pattern for tests.. so in the top level CMakeList.txt begin to collect the tests in this pattern ..
#initialize ctests framework at the top level once
include(CTest)
enable_testing()
#add the tests dir
add_subdirectory(tests)
#add test command
add_test(NAME "RTNeural accuracy tests" COMMAND rtneural_tests all)
this then percolates all the tests and you can, for example , run all of them using
make test
as part of the normal build procedure
you can use this also to enforce tests to run when considering pull requests as another benefit.
I can make the changes and submit another PR ?
Kind Regards
P

Yeah, using CTest is a good suggestion as well. I believe CTest also provides a way to run tests in parallel which could help speed up the CI pipelines.
I would just add to your suggestion, that we should only do include(CTest) and enable_testing() if the BUILD_TESTS CMake option is set.
If you'd like to make a PR with both changes, that would be fantastic! (or you could update this PR, your choice)
The intention behind copying the tests binaries to a dedicated folder was to avoid confusion due to CMake placing the binaries in different locations depending on the CMake generator or build configuration being used. (Although I suppose this would be less of an issue when using CTest)
Thanks!

So I ended up setting up CTest with individual test commands (not the all command), and that seems to be working well (#112). I haven't tried doing the JSON dump yet, but that sounds quite interesting as well!


🌱 chore: update go version in release test workflow

What this PR does / why we need it:
Release workflow is failing after the move to Go 1.22. This PR explicitly configures the action to run the new Go version.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Checklist:

[x] squashed commits into logical changes
[ ] includes documentation
[ ] adds unit tests
[ ] adds or updates e2e tests

There are a couple of CI errors:

E2E should be more stable when https://github.com/rancher-sandbox/rancher-turtles/pull/421 is merged.
Linter errors should be fixed when https://github.com/rancher-sandbox/rancher-turtles/pull/396#pullrequestreview-1919944347 is merged.


there is no space in the text between the sentences

Description
there is no space in the text between the sentences
(Optional:) Please add any files, screenshots, or other information here.

(Required) What is this issue most closely related to? Select one.
Choose One
Internal issue ID
a4f9c777-33c2-4cd5-96a0-6e057bbd37f6
Paper URL
https://arxiv.org/html/2312.10997v5
Browser
Chrome/<IP_ADDRESS>
Device Type
Desktop

This is the same in the PDF I believe? Please contact the authors directly for questions on document content.
This repository only tracks the quality of the experimental HTML format.


uswds alignment

Is this just ensuring the use of USWDS for our site's style system? Title isn't clear to me.


Add new virtual method QPDFObject::getStringValue

Avoid dynamic casting.
I see an improvement of approx 0.5%.

@jberkenbilt  The CI failure doesn't look to me to be a problem with the commit.
+ sha256sum distribution/qpdf-11.2.1-doc-ci.zip
4cf4055d2a4f4918b801e039e8e27f88454d94d1f14bb99a4344d7921502809a  distribution/qpdf-11.2.1-doc-ci.zip
Traceback (most recent call last):
  File "/home/runner/work/qpdf/qpdf/build-scripts/download-external-libs", line 37, in <module>
    [(r['tag_name'], r) for r in releases
  File "/home/runner/work/qpdf/qpdf/build-scripts/download-external-libs", line 38, in <listcomp>
    if r['prerelease'] is False],
TypeError: string indices must be integers

The CI failure doesn't look to me to be a problem with the commit.

Looks like a transient failure with github. I reran the jobs.

Looks like a transient failure with github. I reran the jobs.

Thanks

I see an improvement of approx 0.5%.

Or it could be 1.5%. Who knows. I feel reasonably confident that 11.3 will be over 1.35 times faster than 11.2 and well over twice as fast as 10.6 (excluding JSON tests) but for anything under 1% over 20 repetitions  the significance of what looks like an improvement is difficult to judge. I think I'll open a discussion topic on performance testing.

I think I'll open a discussion topic on performance testing.

There's an existing discussion on Windows performance. You might want to mention your discussion there.


Cannot test by Chrome browser

The test by Chrome browser fails because Chrome browser does not allow third party's cookie in private mode.

Does this manifest itself via a login failure? That's what I'm seeing when trying to use automated tests with chrome.
e.g.
Failed   RunTestAutomation
Error Message:
Test method Microsoft.PowerApps.TestAutomation.Tests.TestAutomation.RunTestAutomation threw exception:
System.InvalidOperationException: Login failed after 3 attempts. Exception Details: System.Exception: Login page failed on login retry.
at Microsoft.PowerApps.TestAutomation.Api.OnlineLogin.<>c.b__12_1(IWebDriver f) in D:\a\1\s\Microsoft.PowerApps.TestAutomation.Api\Pages\OnlineLogin.cs:line 188

@Benelio , @karamem0 ,
May I know where i can find the steps to execute a test in this repo -microsoft/PowerAppsTestAutomation?
Once after cloned this code, what are the steps i should follow to execute a sample test?
Do we have any documentation (Getting started page is blank) ,if not Could you please guide how you guys executed the tests in this repo?

Hi @itsmyownview,
This document will be useful to you.
https://docs.microsoft.com/en-us/powerapps/maker/canvas-apps/test-studio-classic-pipeline-editor

Built pipeline following https://docs.microsoft.com/en-us/powerapps/maker/canvas-apps/test-studio-classic-pipeline-editor
However for both Chrome as well as Firefox, it always fails with following error;
Failed   RunTestAutomation
Error Message:
Test method Microsoft.PowerApps.TestAutomation.Tests.TestAutomation.RunTestAutomation threw exception:
OpenQA.Selenium.ElementNotInteractableException: Element  is not reachable by keyboard
Any idea how can I resolve this issue?


rpc tests change things in the repository

rik@bilouchun:neo-go$ git status
On branch mastev
nothing to commit, working tree clean
rik@bilouchun:neo-go$ make test
?       github.com/CityOfZion/neo-go/cli        [no test files]
?       github.com/CityOfZion/neo-go/cli/server [no test files]
?       github.com/CityOfZion/neo-go/cli/smartcontract  [no test files]
?       github.com/CityOfZion/neo-go/cli/vm     [no test files]
?       github.com/CityOfZion/neo-go/cli/wallet [no test files]
?       github.com/CityOfZion/neo-go/config     [no test files]
?       github.com/CityOfZion/neo-go/examples/engine    [no test files]
?       github.com/CityOfZion/neo-go/examples/iterator  [no test files]
?       github.com/CityOfZion/neo-go/examples/runtime   [no test files]
?       github.com/CityOfZion/neo-go/examples/storage   [no test files]
?       github.com/CityOfZion/neo-go/examples/token     [no test files]
?       github.com/CityOfZion/neo-go/examples/token/nep5        [no test files]
?       github.com/CityOfZion/neo-go/examples/token-sale        [no test files]
ok      github.com/CityOfZion/neo-go/pkg/core   3.197s  coverage: 48.7% of statements
ok      github.com/CityOfZion/neo-go/pkg/core/storage   (cached)        coverage: 22.1% of statements
ok      github.com/CityOfZion/neo-go/pkg/core/transaction       (cached)        coverage: 36.5% of statements
ok      github.com/CityOfZion/neo-go/pkg/crypto (cached)        coverage: 57.6% of statements
?       github.com/CityOfZion/neo-go/pkg/interop/account        [no test files]
?       github.com/CityOfZion/neo-go/pkg/interop/asset  [no test files]
?       github.com/CityOfZion/neo-go/pkg/interop/attribute      [no test files]
?       github.com/CityOfZion/neo-go/pkg/interop/block  [no test files]
?       github.com/CityOfZion/neo-go/pkg/interop/blockchain     [no test files]
?       github.com/CityOfZion/neo-go/pkg/interop/contract       [no test files]
?       github.com/CityOfZion/neo-go/pkg/interop/crypto [no test files]
?       github.com/CityOfZion/neo-go/pkg/interop/engine [no test files]
?       github.com/CityOfZion/neo-go/pkg/interop/enumerator     [no test files]
?       github.com/CityOfZion/neo-go/pkg/interop/header [no test files]
?       github.com/CityOfZion/neo-go/pkg/interop/input  [no test files]
?       github.com/CityOfZion/neo-go/pkg/interop/iterator       [no test files]
?       github.com/CityOfZion/neo-go/pkg/interop/output [no test files]
?       github.com/CityOfZion/neo-go/pkg/interop/runtime        [no test files]
?       github.com/CityOfZion/neo-go/pkg/interop/storage        [no test files]
?       github.com/CityOfZion/neo-go/pkg/interop/transaction    [no test files]
?       github.com/CityOfZion/neo-go/pkg/interop/util   [no test files]
?       github.com/CityOfZion/neo-go/pkg/io     [no test files]
ok      github.com/CityOfZion/neo-go/pkg/network        (cached)        coverage: 14.3% of statements
ok      github.com/CityOfZion/neo-go/pkg/network/payload        (cached)        coverage: 56.6% of statements
ok      github.com/CityOfZion/neo-go/pkg/rpc    0.085s  coverage: 46.0% of statements
?       github.com/CityOfZion/neo-go/pkg/rpc/result     [no test files]
?       github.com/CityOfZion/neo-go/pkg/rpc/wrappers   [no test files]
ok      github.com/CityOfZion/neo-go/pkg/smartcontract  (cached)        coverage: 35.5% of statements
ok      github.com/CityOfZion/neo-go/pkg/util   (cached)        coverage: 53.4% of statements
ok      github.com/CityOfZion/neo-go/pkg/vm     (cached)        coverage: 46.2% of statements
?       github.com/CityOfZion/neo-go/pkg/vm/cli [no test files]
ok      github.com/CityOfZion/neo-go/pkg/vm/compiler    (cached)        coverage: 69.4% of statements
ok      github.com/CityOfZion/neo-go/pkg/vm/tests       0.206s  coverage: 0.0% of statements
?       github.com/CityOfZion/neo-go/pkg/vm/tests/bar   [no test files]
?       github.com/CityOfZion/neo-go/pkg/vm/tests/foo   [no test files]
?       github.com/CityOfZion/neo-go/pkg/vm/tests/foobar        [no test files]
ok      github.com/CityOfZion/neo-go/pkg/wallet (cached)        coverage: 50.4% of statements
rik@bilouchun:neo-go$ git status
On branch mastev
Changes not staged for commit:
  (use "git add/rm <file>..." to update what will be committed)
  (use "git checkout -- <file>..." to discard changes in working directory)

        deleted:    pkg/rpc/chains/unit_testnet/000024.log
        deleted:    pkg/rpc/chains/unit_testnet/000026.ldb
        modified:   pkg/rpc/chains/unit_testnet/CURRENT
        modified:   pkg/rpc/chains/unit_testnet/LOG
        deleted:    pkg/rpc/chains/unit_testnet/MANIFEST-000025

no changes added to commit (use "git add" and/or "git commit -a")

This is wrong, testing shouldn't change anything in the repository.

https://github.com/nspcc-dev/neo-go/blob/c67217159fd4f13263e7f70e7a142fa449c853c3/pkg/rpc/server_test.go#L242
This test is more e2e rather then unit test since it starts neo go node server. That's why it created same package with storing chains. And it modifies previous files CURRENT and LOG which were used in previous test run.
Solution could be:

Add unit_testnet to .gitignore. Yes each test run locally will modify previous test's run results.
Make clean up of the repo("unit_testnet" before each test run and after). But cleaning them after test run won't allow us to figure what happened wrong in case we want to do that.

Suggest to do 1. and first part of 2.
WDYT @roman-khimov ?

I think at least it should always start from the well-known clean state, so just ignoring these changes seems wrong to me. Some thoughts:

making an archive of this chain and then unarchiving before the test can help, but it might be non-trivial to reliably clean up after the test run
making some use of in-memory DB (rather than LevelDB), prefilling it in some way and then running RPC calls against it

Saving DB state after the test run shouldn't be the priority IMO as this test (and its failures) should be reproducible from the well-known state.

BTW,
The go tool will ignore a directory named "testdata", making it available
to hold ancillary data needed by the tests.

Not solving the problem, but something to keep in mind if there is some file-level storage needed.

The idea to have an archive as a starting point for testing is the one that it is followed by neo-python:
https://github.com/CityOfZion/neo-python/blob/master/neo/Utils/BlockchainFixtureTestCase.py#L210
I would suggest doing the same.
Basically, before running the RPC-test cases we would need to unarchive to a specific test directory and one the test cases are completed we would need to destroy the test directory.
In addition, FYI, the test blockchain used in the RPC test cases is coming from the neo-python-privnet-unittest image. So there are quite a lot of test cases that we can implement in neo-go based on neo python

Yup, I see. In this case it looks like more e2e test rather than unit. And I believe it should be separated to different package. And even having separate CI step to run it would make more sense for me.
I see this as separate step with running such tests agains dockerized env after:
build
unit tests
As of now I'm reworking it to look more like component test(not starting real DB with real files but having in memory DB with pre-setup values in test).


Custom modal dialogs partially broken in 0.4.0

It seems that the custom modal dialogs are partially broken in nativescript-angular 0.4.0 and Angular2 RC5.  I say partially broken because the custom modal appears to work fine when loaded in the first child NgModule, but in other modules I receive the following error from page.ios.ts (here):
Parent page is not part of the window hierarchy. Close the current modal page before showing another one!
I've created a sample based on the Ng2-RC5 branch of sample-groceries, and my sample can be found here.  I'm currently showing the modal in a setTimeout after 5 seconds to quickly show the problem and not bother with modifying the sample's UI.  In the login module, everything works as expected; however, I see the error in groceries module.  I have only tested iOS and don't know if this is a problem on Android devices.

I think the problem is related to navigation (may be it is the same you are thinking), checkout here: https://github.com/NativeScript/NativeScript/issues/2759

Thanks @ignaciolarranaga!  I was hoping this was fixed with the latest changes to the ModalDialogService, but I haven't had a chance to check yet.  By looking at your issue and sample, it seems this is not the case.

@rrjohnson85, check out the last comment in here #459, may be it is also your problem.

@ignaciolarranaga Thanks, but I don't think that is the problem since I added the ModalDialogService to providers in the root module.  I didn't try creating a core module, but I don't think that will change anything.
It could be that the problem is fixed by the latest changes to the ModalDialogService, but I haven't been able to check.

I added the ModalDialogService to providers in the root module

You should add it to every module that opens a dialog page. If you have just one provider in your root module, a page-router-outlet page that tries to open a new dialog will get the wrong parent page, and likely crash.
I just scheduled #460 for the current milestone so we document all that.

Thanks @hdeshev!  I added the ModalDialogService to each page that opens a modal - the problem seems to be resolved.  Closing this ticket.

@rrjohnson85 We have done some refactoring of the way the dialogs are shown in #489. Summary:

No need to use the modal-dialog-host directive anymore.
No need to provide ModalDialogService in your module or component. It is now exported by the NativeScriptModule.
You should inject a ViewContainerRef and pass it in the options object when calling showModal. This eliminates the need of adding modal-dialog-host in your template and makes the whole process more explicit and less error-prone. It is also the current approach in the angular-material2 dialog.


Initial Travis CI configuration.

This commit introduces a Travis CI configuration for Levigo, which
will setup a semi-deterministic test environment for Levigo itself,
including dependencies and does not rely upon brittle assumptions of
Travis outside of build essentials and curl.
It is a child of https://github.com/jmhodges/levigo/pull/28.

Thank you so much for this! I didn't do right by you by leaving you hanging, but with just a few small changes, levigo now has a CI system


Manual installation instructions and iOS 7 support

Our app is still required to support iOS 7, so CocoaPods / Carthage are not considered as options. Could you please give any guidance on manual installation of the library? Or would you recommend against using Moya in this case? Thank you!

Add the .xcodeproj to your project, and then link the Moya.framework output against your project.
Let me know if you have any issues or need more clarification.

But doing that wouldn't produce a framework as well?
Which are not available to be used on 7 iirc?
--
Esteban Torres
(+506)8813-0934
Twitter: @esttorhe
On Wed, Aug 19, 2015 at 6:13 AM -0700, "Justin Makaila"<EMAIL_ADDRESS>wrote:
Add the .xcodeproj to your project, and then link the Moya.framework output against your project.
Let me know if you have any issues or need more clarification.
—
Reply to this email directly or view it on GitHub.

@justinmakaila, @esttorhe: In iOS 7 we cannot use dynamic frameworks, so probably there's no way to install Moya and neither ReactiveCocoa (which seemingly dropped iOS 7 support at some point). Please let me know, if I'm wrong.

Oh, sorry, just woke up. Skimmed it and missed that part ;)
You can just add the source files straight to the project.

@justinmakaila: It was the first thing that I did. : ) But then you need to add ReactiveCocoa, which is not possible for iOS 7. From their Readme page:

ReactiveCocoa supports OS X 10.9+ and iOS 8.0+.

I realized this limitation only after asking the question. Seems that the Moya is an iOS 8+ library as well.

Maya itself should be usable without frameworks, but would require modification.
Moya is built atop of Alamofire, so you'd need to have those .swift files in your project. Then you'd drop in the Moya .swift files, but that's not enough. Moya has import Alamofire statements that need to be removed first. Then in should work.
A similar process should work for ReactiveCocoa, but @mikejd you're right – RAC 3 is iOS 8+ only.

@ashfurrow Thank you for clarification! I realized that there's some manual way to get things work, but it seemed tedious to try. Finally we decided to simply drop iOS 7 support, so this no more an issue.

Cool, glad to hear it worked out! :tractor:


Remove Deprecated Union Types

This PR removes support, documentation, tests, and examples using the deprecated union types.
Almost all of this change is either from generated code (from antlr) or mechanical removal of code for handling these types.
The only notable exceptions are

6ba5791 which updates Paxos.qnt to use sum types
ba9c25a which updates lang.md, to document sum types and their use

A forthcoming PR will expand on the documentation in other needed places.

[x] Tests added for any new code
[x] Documentation added for any new functionality
[x] Entries added to the respective CHANGELOG.md for any new functionality
[ ] Feature table on README.md updated for any listed functionality

in Apalache we have VariantFilter, which I don't think we can implement in quint

I think we could do so, it would just require exposing the extensibilty of the underlying extensible variants. This may be as simple as removing the isDefined check from our constraints and adding an equivalent operator, but it goes against the decision for simplicity reached in the course of discussion around the sum type RFC.  This is discussed in https://github.com/informalsystems/quint/blob/main/doc/rfcs/rfc001-sum-types/rfc001-sum-types.org#why-not-support-polymorphic-variants and the generalization is presented in the immediately preceding section.


Cloud Credential - 'Region' field is not a mandatory field on the Add Cloud Credential page for Amazon type

What kind of request is this (question/bug/enhancement/feature request): bug
Steps to reproduce (least amount of steps as possible):

On the Rancher UI, go to Cloud Credential page.
Click on 'Add Cloud Credential'.
Select Cloud Credential type: Amazon and enter the Access Key and Secret Key (Mandatory fields)
Click on create.

Expected Result:
Region field is made mandatory or has a value by default, so that appropriate error message gets populated.
Actual Result:
Error undefined ConfigError: There was a problem validating your keys. Enter valid credentials and try agian. is seen on the UI.

Other details that may be helpful:
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): rancher:master
Installation option (single install/HA): HA

The region field is required to validate keys. We validate the keys by making a call to the AWS api. In order to call the AWS api you must give a region. Technically it is not required to save the resource but I don't think that matters nor do I think it would be useful to explain this to the user.

Verified on rancher:master. 'Region' field has been marked mandatory. If you don't enter region, it throws an appropriate error You must select a region in order to validate your keys.