Returns results in San Francisco despite different zip code specified

Hi,
I am attempting to scrape every Whole Foods in the USA. Frankly, the best way I have figured to do this is to use input queries that are essentially "whole foods near zip xxxxx, whole foods near zip xxxxy, etc" for each zip code. However, when I do this, I get tons of duplicate results that are all around San Francisco. I've tried tweaking zip codes and adding quotes, to no avail.
I think this is because Google Maps' default location opens to San Francisco. In the debug mode it is in incognito, yet still does this.

@R-Rajaie I think I can pass the long/lat you want in the browser. Will try to integrate that

@R-Rajaie check the new options:
-geo string
set geo coordinates for search (e.g., '37.7749,-122.4194')
-zoom int
set zoom level (0-21) for search


Vulnerability roundup 97: jetty-9.4.31.v20200723: 1 advisory [3.7]

search, files

[ ] CVE-2020-27218 CVSSv3=3.7 (nixos-20.09)

Scanned versions: nixos-20.09: 50c8d1e0deb.

See also: #102797

fixed in https://github.com/NixOS/nixpkgs/pull/111410


feat(node): Remove blocking calls in the FFI

Since we replaced RecordBatchReader> with RecordBatchIterator, we no longer need blocking calls to send RecordBatch to the rust core
Fixed js-transformers example
Tested with unit test, examples and some unit test

Could you also change the transformersjs notebook in docs/src/examples file as well?


BANNABLE !

My boyfriend just got banned after using R3nzSkin ONE single time. while on his smurf, but it was a bought account. I've been using the software a while ago and i've NEVER got banned, but ill stop using it rn. This is just a warning for all people, it might be bc of the bought account but take care yall.

Hello! I'm a long time RU and EUW server player. My first ever account on EUW was bought and i've used this skinchanger while playing on it and on RU server. recently got banned on EU as well. Seems like EU is getting a banwave. I would love to know if  your boyfriend's account was bought from a website or was it his own personal account. Just curious

Joined 5 hours ago ???

I think you're trying to defame this project.

Riot did banwave for scripters and botted accounts prob you banned for this.

I think you're trying to defame this project.

LMAO i literally just joined to let people know about it. I dont use github at all, not much XDDD???

Hello! I'm a long time RU and EUW server player. My first ever account on EUW was bought and i've used this skinchanger while playing on it and on RU server. recently got banned on EU as well. Seems like EU is getting a banwave. I would love to know if your boyfriend's account was bought from a website or was it his own personal account. Just curious

It was a bought account from a website, o it might be as well bc of both things, the skin changer and the seller that has been targeted already or something like that, bc he got an account for his friend and got banned bc third party programs but he didnt used anything, so is really really weird

You seem like a sad hater or something?   It doesn't matter if the account is botted or not/ because this software triggers the ban.

I'm still using it every single day and i have been using it since it came out, i've never been banned because of this tool or any other skin tool, you lot must be special, which i think you might be, just not in the good way

I've been playing on my main 600 level account every day and I can say the software is undetected.


MessagePayload.create

MessagePayload.create requires MessagingType parameter but is omitted in the sample code. There was another API difference. Is messenger4j 1.0.0 API current?

Btw, thank you for updating Spring Boot Quickstart!

Thanks for reporting!
I've fixed it now on master.
Version 1.0.0 ist the latest version.


Implementation details required

Hello,
I'm trying to make a good integration for my React GraphQL project.
I was using GraphQL code Generator at first but I switched to GraphQL config because i can also generate code using it.
Context
I'm using three separated project (2 frontend: 1 mobile app using React Native and one web app using ReactJS. 1 backend using apollo server).
I'm trying to get types / import completion in the frontend apps, also validate schema against remote schema.
Issues / Success
✅ I'm able to

Run graphql validate and get proper results, also graphql coverage detect my used queries and everything can be validated before commit which is great.
Get both graphql-files-modules.d.ts and graphql-operations-types.ts generated in my project folder on graphql codegen

⛔ I'm not able to

Get auto-completion inside .gql files based on my remote types.
Get .gql file path auto-completion in .tsx / .jsx files.

Specifications
My folder structure with the .gql file

VSCODE Version: 1.40.2
Commit: f359dd69833dd8800b54d458f6d37ab7c78df520
Date: 2019-11-25T14:52:45.129Z
Electron: 6.1.5
Chrome: 76.0.3809.146
Node.js: 12.4.0
V8: 7.6.303.31-electron.0
OS: Darwin x64 19.0.0 (MACOS catalina 10.15.1 (19B88))

"dependencies": {
    "@apollo/react-hooks": "^3.1.3",
    "@next/bundle-analyzer": "^9.1.4",
    "apollo-boost": "^0.4.4",
    "date-fns": "^2.8.1",
    "graphql": "^14.5.8",
    "i18next": "^19.0.1",
    "i18next-browser-languagedetector": "^4.0.1",
    "next": "9.1.4",
    "next-compose-plugins": "^2.2.0",
    "next-plugin-graphql": "^0.0.2",
    "next-with-apollo": "^4.3.0",
    "react": "16.12.0",
    "react-cookie-consent": "^2.5.0",
    "react-dom": "16.12.0",
    "react-i18next": "^11.2.3"
  },
  "devDependencies": {
    "@graphql-codegen/typescript": "canary",
    "@graphql-codegen/typescript-graphql-files-modules": "^1.9.1",
    "@graphql-codegen/typescript-operations": "^1.9.1",
    "@test-graphql-cli/codegen": "canary",
    "@test-graphql-cli/coverage": "canary",
    "@test-graphql-cli/validate": "canary",
    "@types/node": "^12.12.14",
    "@types/react": "^16.9.13",
    "graphql-cli": "canary",
    "husky": "^3.1.0",
    "lint-staged": "^9.5.0",
    "prettier": "^1.19.1",
    "tslint": "^5.20.1",
    "tslint-config-prettier": "^1.18.0",
    "tslint-plugin-prettier": "^2.0.1",
    "tslint-react": "^4.1.0",
    "tslint-react-hooks": "^2.2.1",
    "typescript": "^3.7.2"
  },

Logs Of TS Server || GraphQL Language Service
GraphQL server
I had nothing before switching to rc instead of config. But I had no completion before too…
Caught the error Error: Request initialize failed with message: ".graphqlconfig" file is not available in the provided config directory: /Users/ahanss/js-dev/perso/jymiz-renew/staff.jymiz.com
Please check the config directory.

[Info  - 10:00:45 GMT+0100 (Central European Standard Time)] Using tsserver from: /Users/screamz/js-dev/perso/site-renew/staff.site.com/node_modules/typescript/lib
[Info  - 10:00:45 GMT+0100 (Central European Standard Time)] <syntax> Forking...

Thanks

your first issue has been addressed, however there is no plan to continue supporting #import syntax in graphql as I understand, as that is not spec, and this is soon to become the official GraphQL extension.
you should find a lot of features are working across the workspace, including fragment completion, as long as you use documents to include them in the new graphql config@3 pattern


Adding static html support as a coverage service

I'm fairly novice on ruby and slather's codebase, so if there are codes that need to be improved or probably a guideline that I need to conform to, please inform me.
This will generate static html files, so reviewing code locally can be more thorough than using the simple output. Often the simple output is enough, but you only know how much lines instead of which lines are being covered.
This is useful for me because it will just open in the browser and it doesn't need any connection or application (no need coverstory mentioned in #49).
Here's what it looks like :tada:

One thing to notice, although the build is passing, coveralls sends an error message:
Coveralls encountered an exception:
Errno::EMFILE
Too many open files - getcwd
/Users/travis/build/ixnixnixn/slather/travis_bundle_dir/ruby/2.0.0/gems/coveralls-0.8.1/lib/coveralls/configuration.rb:116:in `pwd'
/Users/travis/build/ixnixnixn/slather/travis_bundle_dir/ruby/2.0.0/gems/coveralls-0.8.1/lib/coveralls/configuration.rb:116:in `pwd'
/Users/travis/build/ixnixnixn/slather/travis_bundle_dir/ruby/2.0.0/gems/coveralls-0.8.1/lib/coveralls/configuration.rb:171:in `relevant_env'
/Users/travis/build/ixnixnixn/slather/travis_bundle_dir/ruby/2.0.0/gems/coveralls-0.8.1/lib/coveralls/configuration.rb:9:in `configuration'
/Users/travis/build/ixnixnixn/slather/travis_bundle_dir/ruby/2.0.0/gems/coveralls-0.8.1/lib/coveralls/api.rb:64:in `apified_hash'
/Users/travis/build/ixnixnixn/slather/travis_bundle_dir/ruby/2.0.0/gems/coveralls-0.8.1/lib/coveralls/api.rb:21:in `post_json'
/Users/travis/build/ixnixnixn/slather/travis_bundle_dir/ruby/2.0.0/gems/coveralls-0.8.1/lib/coveralls/simplecov.rb:72:in `format'
/Users/travis/build/ixnixnixn/slather/travis_bundle_dir/ruby/2.0.0/gems/simplecov-0.10.0/lib/simplecov/result.rb:46:in `format!'
/Users/travis/build/ixnixnixn/slather/travis_bundle_dir/ruby/2.0.0/gems/simplecov-0.10.0/lib/simplecov/configuration.rb:159:in `block in at_exit'
/Users/travis/build/ixnixnixn/slather/travis_bundle_dir/ruby/2.0.0/gems/simplecov-0.10.0/lib/simplecov/defaults.rb:60:in `call'
/Users/travis/build/ixnixnixn/slather/travis_bundle_dir/ruby/2.0.0/gems/simplecov-0.10.0/lib/simplecov/defaults.rb:60:in `block in <top (required)>'

I've tried to close all the files in my specs but it doesn't remove the error message.

Thanks! I'll take a look at this

This is super cool.

Thanks! Please inform me if there's anything that I can improve.

This is awesome @ixnixnixn :+1:

@Ashton-W thanks mate! Although I haven't put any js stuff for sorting and filtering.

Might be best to get this merged before adding that - I'm keen to help.

OMG. can't wait to be able to use this!!!

@Ashton-W @3lvis probably there are more experienced rubyist in this thread, but do you know how to rectify the exception from coveralls? It's currently passing but coveralls seemed to be complaining :confused:
Coveralls encountered an exception:
Errno::EMFILE
Too many open files - getcwd

Also waiting 🚬

Really cool. Would love to see this merged :thumbsup:

Updates! :tada:

after slathed it'll open index.html automatically
no more Errno::EMFILE Too many open files - getcwd after updating to ruby 2.2
syntax highlighting with highlight.js

Tests are passing, everything should be sweet :+1:

I think opening the browser should be tied to an option flag, CI servers need a way to stop it.

FWIW, https://github.com/jonreid/XcodeCoverage allows this to be configured via a --show flag.

Favoring the '--show' flag as well. On a CI server you only need the html output for publishing.

Hmm, fair point. I do want to open it by default because I was thinking that this will only be for local usage. Hence, I missed the CI related issues.
My plan will be printing the path by default, and if --show is flagged then it will open the browser. I'll give it a go tonight.

Another update 🎉

using --show flag for opening "index.html" automatically, otherwise print out path by default
coverage list supports sorting and filtering via List.js

:+1: thanks @ikhsan and everyone else.

Yay! Thanks Mark and all! :tada:

Anyways, I've filed another minor PR to update the docs related to the html support (removing -h, updating flags, and my link) : #90


Set workspace.library in workspace instead of user settings

would it be possible for your WoW API VSCode extension to not set the workspace library in the global LuaLS settings? It should be simple enough as it already creates a .vscode folder with a settings.json to detect globals. Only reason I'm asking is because I have to remove the library links when I need to work on another games code.

Fixed in 0.15.0


how to configure dwm.c ?

How to configure dwm.c.
I want to add key binding for screenshot utility.

Hi, if you want to add screenshot utility add in the configuration file ft:config.def.h no need to change dwm.c file. Can you please specify which tool do you want to add? so that i can configure accordingly...
options are: 1) imagemagick
2) scrot
3) flameshot
or any other you are known off?
thanks:)


how to compile it on windows

What is win.h? I cannot find anything about it on the internet

https://github.com/xunboo/llama2win
just copy the mmap-windows.c from this repo and change the run.c:
#if defined _WIN32
    // #include "win.h"
    #include <stdint.h>
    #include "mmap-windows.c"
#else


Load model locally

Hi,
I'm making an app that uses Tauri, and I want to bundle the model with the application (since I want it to be offline). Are there ways to load the model locally?

If I understand correctly, you need to loaf a local model, right?

Well, Tauri uses the system's browser to render the app, so I guess you can call it a 'web' app.
But it does run WASM.
Here, I put a test to see if WebAssembly works or not:

Here's the result:

So yes, it does support WASM. My question is whether there's a way to load the model instead of letting the user download it. if not, are there some ways to achieve that?

In that case, you can download a model via a call to createModel with your selected URL, path, and id.
This is the norm. I don't recall the user having to download the model themselves. A call to createModel with fetch and cache the model for you.

Okay. i did what you said.
But i got a problem, i'm getting this error:
10-15 20:31:05.080  7532  7532 E Tauri/Console: File: http://tauri.localhost/js/Vosklet.js - Line 1 - Msg: Uncaught (in promise) DataCloneError: Failed to execute 'postMessage' on 'Worker': SharedArrayBuffer transfer requires self.crossOriginIsolated.
i looked around and found out that CORS policy are blocking tauri's localhost. they said i should use it's http plugin. how can i edit vosklet.js to make it use tauri's http plugin?

Can you somehow use that plugin to inject the COOP AND COEP headers into your app using that HTTP client?

hi msqr1 ,
I am  trying to add new language models to vosklet . How can I do that. BTW I am engaged in a linguistic project and need more assistance . Please contact at:
<EMAIL_ADDRESS>to explain more about my  project .

We can talk about that over in #18. It seemed off topic here.


Many xxxxxc combinators (ones that takes input and returns result directly) have not been update to using the Parser trait

Like this one:
https://github.com/Geal/nom/blob/38bb94e7bc45c2b4ef0c473e0e5c03ab134cdff3/src/combinator/mod.rs#L85-L92
Is this intentional or is it due to no one having submitted any PR yet?

those combinators are there only for use inside macros. Are there cases where you want to use them?

They will be removed in nom 7


some error about Promise.all

In es2015,
Promise.all([1, 2, 3, 4, 5]).then(console.log)
print "[1,2,3,4,5]"
but in this lua Promise
Promise.all({1, 2, 3, 4, 5}).andThen(print)
do nothing.

ho, it must use ":", not "." before andThen
this right Promise.all({1, 2, 3, 4, 5}):andThen(print)

Ya. 👍


react-native-esc-pos can work with ios?

Work perfect on android but not working on ios

@KannarFr you can try this library - it supports iOS too
https://github.com/tr3v3r/react-native-esc-pos-printer


Save column and grid layout

Has anyone done this per logged in user.  Just want to know the best way to approach this?  Thinking about storing the settings in a cookie.
Requirements

Save column order
Save columns not hidden

See
https://volkanceylan.gitbooks.io/serenity-guide/content/grids/persisting_settings.html
Best Regards


update filter to include .Answer

Add .Answer to the search field, understanding that .answer was removed during multi-answer support. (please confirm @rgossiaux @asdfryan)

it looks like it might just be generated off of the accessor and added to the values on the .Answer key because that's the name of the column
From table columns:
  {
    Header: "Answer",
    accessor: (row) => row.guesses.map(({ text }) => text).join(" "),
    Cell: AnswerCell,
  },

I'm not sure how I feel about relying on that behavior.

it looks like it might just be generated off of the accessor and added to the values on the .Answer key because that's the name of the column
From table columns:
  {
    Header: "Answer",
    accessor: (row) => row.guesses.map(({ text }) => text).join(" "),
    Cell: AnswerCell,
  },

I'm not sure how I feel about relying on that behavior.

I think you can add an id of answer as one of the options for a column and that will be the accessor name, haven't confirmed though

I think you can add an id of answer as one of the options for a column and that will be the accessor name, haven't confirmed though

I think you can add an id of answer as one of the options for a column and that will be the accessor name, haven't confirmed though

Yes! This is a much nicer solution.

I think you can add an id of answer as one of the options for a column and that will be the accessor name, haven't confirmed though

Yes! This is a much nicer solution.


Consider using OCI compliant media-types

Describe the problem/challenge you have
I am trying to push Imgpkg artifacts to a Zot registry. Zot registry describes itself as a "vendor neutral" OCI registry. In practice this just means that Zot refuses to accept any image or artifact that uses a Docker (or any other non-OCI) specific mediaType.
For more information about this Zot's refusal to work with Docker artifacts, see this issue:

https://github.com/project-zot/zot/issues/724

You can already see where I am going with this:
$ imgpkg push -i localhost:5000/test-bundle:v1.0.0 -f anyfile.txt --registry-insecure

imgpkg: Error: Writing 'localhost:5000/test-bundle:v1.0.0':
  Writing image:
    PUT http://localhost:5000/v2/test-bundle/manifests/v1.0.0:
      MANIFEST_INVALID: manifest invalid; map[description:During upload, manifests undergo several checks ensuring validity. If those checks fail, this error MAY be returned, unless a more specific error is included. The detail will contain information the failed validation. mediaType:application/vnd.docker.distribution.manifest.v2+json]

Describe the solution you'd like
Since it's probably unlikely for Zot to ever accept mediaTypes that are not defined by OCI, I would like imgpkg to (optionally) support the creation of bundles using standard OCI mediaTypes for its manifests and layers.

Vote on this request
This is an invitation to the community to vote on issues, to help us prioritize our backlog. Use the "smiley face" up to the right of this comment to vote.
👍 "I would like to see this addressed as soon as possible"
👎 "There are other more important things to focus on right now"
We are also happy to receive and review Pull Requests if you want to help work on this issue.

For more information about this Zot's refusal to work with Docker artifacts, see this issue:

Regardless of what the zot project does, I respectfully urge the carvel team to look at becoming OCI compliant.
For reference, helm already is: https://helm.sh/docs/topics/registries/

@ChristianCiach I believe you would be interested in this proposal.


MAINT: Audit CIs using zizmor

Not 100% sure about adding this as a pre-commit hook because at least on latest Ubuntu 24.10 I needed a newer rust than was installed on the system, but it was pretty easy to install. An alternative would be to add it as a GitHub action.

Opened https://github.com/woodruffw/zizmor/issues/236 about pre-commit.ci, we'll see what they say...

Actually they responded to the issue and might try to make it Rust 1.75-compatible, which would work with at least Ubuntu 22.04+. Not sure about macOS or Windows, though.
Could add it to the Style GHA run I suppose

I don't like the idea of all contributors (not just maintainers!) needing to install or update Rust just to make a change to MNE. That alone sways me pretty strongly toward GHA / running zizmor only in cloud. The only hesitation I have is that, if someone did introduce a security hole into one of our GHAs, it would be nice to catch that hole locally before it got pushed into a PR and executed.

no need to install rust anymore! It's a regular python hook - suspiciously similar to ruff

thanks @larsoner and @graingert !


Adding property to build so that liquibase-scripting shows up in the tarball

Impact

[ ] Bug fix (non-breaking change which fixes expected existing functionality)
[ ] Enhancement/New feature (adds functionality without impacting existing logic)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description

Things to be aware of

Things to worry about

Additional Context

just fyi, it wont break the build because the property isn't used by anything right now.
It is referenced in the DAT-16671 branch that @wwillard7800  on right now.


How can I ignore "null" values when drawing a PlotSignalXY or PlotSignal

(Or maybe a different plot type would make more sense - suggestions?)
I am pulling data from a table to be plotted. Each record has a timestamp and multiple associated values. I am able to successfully pull this into a PlotSignalXY and have been having all sorts of fun with all the various ScottPlot options - well done!
However, in some cases, I'll have some null values for various reasons. Is there some way to not include this data value in the plot?
I suppose I could build the xs and ys and strip out the null values. However, I'm driving multiple plots from a single data source (and using a shared xs array). I'd rather be able to set the specific y value to something like double.NaN to just skip the y. (This aligns with my earlier question about time gaps) Any thoughts?
If not, do you have any guidance as to where to start my search if I wanted to attempt a solution? (It would make sense to apply this analysis before the time-gap logic)
Thanks!

Hey @Wyrus,
I'm happy to hear you are enjoying ScottPlot! Similar to #618, line plots do not currently support disconnected data. Multiple lines means creating multiple line plots. This shouldn't be too hard to do on your end though.  I recognize this feels inefficient (requiring copying the data before plotting it), but part of what makes ScottPlot so fast is that it draws scatter plots and signal plots as a single, unbroken line.
I'm working on alternative solutions (e.g., scatter plots that display double.NaN as gaps in data), but for now displaying gaps in data is not currently supported.
I hope this helps! Let me know if more questions pop up along the way
Scott

For anyone worried about actual performance with this approach, I just finished some testing...
INCLUDING the time to analyze the source data, break it into contiguous chunks, allocate data arrays for each chunk, clear the existing plot, add a new PlotSignalXY for each of 10 chunks, and render...
It took roughly double the time as a single PlotSignalXY with all the points that re-used the data buffers.
What that suggests to me is that performance of ScottPlot is heavily weighted toward point count, and not near as much on the number of plots.
I'll say it again... Duly impressed!
Oh by the way, building contiguous chunks myself gives me full control of the gap fill behavior as well. :)

It's often surprising to learn where the bottlenecks are! Most of the optimizations are for continuous data with fixed inter-point spacing, and the farther you go from that model the more it takes testing to figure out the best solution...
Thanks for following-up with your findings @Wyrus, and I'm happy you found a solution that works for you!


fix(behavior_velocity_planner): consider all lane_ids of path points

Signed-off-by: tomoya.kimura<EMAIL_ADDRESS>Description
In some cases, intersection module does not launch properly for a short lane.
For example, in the following scene,

the intersection lane id: 10274 not included at the front index of path points in /planning/scenario_planning/lane_driving/behavior_planning/path_with_lane_id
[.... orientation=geometry_msgs.msg.Quaternion(x=0.0, y=0.0, z=0.43374394501706726, w=0.9010361758337072)), longitudinal_velocity_mps=5.55555534362793, lateral_velocity_mps=0.0, heading_rate_rps=0.0, is_final=False), lane_ids=[10527]), autoware_auto_planning_msgs.msg.PathPointWithLaneId(point=autoware_auto_planning_msgs.msg.PathPoint(pose=geometry_msgs.msg.Pose(position=geometry_msgs.msg.Point(x=61821.89735023088, y=56312.33227842826, z=77.89040236548254), orientation=geometry_msgs.msg.Quaternion(x=0.0, y=0.0, z=0.413801767695025, w=0.9103670122826686)), longitudinal_velocity_mps=5.55555534362793, lateral_velocity_mps=0.0, heading_rate_rps=0.0, is_final=False), lane_ids=[10527, 10274]), autoware_auto_planning_msgs.msg.PathPointWithLaneId(point=autoware_auto_planning_msgs.msg.PathPoint(pose=geometry_msgs.msg.Pose(position=geometry_msgs.msg.Point(x=61827.47889463101, y=56317.81207744501, z=77.95071582166267), orientation=geometry_msgs.msg.Quaternion(x=0.0, y=0.0, z=0.4365995768463751, w=0.8996559395110812)), longitudinal_velocity_mps=0.25223785638809204, lateral_velocity_mps=0.0, heading_rate_rps=0.0, is_final=False), lane_ids=[10513]), .....]

In such a cases, the intersection module does not launch for lane 10274.
I fixed this bug by considering the lane_id of the second and subsequent indexes.

Related links

Tests performed
Confirmed that the intersection module launches properly even for a short intersection lane.
(We can confirm the launch of a intersection module by following standard output:
[planning.scenario_planning.lane_driving.behavior_planning.behavior_velocity_planner]: register task: module = intersection, id = 10274

)

Notes for reviewers

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[ ] I've confirmed the contribution guidelines.
[ ] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.
[ ] The PR has been properly tested.
[ ] The PR has been reviewed by the code owners.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.
[ ] The PR is ready for merge.

After all checkboxes are checked, anyone who has write access can merge the PR.

[Q]
Is it necessary to register all lane_ids at intersection to handle correctly?
Could you describe the edge case where unique first lane id registered is not enough?
I will be a bit cautious to add this change because calculation cost can be up to 3 times more than before to register all lane ids.


State management

Figure out a state management solution, doesn't have to be Redux.

Settled on Zustand.


Add TCKs to test the configuration

We need to test the property mp.telemetry.tracing.enabled=true/false. The default is false for 1.0 release.

Also need to update spec from mp.telemetry.trace.enabled to mp.telemetry.tracing.enabled

@fmhwong will take this.


UI Overhaul

Make UI Responsive

contributed https://github.com/dbis-uibk/relax/commit/50a163b3181dabeb15ce346bbf470e0b65bbc618 https://github.com/dbis-uibk/relax/commit/c87bff08fb01969d4842aaee976637919adbf73f https://github.com/dbis-uibk/relax/commit/173cb7d206aa00ebf2c1823105fbcd9380b34279

contributed 7a7239b06d6fd6702c42f19aa4eacb2a22130861 and a8a4e500a896537332a48d2b4370d9fce1b469da


Breakpoints

Is it, currently, possible to add breakpoints in Dev Tools - like pure HTML/JS? If not, is it possible to add them in code?
Finally, if neither are possible, will this be a feature in the future?

this worked REALLY well with html-vue-loader..  not sure why the vm displays this one all on one line?

still no way to debug normal code (not a single line) ?

I plan to add a devMode : boolean option that will preserve, as far as possible, the originale script formating.


default param values not getting added to bedrock requests on any provider

after 0.63.1 (#915) claude returns
API call error: ValidationException: Malformed input request:
#: subject must not be valid against schema {"required":["messages"]}
#: required key [max_tokens] not found
#: required key [anthropic_version] not found, please reformat your input and try again.

the problem is likely this: https://github.com/promptfoo/promptfoo/pull/915/files#r1633789271

oof, thanks for catching.  I tested everything except for Claude.
#917 should resolve this.  Also added some quick tests that would've caught this issue.


SubscribeQueue available for .net core

I see that, along with the (announced) deprecation of sync methods for AWS service calls, specifically within the SQS service the SubscribeQueue method is not available for .net core, and as far as I could see, there are no other alternatives.
Is there a strong reason why this isn't supported? And can we expect it to be supported in the near future?

Sync methods aren't deprecated and will continue exist in the main .NET 3.5 and 4.5 version of the SDK. The .net core version of the SDK is based around System.Net.Http.HttpClient which only suports async calls which is why that version of the SDK only exposes async calls. The .NET 3.5 and 4.5 version of the SDK are build around HttpWebRequest which supports both sync and async in .NET 3.5 and 4.5 but in .net core HttpWebRequest has greatly reduced functionality.
SubscribeQueue is a utility method we added to the SDK and is not an actual service method. There are many of these utility methods that are missing in the .net core version because of the async only nature but we are looking into reworking them so we can add them back. It is our intention of getting most of the utility classes and methods available in the .net core version.

@normj I meant specifically for the .net core version, sorry for the ambiguity.
Great to know that! We are, for now, just adding our own extension methods that mimics the original ones, but use the async calls.

Cool, I think what we will do is add a SubscribeQueueAsync eventually that does the same as SubscribeQueue but call the service async methods.

Closing this as this is part of a great effort we need to do for the .NET Core version to support more of the utility classes in the SDK.


[tools]Fix typo for CleanUp tool powershell script

Summary of the Pull Request
Fixed typo in CleanUp_tool.ps1

PR Checklist

[ ] Closes: #25747
[ ] Communication: I've discussed this with core contributors already. If work hasn't been agreed, this work might be rejected
[ ] Tests: Added/updated and all pass
[ ] Localization: All end user facing strings can be localized
[ ] Dev docs: Added/updated
[ ] New binaries: Added on the required places

[ ] JSON for signing for new binaries
[ ] WXS for installer for new binaries and localization folder
[ ] YML for CI pipeline for new test projects
[ ] YML for signed pipeline

[ ] Documentation updated: If checked, please file a pull request on our docs repo and link it here: #xxx

Detailed Description of the Pull Request / Additional comments
Powershell script has – instead of - in some places. I changed it and tested in my system.
First time doing PR. Sorry if I missed some steps for PR.

Validation Steps Performed

Hi @Sajad-Lx ,
Thanks for opening the PR. What typo is this PR specifically fixing? Why was it wrong before, it seems you're substituting for an equivalent character. What impact does this PR actually have?

Hi @jaimecbernardo ,
I'm adding an attachment below for more detail if this helps. Thank you for your time.

Hi @jaimecbernardo , thank you for checking it out.
PS: I happened to stumble on this script and thought I can make my first PR.


improve strictdoc rendering of imported TSRM

both "None" titles and the large meta fields

some improvements proposed https://github.com/strictdoc-project/strictdoc/pull/677

merged there!


wip: update github action lint pr

Description
Updated the github action to lint the PR commit message when the PR contains a single commit since GH will use this commit for the commit message when it's merged.
Pull Request Checklist

[ ] Ask the contributors if a similar effort is already in process or has been solved.
[x] Review the contribution guidelines.
[ ] Use staging as your pull request's base branch. (All PRs using production as its base branch will be declined).
[ ] Ensure all gulp scripts successfully compile.
[x] Update, remove, or extend all affected documentation.
[x] Ensure no private Dialpad links or info are in the code or pull request description (Dialtone is a public repo!).
[x] Request a review from Brad Paugh, David Becher, or Drew Chandler.

Obligatory GIF (super important!)

Marked as WIP since it's not validating the correct commit


Missing license

Hey I was looking at this library and I would like to use it in a closed-source application. I was wondering if you have any licensing information available that could help me evaluate this library.
Btw amazing work :)

Hi! i'm sorry for the ommision, the idea is to use a permisive MIT licence. The only problem with this is that some non-essential files (like linker script and startup file) is licenced under GPL.
I queue the rewrite of ld script and startup code and add LICENCE file (and header licence in .h/.c) for doing in few days (when my daily work allows me)
meanwhile you can go testing the code and feel free to post any enancement or issue to resolve

@martinribelotta we could help you with the rewrite of the ld script and startup code if you can make an issue with your intent if that helps speed things up


Do the  mappings work natively on Mac?

Thanks for the great plugin.
I'm trying hard to get it to work on a mac in iterm2 with vim version  7.4.488.
My .vimrc is pretty big so I'm eliminating any possible conflicts by using a plain vanila .vimrc
with nothing in it but your plugin.
None of the <M mappings work for me.
Am I missing some obvious thing specific to Mac. Anybody else has/had this issue?
I saw that there is a possibility to set your own mappings using the 'sexp-explicit-mappings' directive.
But I like the default mappings so want to stick to those..
Thanks

Mapping meta does not work in the terminal because Vim cannot tell the difference between meta and esc. Neovim in the terminal works fine with meta maps (it's also possible to escape), but it means that the opposite issue occurs: https://github.com/neovim/neovim/issues/2017

Hi @gsavovski,
I make Alt and Mod4 mappings work in my terminal by explicitly setting them on
startup.
For example, if your terminal sends \033a when you press <M-a> on your
keyboard, you can teach vim this keystroke with:
execute "set <M-a>=a"
" We must use a literal Escape character

Unfortunately, some keys cannot be set, but they can be mapped in the usual
way.
Therefore, to make <M-…> keys work in a terminal that is not in an 8-bit
mode:
function! s:Setmap(map, seq)
    try
        " Some named values can be `set'
        execute 'set ' . a:map . '=' . a:seq
    catch
        " The rest can simply be mapped
        execute 'map  <special> ' . a:seq . ' ' . a:map
        execute 'map! <special> ' . a:seq . ' ' . a:map
    endtry
endfunction

" Special named keys that cause problems in set and map calls
let namedkeys = { ' ': 'Space', '\': 'Bslash', '|': 'Bar', '<': 'lt' }

" Map Alt + ASCII printable chars
for n in range(0x20, 0x7e)
    let char = nr2char(n)
    let key  = char

    if has_key(namedkeys, char)
        let char = namedkeys[char]
        let key  = '<' . char . '>'
    endif

    " Escaped Meta (i.e. not 8-bit mode)
    "  * Esc-[ is the CSI prefix (Control Sequence Introducer)
    "  * Esc-O is the SS3 prefix (Single Shift Select of G3 Character Set)
    if char !=# '[' && char !=# 'O'
        call s:Setmap('<M-' . char . '>', "\<Esc>" . key)
    endif
endfor

unlet namedkeys n key char
delfunction s:Setmap

You can actually do something similar for other modifier keys if you can
configure your terminal to send custom modifier prefixes.
For instance, I bind the rarely used <4-…> keys to my Mod4/Super key by
configuring my terminal (rxvt-unicode) to send \033\007 before the keycode.
Then in the loop above I call:
    call s:Setmap('<4-' . char . '>', "\<Esc>\<C-g>" . key)

https://github.com/guns/haus/blob/a0f50b7a8b9eb6a2724b1b815e4100d026244628/etc/vim/local/modifiers.vim#L42-L51
Having another modifier available for mappings in vim is quite lovely.
(Super + s for :update<CR> for instance)
I've always meant to package this bit of vim voodoo into a plugin, but never
have gotten around to it. In fact, when I originally wrote vim-sexp, I had
been using this setup for so long that I forgot that <M-…> does not work in
vim by default.
Hence https://github.com/tpope/vim-sexp-mappings-for-regular-people
:)

@justinmk
Oops, forgot about that side effect. Appending my first post now...

Okay, I've updated my comment with discussion of ttimeout{,len}.
It's surely a hack, but a serviceable one nonetheless.

@gsavovski
One more thing I forgot to mention: Meta mappings work by default in vim if
you can configure your terminal to set the 8th bit on meta/alt.
Most terminals can be configured to do this, but often do not do it by
default. I imagine iTerm has this option as well. IIRC, the default OS X
Terminal also has this option.

Hi @justinmk
Thanks for the kind explanation.
I actually remembered that I never could
use the Alt/Meta and Command keys since
I didn’t have any mappings for them in vimrc.
Hi @guns,
Thanks for opening this new door :-)
I already had set in my iTerm2 that the left option
acts as (+Esc).
When I press any of the <M-S-hjkl> combination
in terminal by catting the output I get:
-=} cat
^[H^[J^[K^[L
So I tried explicitly setting:
execute "set <M-S-j>=^[J"
execute "set <M-S-k>=^[K"
execute "set <M-S-h>=^[H"
execute "set <M-S-l>=^[L”
The ASCII code for ‘^[‘ is ‘\094 \091’.
Then I just put your code snippet in my vimrc and
it really worked. I can not even believe it :-)
Thank you so much!
Few more questions bug me..
Why did the explicit setting I mention above didn’t work?
Out of curiosity looking at your modifiers.vim code you have a lots
of mappings using the arrow keys, what do you accomplish doing that?
And lastly what do you mean by rarely used <4-…> mapping to be used as super?
Is that the actual number 4?
Thanks again this is great :-)
By the way the plugin already worked great in MacVim but
I'm much more of terminal user, therefore this is super exciting..

I already had set in my iTerm2 that the left option acts as (+Esc).

Sounds like turning this off might enable meta-sets-top-bit mode, which is
really the best no-hack option if you just want to enable meta keys in your
terminal. You should give it a try!

So I tried explicitly setting:
execute "set =^[J"
execute "set =^[K"
execute "set =^[H"
execute "set =^[L”
The ASCII code for ‘^[‘ is ‘\094 \091’.
…
Why did the explicit setting I mention above didn’t work?

The ^[ is actually the standard printable representation for ASCII 033
(Escape). This character can be produced with the Escape key, or by pressing
<C-[>. Actually, the first 32 unprintable ASCII control characters can be
produced by pressing Ctrl and the corresponding ASCII equivalent 0x40 up the
table:
Ctrl + @ (0x40) -> 0x00 (Other keys can send NUL; I like Ctrl + Space)
Ctrl + A (0x41) -> 0x01
Ctrl + B (0x42) -> 0x02
Ctrl + C (0x43) -> 0x03
…
Ctrl + _ (0x5f) -> 0x1f

Of course, we don't type Ctrl-Shift-C when we want to send interrupt
through a terminal, so the actual key sent is upper cased when the Control
modifier is active. This convenience is why Ctrl-a and Ctrl-Shift-a
are indistinguishable in most terminal programs. Lots of people find this
annoying, but I personally enjoy this feature.
Here is the relevant bit in vim:
https://github.com/vim/vim/blob/60cce2f/src/ascii.h#L40-L41
#define Ctrl_chr(x)	(TOUPPER_ASC(x) ^ 0x40) /* '?' -> DEL, '@' -> ^@, etc. */
#define Meta(x)		((x) | 0x80)

Here we can also see why configuring your terminal to set the top bit when
Meta is active makes Meta mappings work out of the box.

Then I just put your code snippet in my vimrc and
it really worked. I can not even believe it :-)

I've edited my comment again to remove an errant debugging echo string(), so
you may want to remove that from your vimrc.
You've convinced me to start work on packaging this up as a plugin since it
does not seem to be common knowledge. It is in the documentation, but it's
easy to miss: :help :set-termcap.

Out of curiosity looking at your modifiers.vim code you have a lots of
mappings using the arrow keys, what do you accomplish doing that?

It allows mapping Modifier + Arrow keys. I don't ever find myself touching the
arrow keys, so it was mostly a way to experiment with setting key codes.

And lastly what do you mean by rarely used <4-…> mapping to be used as super?
Is that the actual number 4?

Yes, the <4- modifier is actually a mouse event modifier, but you can
bind it in regular mappings. For example, I configure my terminal to send
\033\007… when I type Super + …, then I map this sequence to produce the
corresponding <4-…>:
execute "map  <special> \<Esc>\<C-g>… <4-…>"
execute "map! <special> \<Esc>\<C-g>… <4-…>"

Now this <4-…> key can represent Super + … in your vimrc:
map <4-s> :<C-u>update<CR>

This is my favorite Vim hack, but it has the nasty side effect of borking
<Esc>, which is why I retracted my discussion of it.
Like I mentioned before, I use Ctrl-C as my escape anyway, so this does not
bother me.
I suspect there is a way to abuse the termcap key code system to accomplish
the same without this downside. If I find one I'll report back here.

By the way the plugin already worked great in MacVim but I'm much more of
terminal user, therefore this is super exciting..

Good to know. I compile vim with --disable-gui, so I am very unfamiliar with
GUI vim builds.


NOTE (macos): you can use fixup-nix-deps FILE, to fix iconv, ffi, and zlib dependencies that point to the /nix/store
CABAL_DIR set to /Users/yvan/.cabal-static
DYLD_LIBRARY_PATH set to /nix/store/jc495w1m1z7ph6hhad875kj66q3mggr1-openssl-3.0.8/lib
Quirks:
        if you have the zlib, HsOpenSSL, or digest package in your dependency tree, please make sure to
        cat /nix/store/4yrk2l4kp94xyh8mb9hsa9mx84sgifmy-cabal.project.local >> cabal.project.local

[~/IOHK/devx]$ cat /nix/store/4yrk2l4kp94xyh8mb9hsa9mx84sgifmy-cabal.project.local
package digest
  extra-lib-dirs: /nix/store/7n13ddp1m3340ls4x3zx9qkjhqidi08x-zlib-1.2.13/lib /nix/store/zb8jg7p56k0071svhqa3hqgl40d1h93m-pcre-8.45-bin/lib
constraints:
  HsOpenSSL +use-pkg-config,
  zlib +pkg-config,
  pcre-lite +pkg-config

@KtorZ is that PR a correct fix for your issue?

@yvan-sraka that sounds slightly better indeed, but still sub-optimal if I can say as we still only know about the location of that file after entering the shell. And there's not much scripting possible from it.
What could be nice would be to be able to build that file as a standalone derivation? So that we can pre-build the cabal.project.local prior to entering the nix shell.

So if this was instead a function that was exposed
function patchProjectLocal() {
  cat /nix/store/... >> "$1"
}

Would that do?
exposing it outside of the DevX shell might be a bit tricky as that's our single entry point for GH Actions.
We could probably expose it as separate attributes in the flake though if someone wanted to
nix build .#ghc810-iog-cabalProjectLocal
or something?

I like the idea of a separate attribute more than the function.

I like the idea of a separate attribute more than the function.

It could be both, but you wouldn't be able to use the separate attribute from the devx GitHub actions.

I updated the PR, now you have e.g. nix build ".#cabalProjectLocal.static" :)


Warning HEX mode

Add warning that some queries might not be executable on the SAP HANA HEX engine

LGTM (but I'm the author 😳)


fix grammatical error in vignette

EVAL lets us know whether there would be issues

Thank you very much for this PR!

Always a pleasure, especially if its thats simple :)
BTW, awesome package, thank you for providing it!

This is a strange place to ask but, if you have the interest and time, could you check whether my German translations are reasonable? They live in https://github.com/rich-iannone/pointblank/blob/master/R/utils_text.R and in https://github.com/rich-iannone/pointblank/blob/master/R/utils_text_table_scan.R. For a time I worked in Germany (i.e., had good German language skills) but that was a long time ago.

Not a strange place to ask, if you ask me...
Sure, I can do that. I will send a PR later today or tomorrow.

Quick question, do you have an example that uses most of the translations of https://github.com/DavZim/pointblank/blob/master/R/utils_text.R?

Also: do you want to allow the console logs (i.e., ℹ Step 2: an evaluation issue requires attention (ERROR). in https://github.com/rich-iannone/pointblank/blob/487ab0e76eeff14477e8c3dd13ce67f3aa855c04/R/interrogate.R) to be in different languages?

I apologize for the delay in getting back to you (had to fix some pretty bad bugs in master that was preventing the report from displaying reliably without error).
For an example that gets at least some of the translations visible, there is this:
library(pointblank)
library(tidyverse)

al <- action_levels(warn_at = 0.1, stop_at = 0.2)

agent <-
  create_agent(tbl = small_table, actions = al, reporting_lang = "de") %>%
  col_vals_gt(vars(g), 100, preconditions = ~ . %>% dplyr::mutate(g = a + 95)) %>%
  col_vals_lt(vars(c), vars(d), preconditions = ~ . %>% dplyr::mutate(d = d - 200)) %>%
  col_vals_in_set(vars(f), c("low", "mid", "high", "higher")) %>%
  col_vals_not_in_set(vars(f), LETTERS[1:8]) %>%
  col_vals_between(vars(c), 0, 10) %>%
  col_vals_not_between(vars(c), 100, 500) %>%
  col_vals_regex(vars(b), ".-...-...") %>%
  col_vals_not_null(vars(a)) %>%
  col_is_date(vars(date)) %>%
  rows_distinct(columns = vars(a, b, c, d, e, f)) %>%
  rows_distinct() %>%
  conjointly(
    ~ col_vals_in_set(., vars(f), c("low", "mid", "high", "higher")),
    ~ col_vals_not_in_set(., vars(f), LETTERS[1:8]),
    ~ col_vals_between(., vars(c), 0, 10)
  )

Also, I do think the console logs should be translated. Thank you for reminding me of that!
I'm going to open a new GH issue to further this discussion, and I'll bring a summary of these discussions to that issue.
Thanks!

@DavZim I opened the new issue: https://github.com/rich-iannone/pointblank/issues/115


Crash with Rubidium 0.6.5

Mod Name
Rubidium
Mod Homepage
https://modrinth.com/mod/rubidium/
Describe the bug
Crashes when loading game
Steps to reproduce
1.install Rubidium 0.6.5
2.Open the game
3.Crash
...
Logs
https://pastebin.com/iscnCmP0
Additional context
Forge Version 47.1.3
Mods:
Connector-1.0.0-beta.4+1.20.1-full
fabric-api-0.86.1+1.8.1+1.20.1
rubidium-0.6.5
Details:
There's no crash report be created in the crash, and the latest version of rubidium did not crash like this.

is there any hs_err_pid_xxxx.log generated in the minecraft folder?

However, the game runs flawlessly with Rubidium 0.7.0a...

is there any hs_err_pid_xxxx.log generated in the minecraft folder?

no, the game always crash in the "Scanning mod candidates" progress
there's no more logs can be found

However, the game runs flawlessly with Rubidium 0.7.0a...

sadly, 0.7.0a has many compatibility issues with some mods like create
that's why I choose the old one

I've investigated this and found out Rubidium 0.6.5 contains a conflicting package from fabric loader, which causes a split package error at runtime.

Rubidium 0.6.5 crashes due to a package conflict, where rubidium includes classes from fabric loader without relocating them. It looks like they're only using the version util classes though, so I'm not sure what prevented them from shadowing properly. We cannot relocate our classes, as they are expected to be available by dependents.

Sinytra/ForgifiedFabricAPI#22

Unfortunately, as a result, the Forgified Fabric API (and Connector) are incompatible with Rubidium 0.6.5. If newer versions of it cause compatibility issues with mods, they should be reported to the authors so they can be fixed properly. It is the best way forward.

rubbidion have so many problems of compabiltyes with the most part of the mods its better embeddium the mod fails with oculus (iris) but the most part of the mods can load with embeddium I'm sorry if I wrote wrong, my level of English is like that of an anemone.


echo_dec

Hi, I'm Faruq, I just tried your code, the code really awesome, I want to ask, is it possible to use your code in the encrypted file such the enrypted file from docx file that I've encrypted first? I tried echo data hiding in single kernel btw.
Thank You

Hello @faruqamar, I didn't understand your question clearly. Do you mean "hiding docx files into audios"  or "hiding encrypted text into audios" with this technique"?
Though the answer is "yes" for both questions. You can embed anything in binary format. But, echo hiding is very limited in terms of embedding capacity. If you use an audio signal sampled at 44.1 kHz, using 8000 samples for one bit, and having 1 kb of data to hide, you need an audio clip around 25 minutes long. If you are going to use wav files, I would suggest you to use LSB method.

yes for both, I use flac audio sampled at 96 kHz 32 bit, is it still possible for hiding encrypted file or docx file  and the size about 600 kb into audios using echo data hiding?

@faruqamar You can use this method while hiding encrypted text. But consider that you may need to change character decoding type (conversion of retrieved bit sequence) since your encryption method may have special characters. However, you will have problem with the size of file while hiding 600 kb of data if you use this method in your case.  You would need 600*25 minutes long audio clip.
As I mentioned above, I guess "least significant bit coding" will suit your problem better if you are working with audio codecs that have lossless compression such as WAV and FLAC.

well since I have essay about hiding encrypted text (docx) into flac audio using echo hiding, and I only have few minutes for the audio, probably I should change the docx file data into txt, thank you very much sir for your code and your answer, have a nice day sir :)

@faruqamar I am glad to hear that this repository was helpful for you. Good luck in your essay. I am closing this issue on github.


fix issue with displaying languages

When using delimiters to avoid the {% raw %} stuff, the toolbar, which is rendered on the base html doesn't display languages correctly.

refactored the language list to show correctly

The issue is present also at the bottom of the TPOS extension (LNbits version and link to API docs), see:

Also the dark mode tooltip:

Btw, why does this only affect TPOS extension and not the others?

it should be fixed in tpos, imho, but also here it nicer to have it like that.

There are more extensions that have the same issue, so we should either fix it in LNbits core or we fix all the extensions:

tpos
satspay
boltcards


bus after login

✓ Ready in 1857ms
○ Compiling /middleware ...
✓ Compiled /middleware in 810ms (283 modules)
○ Compiling / ...
⠹ Preparing Convex functions...
⠸ Checking for index or schema changes...
⠦ Checking for index or schema changes...
⨯<EMAIL_ADDRESS>(1330:18) @ UserAPI.eval [as request]
⨯ Error
at async Header (./components/ui/Header.tsx:16:18)
null
⨯<EMAIL_ADDRESS>(1330:18) @ UserAPI.eval [as request]
⨯ Error
at async Header (./components/ui/Header.tsx:16:18)
digest: "1037097554"
⠧ Checking for index or schema changes...
⠼ Checking for index or schema changes...
⠼ Backfilling indexes and checking that documents match your schema...
⠇ Backfilling indexes and checking that documents match your schema...
⠦ Backfilling indexes and checking that documents match your schema...
✓ Compiled /dashboard in 348ms (1418 modules)
✔ 14:50:42 Convex functions ready! (5.54s)
⨯<EMAIL_ADDRESS>(1330:18) @ UserAPI.eval [as request]
⨯ Error
at async Header (./components/ui/Header.tsx:16:18)
null
⨯<EMAIL_ADDRESS>(1330:18) @ UserAPI.eval [as request]
⨯ Error
at async Header (./components/ui/Header.tsx:16:18)
digest: "1037097554"
null
⨯<EMAIL_ADDRESS>(1330:18) @ SessionAPI.eval [as request]
⨯ Error: Not Found
at async getAuthToken (./app/auth.ts:8:12)
at async ServerDashboardHomePage (./app/dashboard/page.tsx:17:19)
null
⨯<EMAIL_ADDRESS>(1330:18) @ SessionAPI.eval [as request]
⨯ Error: Not Found
at async getAuthToken (./app/auth.ts:8:12)
at async ServerDashboardHomePage (./app/dashboard/page.tsx:17:19)
null

I have just now installed this application fresh I didn't get any of these issues, can you recheck all the points mentioned on the read-me file again?

I have just now installed this application fresh I didn't get any of these issues, can you recheck all the points mentioned on the read-me file again?

mac m1

https://www.loom.com/share/f8fdfd526310474086624712578de506?sid=5574ba68-ee90-468f-b4ea-ebf6574b4607


04/24/2018 02:56 : Create Pull Request for branch master -> prTesting1_2018042402

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestCon1.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


add filter button

Generated a new column of the dataframe with randomly generated strings ("Accepted" or "Rejected") Created a dropdown button that works with some javascript code by filtering the plot rsults based on the string value
By achieving this the problem is that the legend is no more present, nor is clickable
Fixes #344
Reopens #339
Description
Checklist

[ ] Added link to Github issue or Trello card
[ ] Added tests

Codecov Report

Merging #350 (aba36d4) into main (ba377a4) will decrease coverage by 57.31%.
The diff coverage is 0.00%.

@@             Coverage Diff             @@
##             main     #350       +/-   ##
===========================================
- Coverage   84.66%   27.35%   -57.32%
===========================================
  Files          42       43        +1
  Lines        2524     2559       +35
===========================================
- Hits         2137      700     -1437
- Misses        387     1859     +1472

Impacted Files
Coverage Δ

wellcomeml/viz/visualize_clusters.py
0.00% <0.00%> (ø)

wellcomeml/metrics/ner_classification_report.py
8.69% <0.00%> (-91.31%)
:arrow_down:

wellcomeml/datasets/conll.py
12.50% <0.00%> (-87.50%)
:arrow_down:

wellcomeml/ml/cnn.py
12.25% <0.00%> (-80.40%)
:arrow_down:

wellcomeml/io/s3_policy_data.py
14.70% <0.00%> (-76.60%)
:arrow_down:

wellcomeml/ml/bilstm.py
16.66% <0.00%> (-75.06%)
:arrow_down:

wellcomeml/datasets/winer.py
8.90% <0.00%> (-73.29%)
:arrow_down:

wellcomeml/ml/clustering.py
17.07% <0.00%> (-73.18%)
:arrow_down:

wellcomeml/ml/spacy_ner.py
21.73% <0.00%> (-69.57%)
:arrow_down:

wellcomeml/ml/transformers_tokenizer.py
22.35% <0.00%> (-68.24%)
:arrow_down:

... and 22 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ba377a4...aba36d4. Read the comment docs.


I want to use a different column for provider instead of defalut "email"

I am running into a problem using "email" as a provider in my User table.  I am using the
symmetric_encryption gem to encrypt the email column in my postgres database. This requires to change the name of "email" column to "encrypted_email".  Also I am using "username" and "password" as login credentials with Devise instead of "email / password".
With standard Devise gem everything works fine and I am able to login using regular HTML version of login page. However, I can not get the authentication to work with devise_token_auth, since it insists of
using "email" as provider.
My question is: Is there any way I can use a different column (e.g. username) as a provider instead of
default "email" ? I browsed through the code, and it seems that "email" is hard-coded as a provider in session_controller.rb->create() method.
Please help !!!

This it similar and related to: https://github.com/lynndylanhurley/devise_token_auth/issues/563#issuecomment-195094536
My comment there has some information that might be able to help you get the sign in working, but it would be a hacky solution. I suspect it would require your email field be named exactly email, and the uid set in the headers in responses would be the value of that field.

Closed in favor of #563


Scroll Bar for Year Selection Dropdown on Teams Page Does Not Appear

I'm not sure what's causing this, but I've noticed that some on some team pages the year selection drop down is clipped by the surrounding border, and a scroll bar does not appear.
For some teams, such as Team 1, a scroll bar appears on the year dropdown.

On other teams, though, the scroll bar does not appear.
On Team 7172's page, selecting a year in which no events are recorded (such as 2015) decreases the size of the events box. Attempting to select a new year after this only allows you to chose the top two years.

Fixed


adding custom %TAG entries to file header

I think it would be nice to provide a way for custom extensions to add additional %TAG directives to the top YAML header to shorten tags provided by custom extensions that might lead to long tag strings. ASDF already does this by introducing %TAG ! tag:stsci.edu:asdf/ .
Consider the following example:
#ASDF 1.5.0
%YAML 1.1
%TAG ! tag:stsci.edu:asdf/
--- !core/asdf-1.1.0
# asdf headers
fraction: !<tag:nowhere.org:custom/fraction-1.0.0> [10, 3]
...

With adding  %TAG !foo! tag:nowhere.org:custom/ we get the shorter
#ASDF 1.5.0
%YAML 1.1
%TAG ! tag:stsci.edu:asdf/
%TAG !foo! tag:nowhere.org:custom/
--- !core/asdf-1.1.0
# asdf headers
fraction: !foo!fraction-1.0.0 [10, 3]
...

The only thing needed to get this output is to add the "!foo!": "tag:nowhere.org:custom/" to the tags mapping in the final     yaml.dump_all call in asdf.yamlutil.dump_tree here:
https://github.com/asdf-format/asdf/blob/8ccee8b17a053d6e33ebca81a4ed3f3c3569aee1/asdf/yamlutil.py#L408-L413
Currently tags is (statically) build to simply consist of {"!": "tag:stsci.edu:asdf/"}
One possible way to implement this would be to introduce a new (optional) property for the ExtensionClass that provides the tags.
Example extending the FractionExtension with a new property yaml_tag_context:
class FractionExtension(asdf.AsdfExtension):
    # ...

    @property
    def yaml_tag_context(self):
        return {"!foo!": "tag:nowhere.org:custom/"}

We can now query this property for registered extensions and add them to the list.
I tested this for simple read/write and it works, see this simple diff  https://github.com/asdf-format/asdf/compare/master...CagtayFabry:test_custom_yaml_tag
~Of course it would be nice to only load %TAG values from extensions that are actually used when creating the file (similar to listing them in the history)~ update

Hey cool I didn't know that was possible in YAML!  I like this idea.  We already track which extensions were used when writing the file so it should be straightforward to only add the necessary %TAG directives.

Hey cool I didn't know that was possible in YAML! I like this idea. We already track which extensions were used when writing the file so it should be straightforward to only add the necessary %TAG directives.

yes, getting the extensions from _serialization_context makes this really simple 👍
I updated the example above https://github.com/asdf-format/asdf/compare/master...CagtayFabry:test_custom_yaml_tag

Yes, I was also under the impression that only one of these %TAG was permitted. That would be a nice feature. I've been wondering how we could make tags abbreviated.

I am not specifically familiar with YAML specs in this regard but it appears multiple (different) handles are allowed in YAML 1.1 and 1.2
from https://yaml.org/spec/1.1/

Example 7.6. Tag Prefixes
%TAG !      !foo
%TAG !yaml! tag:yaml.org,2002:
---
- !bar "baz"
- !yaml!str "string"

also, multiple directives must not define the same prefix (I assume multiple handles in principle are allowed)

It is an error to specify more than one “TAG” directive for the same handle in the same document, even if both occurrences give the same prefix.

Resolved by #963


add null filter support

usage { field: { null: true } } for is null filter and { field: { null: false } } for not null filter

Hi! Thanks for the PR. Could you update the readme and show how it works :)

Hi, I've update the readme file. thanks :)

I can see you check params[:null] == true. Will that always work ? I would use ActiveRecord::Type::Boolean.new.type_cast_from_database to be sure that all 't', '1', 1, 'true', 'TRUE' and boolean true work :)
I can fix it later tonight once I merge no worries :)

yea i have test it and it work both for true and false value, but if you think it better to use type cast to make sure it work, thats good :)

@maceonrails it seems good! I thought I could branch in your branch but it's a pull request from another repo so it's impossible :)
Can you add ActiveRecord::Type::Boolean.new.type_cast_from_database in your code and squash your commits to 1 commit ? Then I will merge instantly :)

Not sure how I missed that, merged :)


Modify the instructions of installing certificates required for Visual Studio offline vs2018.8

The instructions of  Install certificates required for Visual Studio offline installation
isn't applicable because:

The certification downloaded in certification folder of vs2017.8 are:

manifestCounterSignRootCertificate.cer
manifestRootCertificate.cer
vs_installer_opc.RootCertificate.cer

it's cer format not p12 format and the names of files  in commandline are different.

In some computers, there is no commandline tool certmgr.exe available, only windows  crtmgr.msc (interactive tool).

Modify the instructions of installing certificates required for Visual Studio offline vs2018.8 using both the interactive tool certmgr.msc or the cmmandline tool (if the tool available).
Explain where the certification should be stored (user/computer account, . etc.)

Please note that in "Option 3 - Install certificates as part of a scripted deployment of Visual Studio" section, we added an example that shows certutil.exe, which Visual Studio 2017 version 15.8 Preview 2 or later uses. If this doesn't work for you, let us know by using the Report a Problem tool to report the issue.


Add method for saving PLC project as a library

This can be considered done, because one can access the PLC projects in an AutomationInterface  instance by, for example, doing this.

There was an issue created for the exception naming and creating a more sensible collection of the exceptions, so marking as approved


Cannot import without armatures

There should be a setting to not create any armatures & instead compute the relative positioning per object

Dropping this feature


Provisioning

Add instructions to provision openssh server.
https://learn.microsoft.com/en-us/windows/configuration/provisioning-packages/provision-pcs-for-initial-deployment

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 18a5a4e5-2604-23c2-1374-68a6974170bc
Version Independent ID: f93809cc-eb5e-6b9b-ed58-c8811a8ee21f
Content: Get started with OpenSSH for Windows
Content Source: WindowsServerDocs/administration/OpenSSH/OpenSSH_Install_FirstUse.md
Service: windows-server
Sub-service: administration
GitHub Login: @robinharwood
Microsoft Alias: roharwoo

We hugely appreciate your contribution, feedback, and support to our technical documentation.
We’ve phased out GitHub Issues on this repository as the feedback mechanism for content and have replaced it with a new feedback system. For more information see, Content User Feedback. As part of this rollout, we are migrating your issue where our team will use your comments to improve the documentation experience. Unfortunately, this does mean you won't be able to access this issue after the migration is complete.
All issues will be migrated by Thursday 29th August 2024.
Once again, thank you for your valuable feedback, we hope you’ll continue to contribute using our new feedback system.


Add badges to Readme

Please stop asking me to review draft PRs. That defeats the purpose of them.


Feature/router

Added factory() to Route interface and all routes/routestacks
Added new syntax to Segment route to fix a few problems

Reviewed, merged, and pushed to master.
Please actually run the tests before submitting the pull request. :) They didn't run on initial merge, and I did a lot of cleanup both in the tests (updated several to use data providers; switched from using constructors to using the new factory() methods), as well as the code (the tests found several problems, as it turned out, when they were able to run).
All is now running on master, and I've updated all sample/sandbox apps I have control of to ensure they work with the changes.


chore(prettier_conformance): ignore stage3 import assertions and import attributes

Current dependencies on/for this PR:

main

PR #1432   👈

This stack of pull requests is managed by Graphite.

Merge activity

Nov 19, 5:47 AM: @Boshen merged this pull request with Graphite.


fix for duplicate applying nodes

Make duplicate entries for .status.applying on plans magically disappear
by correctly sorting the return value for SelectConcurrentNodes.
Fixes #132
Additionally, address a logic bug introduced in #126 that could also
spin out of control.
See
https://github.com/rancher/system-upgrade-controller/commit/90e3d082a41fefaf09d14286b880c54a28af6c21
Signed-off-by: Jacob Blain Christen<EMAIL_ADDRESS>

See also, https://github.com/rancher/rancher/issues/32050#issuecomment-904762457


⚠ Cioos National CKAN has degraded performance

In 1e95213, Cioos National CKAN (https://catalogue.cioos.ca/) experienced degraded performance:

HTTP code: 200
Response time: 9225 ms

Resolved: Cioos National CKAN performance has improved in 1d4cad5 after 8 minutes.


Switch organization

This PR modifies the namespaces to use com.koddi instead of com.github.mcross1882

Tests failing because of API rate limits.


devicescript build does not build package.json entrypoint

It should look at package.json to find the default file.
$ devicescript build
using devs: v1.10.1, runtime: v6.2.0, node: v18.14.2 from /workspaces/devicescript/vscode/sampleprj/node_modules/@devicescript/cli/built
src/main.ts does not exist
Done in 0.45s.
@pelikhan ➜ .../vscode/sampleprj/packages/cowsay (rxjs) $

see rxjs branch, run yarn build in `observables.

Building package entry point would result in an empty binary (none of the functions are used at the top-level), so it's likely not what we need. I could change it so it finds any entry point (in this case src/main-test.ts) in src/main.ts is not found, or you can rename main-test.ts to main.ts

Right, using main.ts instead of main-test.ts would work.

Opted for using main.ts and not main-test.ts


Opening the repository takes a long time

Problem: Opening a large repo/package takes around 200,000 milliseconds.
We are looking at using abapGit to sync our ABAP main package to our internal github.
We are toying with the idea of using this for code review, and we cannot restrict objects in one transport to all belong to one package/repo. Hence synching at the highest package is our only way.
It contains around 8000 objects in total and about 30 subpackages.
I see for closed issue #537 is related and solved by limiting the number of entries displayed in the settings but didn't help us.
Most of the time is spent on serialisation, and i understand from the comments it is hard to achieve this without the use of function groups.
My questions is:

Could we persist the serialisation based on change timestamp?
Is there a way to not serialise every time?

I'm new here, so I am happy to contribute to the project (including coding), but I just want to ask the obvious question before wasting my time and everyone's time.

200000 milliseconds = 200 seconds =~ 3 minutes
The plan is to do serialization in parallel, see https://github.com/larshp/abapGit/issues/1714

#2061 might have also dropped performance by a bit for transportable packages. Can you check if lv_render_transports = abap_false. makes a significant difference?
https://github.com/larshp/abapGit/blob/e61108511dec241e9b24a9d61231303525ba6ac3/src/ui/zcl_abapgit_gui_view_repo.clas.abap#L631-L632

200000 milliseconds = 200 seconds =~ 3 minutes
The plan is to do serialization in parallel, see #1714
Function groups: idea is to only have parallel serialization supported in the full install, i.e. not in the 50k lines ABAP report

Thanks for the prompt response Lars.
Can you explain the rationale of serializing everytime, even for unchanged objects on either side? Is this a GIT restriction?

#2061 might have also dropped performance by a bit for transportable packages. Can you check if lv_render_transports = abap_false. makes a significant difference?
abapGit/src/ui/zcl_abapgit_gui_view_repo.clas.abap
Lines 631 to 632 in e611085
lv_render_transports = zcl_abapgit_factory=>get_cts_api(
)->is_chrec_possible_for_package( mo_repo->get_package( ) ).

Thanks for the suggestion.
Very similar ~200 seconds (3.5 mins).

rationale: this is just how it is currently implemented, not all objects have timestamps for latest update, and it is typically only down to the second. So the safe and easy approach is serializing everything every time

Saving timestamps opens for a lot of questions, eg. see https://github.com/larshp/abapGit/issues/1021 where the current non-persisted cache has some problems. Also, what if the serialization format changes, how will the timestamp know? Currently much of the git code works on full files, saved timestamps would probably require this to change.
Parallel serialization is on the other hand, more or less straight forward to implement, and will not need to change any design.
Contributing: get familiar with git and the github flow first, recommend making a small change first, there are lots of bugs 😄 Keep the pull requests as small as possible, this makes it easier to review, and if it is in a wrong direction not too much time is wasted. For larger/central changes recommend discussing the plan/approach first, but its free for all 🆓

@joymike also feel free to join the Slack channel, https://abapgit-slackinviter.herokuapp.com
I might have some time this week and start looking into Parallel Serialization, wont promise anything, just keep an eye on the pull requests, recommend watching the repo

Thanks Lars.
I fully agree parallel serialisation is a safe approach and inline with current design.
However I think caching is also a necessity and don't really see a real issue with has_changed_since concept.
I am familiar with git, and will try with something small first to test the waters.
I'll add further comments to #1021

Also, if your scenario is using git tools for code review, then I'm planning to sync the transports to a branch in background, so the developers does not have to change their workflow. And then performance does not matter too much

Wow... retested code with the latest new parallel feature. This has brought it down to 50-60 seconds (from about 4 mins for 8000 objects previously) using 21 threads which is a massive improvement and now much more acceptable to the end user. Many thanks @larshp . Awesome job!!


Add support to propagate activation lifecycle events when using hierarchical activators

REF: https://groups.google.com/forum/?fromgroups=#!topic/durandaljs/47LDtwD0VwI
When using hierarchical activators, Durandal does not propagate activation lifecycle events (e.g. activate/deactivate) to child activators. This means child view models never get a chance to respond to activation lifecycle events in situations where you have a complex application hierarchy.
A simple example of this problem can be seen in the Durandal Sample App in the Master Details example. If the project view model (defined in project.js) is active and the user navigates away by clicking the "Durandal Home" link, I would expect that the deactivate() method on the project view model would get called but this does not happen. As a result, the deactivate event does not get propagated to the project view model and the user is able to navigate away without any message box being displayed.
A more complex (but common) hierarchy scenario is as follows:
RootShell  (main router)
  SubShell1 (creates a child router)
    Module1 (uses an activator to compose sub view models)
      ViewModelA (exposes a activate method)
      ViewModelB
    Module2
    ...
  SubShell2
    ...

Using the hierarchy above lets say you are in /RootShell/SubShell1/Module1/ViewModelA. When you click on a link to invoke a route that navigates back to the RootShell, then the router will invoke the deactivate() method on the SubShell1 view model but will not invoke the deactivate() method on any of it's children view models (Module1 or ViewModelA). These view models will not be able to respond to these life cycle events.
Suggestions

[ ] Add support to Durdandal to improve this scenario and automatically forward lifecycle events to child activators. This will avoid having to add this pluming manually.
[ ] Update documentation on website to describe how to implement hierarchical activators.

This is not working properly. Here is an example:
parent route:
router.map([
      {route: ['','home'], moduleId:'viewmodels/home'},
      {route:['pages/:page'], moduleId:'viewmodels/pages'},
      {route:['analyse/:id*details'], moduleId:'viewmodels/analyse/index'}
    ]).buildNavigationModel();

child routes defined in viewmodels/analyse/index:
exports.router = router.createChildRouter().makeRelative({
    fromParent: true,
    dynamicHash: ':id',
    moduleId: 'viewmodels/analyse'
  }).map([
    {route:'reports', moduleId:'reports'},
    {route:'charts', moduleId:'charts'}
  ]);

follow these steps:

go to /analyse/12/reports: it should call activate on index and then on reports modules.
now navigate to /home: it calls deactivate on index which is correct but does not call deactivate on child view which is reports
now navigate back to /analyse/12/reports: it calls deactivate on reports (!!!) then calls activate on index and then on reports

All these happens if you mark the index module not to be canReuseForRoute. If you do not add this function to index then it NEVER calls deactivate or activate on child modules in step 3

I have tried above code but it is not working for me, I am disposing y object in deactivate method in child page but as deactivate of child does not call the obj does not dispose and create multiple instance everytime i route in an application.
Please help.

1


Quoting and subcommands

Here's the command I was trying to execute (essentially, pluck feature-branch from wherever it branched off master, and rebase it onto new-base), and the unexpected error message:
@ rebase --onto new-base $(merge-base master feature-branch) feature-branch
fatal: Needed a single revision
invalid upstream 'f1c4e38a3240741ce4cbc5f25b8c53c0a6cad6fafeature-branch'

This invocation succeeded:
@ rebase --onto new-base "$(merge-base master feature-branch)" feature-branch

Is this expected behaviour?

A simpler example to reproduce this:
:echo foo $(rev-parse HEAD) bar

Expected output:
foo f3b965188abd09aa0694270afbb69f8a0200a198 bar

Actual output:
foo f3b965188abd09aa0694270afbb69f8a0200a198bar

Looks like the root of this issue is in the lexer: the RIGHT_PAREN and SUBSHELL_END tokens use the regular expression /\s*\)\s*/. In other words, any whitespace following the ) is taken as part of the ) token.
The current implementation is nice because it handles pretty much all the whitespace at the lexer level, and the parser doesn't need to be peppered with “there might be whitespace here” annotations, but maybe we'll have to go down that road to resolve this issue.


Added Laravel 11 course in hindi

What does this PR do?
Add resource
For resources
Description
Laravel course with projects in hindi
Why is this valuable (or not)?
easy to understand language
How do we know it's really free?
its on youtube
For book lists, is it a book? For course lists, is it a course? etc.
youtube playlist
Checklist:

[x] Read our contributing guidelines.
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x]] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction).
[x] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

no changes


Outdated/confusing docs for setting Container maxWidth in theme.sizes.container

Subject
component
Description
I must admit that I'm a little confused what's the best way/best practice to use the Container.
In the Container docs it says:

By default, it sets the max-width of the content to 60 characters (60ch) but you can customize this by passing custom maxWidth values or changing the container size tokens defined in theme.sizes.container

First of all: I've never been in a project that used the character size as maxWidth (although I can see how it makes sense). Often frameworks would offer maxWidth values for certain breakpoints instead (like bootstrap etc. for example) or in projects we would follow the designers guidelines. Do you have a different experience? What's the reason behind this default? (I am just curious. As long as I can adapt it, I'm fine with this being the default)

you can customize this by passing custom maxWidth values or changing the container size tokens defined in theme.sizes.container

This sounds to me that changing theme.sizes.container would adapt every Container I use. But instead it just changes what I can reference in maxW, right? I find that part a little confusing and I think that adapting maxW everytime is more an exception. I would expect that the docs provide a best practice for the global option and only then describes how to adapt certain cases.
As far as I know my use case can be solved like that:
components: {
  Container: {
    baseStyle: {
      maxWidth: […]
    }
  }
}

Did you get my point? Any other perspectives or other ways your using Container?

Hey @webwelten !
I don't know what exactly was the reasoning behind the default value (maybe @segunadebayo can give you more insights about that). In any case the maxWidth in the Container is adjustable for your needs and use cases.
Overriding the values of the theme.sizes.container in your custom theme, changes the value of the tokens used in the maxWidth props as in this example. This would be my preferred approach You can find the default values in this file: https://github.com/chakra-ui/chakra-ui/blob/main/packages/theme/src/foundations/sizes.ts#L23
Your solution of changing the theme for the Container through a custom theme is also a valid way to address those changes.
Feel free to raise a PR with improvement to the docs. 😊🚀

I can comment that the default value of 60ch is the common approach to the best readability of a single column of text on desktop (a range of 45-75 characters per line, to be more precise).
You have to keep in mind though that this unit is based on the width of the character 0 which depends on the typeface used. Maybe it depends on context if this creates undesirable results when using more than one column?
Maybe this article could help with a sensible implementation of the unit, should you want to use it:
https://webflow.com/feature/control-width-of-text-elements-by-character-count-using-ch-unit

@noobinthisgame checking back in on this.
It's been a while now since this issue was posted. I can open a PR, as the details in the docs about the maxWidth are at the top of the page when they could be better placed with the Container Size section. And can link back to the Sizes Default Theme section in reference the container size token.


Add META-INF service for RestEASY ClientBuilder

Needed to make service lookup for RestEASY work correctly. Otherwise it
will fallback to org.glassfish.jersey.client.JerseyClientBuilder.
Signed-off-by: Andreas Häber<EMAIL_ADDRESS>

Sorry, missed the resteasy-client dependency.


FCE 209 faro logs

This PR introduces gathering logs, metrics, and events from frontend code. For that, we are using Faro Web SDK, which is pushing all of the data to Grafana Alloy container. Alloy then based on config pushes data to proper data storage in our case it is Loki. It partially implements the LGTM stack (Loki for logs, Grafana for visualization, Tempo for traces, and Mimir for metrics).

I would like to have a brief description in PR what's going on here. There is so many changes that I wasn't sure what's exactly connected with what 😅

Test are failing for some reason @Rados13


Failed instance creates just stay in "creating" forever

ERROR: Internal Server Error: {"error": "server error", "status": 500, "traceback": "Traceback (most recent call last):\n  File \"/usr/lib/python3/dist-packages/urllib3/connectionpool.py\", line 601, in urlopen\n    chunked=chunked)\n  File \"/usr/lib/python3/dist-packages/urllib3/connectionpool.py\", line 387, in _make_request\n    six.raise_from(e, None)\n  File \"<string>\", line 3, in raise_from\n  File \"/usr/lib/python3/dist-packages/urllib3/connectionpool.py\", line 383, in _make_request\n    httplib_response = conn.getresponse()\n  File \"/usr/lib/python3.6/http/client.py\", line 1356, in getresponse\n    response.begin()\n  File \"/usr/lib/python3.6/http/client.py\", line 307, in begin\n    version, status, reason = self._read_status()\n  File \"/usr/lib/python3.6/http/client.py\", line 276, in _read_status\n    raise RemoteDisconnected(\"Remote end closed connection without\"\nhttp.client.RemoteDisconnected: Remote end closed connection without response\n\nDuring handling of the above exception, another exception occurred:\n\nTraceback (most recent call last):\n  File \"/usr/local/lib/python3.6/dist-packages/requests/adapters.py\", line 449, in send\n    timeout=timeout\n  File \"/usr/lib/python3/dist-packages/urllib3/connectionpool.py\", line 639, in urlopen\n    _stacktrace=sys.exc_info()[2])\n  File \"/usr/lib/python3/dist-packages/urllib3/util/retry.py\", line 367, in increment\n    raise six.reraise(type(error), error, _stacktrace)\n  File \"/usr/local/lib/python3.6/dist-packages/six.py\", line 702, in reraise\n    raise value.with_traceback(tb)\n  File \"/usr/lib/python3/dist-packages/urllib3/connectionpool.py\", line 601, in urlopen\n    chunked=chunked)\n  File \"/usr/lib/python3/dist-packages/urllib3/connectionpool.py\", line 387, in _make_request\n    six.raise_from(e, None)\n  File \"<string>\", line 3, in raise_from\n  File \"/usr/lib/python3/dist-packages/urllib3/connectionpool.py\", line 383, in _make_request\n    httplib_response = conn.getresponse()\n  File \"/usr/lib/python3.6/http/client.py\", line 1356, in getresponse\n    response.begin()\n  File \"/usr/lib/python3.6/http/client.py\", line 307, in begin\n    version, status, reason = self._read_status()\n  File \"/usr/lib/python3.6/http/client.py\", line 276, in _read_status\n    raise RemoteDisconnected(\"Remote end closed connection without\"\nurllib3.exceptions.ProtocolError: ('Connection aborted.', RemoteDisconnected('Remote end closed connection without response',))\n\nDuring handling of the above exception, another exception occurred:\n\nTraceback (most recent call last):\n  File \"/usr/local/lib/python3.6/dist-packages/shakenfist/external_api/app.py\", line 108, in wrapper\n    return func(*args, **kwargs)\n  File \"/usr/local/lib/python3.6/dist-packages/flask_jwt_extended/view_decorators.py\", line 108, in wrapper\n    return fn(*args, **kwargs)\n  File \"/usr/local/lib/python3.6/dist-packages/shakenfist/external_api/app.py\", line 683, in post\n    return self._instance_start(instance_uuid, instance, network, namespace, placed_on)\n  File \"/usr/local/lib/python3.6/dist-packages/shakenfist/external_api/app.py\", line 723, in _instance_start\n    placement, instance_uuid, namespace)\n  File \"/usr/local/lib/python3.6/dist-packages/shakenfist/external_api/app.py\", line 754, in _instance_start_remote\n    'User-Agent': util.get_user_agent()})\n  File \"/usr/local/lib/python3.6/dist-packages/requests/api.py\", line 61, in request\n    return session.request(method=method, url=url, **kwargs)\n  File \"/usr/local/lib/python3.6/dist-packages/requests/sessions.py\", line 530, in request\n    resp = self.send(prep, **send_kwargs)\n  File \"/usr/local/lib/python3.6/dist-packages/requests/sessions.py\", line 643, in send\n    r = adapter.send(request, **kwargs)\n  File \"/usr/local/lib/python3.6/dist-packages/requests/adapters.py\", line 498, in send\n    raise ConnectionError(err, request=request)\nrequests.exceptions.ConnectionError: ('Connection aborted.', RemoteDisconnected('Remote end closed connection without response',))\n"}

The create here failed because it took more than 300 seconds...
root@sau-cca68-or:~# sf-client instance list
+--------------------------------------+-------------------------------------------------+-----------+------+--------+------------+-------------+----------+
|                 uuid                 |                       name                      | namespace | cpus | memory | hypervisor | power state |  state   |
+--------------------------------------+-------------------------------------------------+-----------+------+--------+------------+-------------+----------+
| dd72321f-5bb8-4fef-80d6-8abca48f60be |                   inst-on-sf-2                  |   system  |  1   |  1024  |    sf-2    |   initial   | creating |
+--------------------------------------+-------------------------------------------------+-----------+------+--------+------------+-------------+----------+

Instead, the instance should be in the "error" state.

I think this one is resolved by the queue work, but we should validate.

This works correctly now and has done for a while.


Optimized resource utilization for decoupled mode

Modified streamer, removing need for Q_srl in decoupled mode (and generally when the number of streams is 1 or 2).

@maltanar A couple of problems with this branch:

it shows up as conflicted in github although in my own environment the conflict doesnt exist
running any of the decoupled mode rtlsim tests for streamingfclayer results in a segfault inside verilator when pyverilator calls the eval function (I assume this is when the verilator objects are actually first created). The python process hangs and the stack trace is dumped to screen, i wasnt able to get more info from it.

Problem 2 fixed

Problem 1 fixed as well, ready for review

I got test failures for 13 of the test_fpgadataflow_fclayer test with this branch:
FAILED tests/fpgadataflow/test_fpgadataflow_fclayer.py::test_fpgadataflow_fclayer_rtlsim[16-16-2-2-DataType.BIPOLAR-DataType.BIPOLAR-DataType.BIPOLAR-decoupled]
FAILED tests/fpgadataflow/test_fpgadataflow_fclayer.py::test_fpgadataflow_fclayer_rtlsim[16-16-1-1-DataType.BIPOLAR-DataType.BIPOLAR-None-decoupled]
FAILED tests/fpgadataflow/test_fpgadataflow_fclayer.py::test_fpgadataflow_fclayer_rtlsim[16-16-1-1-DataType.BIPOLAR-DataType.BIPOLAR-DataType.BIPOLAR-decoupled]
FAILED tests/fpgadataflow/test_fpgadataflow_fclayer.py::test_fpgadataflow_fclayer_rtlsim[16-16-1-1-DataType.BIPOLAR-DataType.BIPOLAR-DataType.INT4-decoupled]
FAILED tests/fpgadataflow/test_fpgadataflow_fclayer.py::test_fpgadataflow_fclayer_rtlsim[16-16-1-1-DataType.BIPOLAR-DataType.INT4-None-decoupled]
FAILED tests/fpgadataflow/test_fpgadataflow_fclayer.py::test_fpgadataflow_fclayer_rtlsim[16-16-1-1-DataType.BIPOLAR-DataType.INT4-DataType.BIPOLAR-decoupled]
FAILED tests/fpgadataflow/test_fpgadataflow_fclayer.py::test_fpgadataflow_fclayer_rtlsim[16-16-1-1-DataType.BIPOLAR-DataType.INT4-DataType.INT4-decoupled]
FAILED tests/fpgadataflow/test_fpgadataflow_fclayer.py::test_fpgadataflow_fclayer_rtlsim[16-16-1-1-DataType.INT4-DataType.BIPOLAR-None-decoupled]
FAILED tests/fpgadataflow/test_fpgadataflow_fclayer.py::test_fpgadataflow_fclayer_rtlsim[16-16-1-1-DataType.INT4-DataType.BIPOLAR-DataType.BIPOLAR-decoupled]
FAILED tests/fpgadataflow/test_fpgadataflow_fclayer.py::test_fpgadataflow_fclayer_rtlsim[16-16-1-1-DataType.INT4-DataType.BIPOLAR-DataType.INT4-decoupled]
FAILED tests/fpgadataflow/test_fpgadataflow_fclayer.py::test_fpgadataflow_fclayer_rtlsim[16-16-1-1-DataType.INT4-DataType.INT4-None-decoupled]
FAILED tests/fpgadataflow/test_fpgadataflow_fclayer.py::test_fpgadataflow_fclayer_rtlsim[16-16-1-1-DataType.INT4-DataType.INT4-DataType.BIPOLAR-decoupled]
FAILED tests/fpgadataflow/test_fpgadataflow_fclayer.py::test_fpgadataflow_fclayer_rtlsim[16-16-1-1-DataType.INT4-DataType.INT4-DataType.INT4-decoupled]

One error from Verilator (didn't check if all cases have the same) seems to be:
%Error: /tmp/finn_dev_maltanar/code_gen_ipgen_StreamingFCLayer_Batch_0_hkan0y0i/project_StreamingFCLayer_Batch_0/sol1/impl/verilog//memstream_singleblock.v:139: [-1:0] Range extract has backward bit ordering, perhaps you wanted [0:-1]

Could you have a look? (ps: it's possible to do e.g. python setup.py test --addopts "-k test_fpgadataflow_fclayer_rtlsim[16-16-1-1-DataType.INT4-DataType.INT4-DataType.INT4-decoupled]" to pick out individual configs)

For some reason the first failing test passed in my environment; the other ones fail because mem depth is 1 and the address signal size is incorrectly calculated. Will fix.

The fclayer rtlsim tests all pass now (looks like the first failed test with an HLS crash that seems to happen sometimes, I couldn't reproduce it either), but now we have another problem. I tried building the  end2end zynqbuild cnv-w1a1 with this branch, to quantify how the resource usage changes. Somehow the mem blocks instantiated by the streamer are implemented as "LUT logic", which runs out of LUTs for the device and fails synthesis, see screenshot above.

I manually set RAM_STYLE=block for some of those layers, still the same result so something mysterious is going on. Looking at the synthesis reports, I can see e.g.
INFO: [Synth 8-3971] The signal "ramb18_wf_dualport:/mem_reg" was recognized as a true dual port RAM template.

Block RAM: Final Mapping  Report
+--------------------+------------+------------------------+---+---+------------------------+---+---+------------------+--------+--------+
|Module Name         | RTL Object | PORT A (Depth x Width) | W | R | PORT B (Depth x Width) | W | R | Ports driving FF | RAMB18 | RAMB36 |
+--------------------+------------+------------------------+---+---+------------------------+---+---+------------------+--------+--------+
|ramb18_wf_dualport: | mem_reg    | 256 x 512(READ_FIRST)  | W | R | 256 x 512(READ_FIRST)  | W | R | Port A and B     | 1      | 14     |
+--------------------+------------+------------------------+---+---+------------------------+---+---+------------------+--------+--------+

but also:
WARNING: [Synth 8-6014] Unused sequential element inst/mem/singleblock.mem/use_ram.ram/mem_reg was removed.

@maltanar After my latest commit i believe the branch should pass all tests that do not require the Pynq shell

Thanks @quetric -- looking good. Just leaving this here for future reference, here are the resources for cnv-w1a1 zynqbuild before and after this PR:
before: Post-synthesis resources (excluding shell): {'LUT': 26487.0, 'SRL': 3964.0, 'FF': 34031.0, 'BRAM_36K': 94.0, 'BRAM_18K': 73.0, 'DSP48': 0.0}
after: Post-synthesis resources (excluding shell): {'LUT': 21135.0, 'SRL': 1665.0, 'FF': 31517.0, 'BRAM_36K': 98.0, 'BRAM_18K': 9.0, 'DSP48': 0.0}


Update dependencies

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 2 (0.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.07%) to 33.836%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/EasyPDF/ClassEasyPDF-Barcode.cs
0
2
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.07%

Covered Lines:
354

Relevant Lines:
961

💛  - Coveralls


feat/place-validation-display

Please review before merging.

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


Re-generating da-y-wrapper causes error in the browser

When re-generating da-y-wrapper even without any changes, the result produces the following error in the browser console when editing a DA page:
TypeError: Cannot read properties of undefined (reading 'state')
    at il.dispatchTransaction (index.js:107:32)
    at il.dispatch (index.js:8:46597)
    at index.js:12:8601
    at ao.mux (index.js:12:3924)
    at ao._forceRerender (index.js:12:8377)
    at Object.view (index.js:12:5740)
    at il.updatePluginViews (index.js:8:44335)
    at new il (index.js:8:41375)
    at initProse (index.js:281:17)
    at DaEditor.updated (da-editor.js:91:25)

Here's what I did:

Take a fresh clone of da-live, run npm i
Sanity check before doing anything -> aem up and check a page on localhost:3000 and see that it works, then kill the process.
run npm run build:da-y-wrapper
run aem up again and then go to the same page. The editor won't properly load and the exception above can be seen in the browser console.

Re-opening because the live preview was still broken.


refactor: convert root to typescript

@haxxmaxx I am considering removing the types.ts file in the codeclimate. What do you think?

Not sure it is good idea. As I mentioned before, we should probably split them up so you have maybe types for the components and types that are used everywhere in seperate files. There is a good reason why a file shouldn't be too big

you can ignore it for now though


-1 Days unsupported (International Date Line)

Describe the bug
As auro-flight begins being used in award flight shopping, international date line concerns need to be addressed.
For instance, take this flight:
https://www.google.com/travel/flights/booking?tfs=CAIQAhpEagwIAxIIL20vMDM0dGwSCjIwMjMtMDEtMDFyBwgBEgNITkwiHwoDR1VNEgoyMDIzLTAxLTAxGgNITkwqAlVBMgMyMDBwAYIBCwj___________8BQAFIAZgBAg&source=flre_fli_share&utm_campaign=sharing
This flight crosses the international date line, causing a "-1 Day". Auro-flight doesn't seem to support negative date calculations.
This will be an issue as we sell more western pacific rim -> HI flights.
To Reproduce

Expected behavior
-1 day to be displayed.

I wouldn't call this a bug, but this is a new feature request. Re-classifying this issue.
@leeejune we should talk with the UI design team about how to best illustrate this experience. -1 Day might not be the right way to do this?
The United example linked as the most minimal designation. I had to look hard to find the -1 day.

Expedia actually shows this as a +1 Day O_O.

This issue requires further refinement with the team.

As discussed with the team, the work appears to be minimal. Update the current +1 Day feature to understand the concept of -1 Day.
Still need to consult with @leeejune and the other designers to be sure that we are simply going to update the current +1 Day UI with this feature.
specification
Update the current javascript that determines +1 Day and support -1 Day
Exit criteria
This issue will be considered complete once we have a working example that uses the GUM to HNL example data and produces the correct results.

As discussed with the team, the work appears to be minimal. Update the current +1 Day feature to understand the concept of -1 Day.
Still need to consult with @leeejune and the other designers to be sure that we are simply going to update the current +1 Day UI with this feature.
specification
Update the current javascript that determines +1 Day and support -1 Day
Exit criteria
This issue will be considered complete once we have a working example that uses the GUM to HNL example data and produces the correct results.

Assigned to @leeejune for design feedback.

Discussed issue with UI team. Will be reaching out to the research team about testing the "-1" solution.

Research will be testing out the solution in future studies. Timeline is not clear, but research has informed me that it will be prioritized for the next study.

Study shows the the "-1 day" solution works. Next steps are to add support for this feature.
More details in this deck.

This is a current issue in production:

@blackfalcon @irma-kurnia-alaska should this issue be closed now that #109 is merged?


Not Overriding Globals Environment Variable

Hello,
I have a couple of functions in my template.yml and I have declared a Globals section in which I see all of them have an env variable TABLE_NAME.
Sam does not seem to pick the Globals section when trying to override the env with --env-vars
Thank you for your help
AWSTemplateFormatVersion: 2010-09-09
Transform: AWS::Serverless-2016-10-31
Description: functions

Globals:
  Function:
    MemorySize: 192
    Timeout: 120
    Environment:
      Variables:
        TABLE_NAME: !Ref JobsTable

Parameters:
  targetStage:
    Description: Define stage to which Lambdas/API Gateways should be deployed.
    Type: String
    Default: dev
    AllowedValues:
      - dev
      - integ
      - prod
    ConstraintDescription: Only stages dev, integ and prod are allowed
  TABLE_NAME:
    Type: String

Resources:
  JobsTable:
    Type: AWS::DynamoDB::Table
    Properties:
      TableName: !Sub "Job-${targetStage}"
      AttributeDefinitions:
      - AttributeName: id
        AttributeType: S
      KeySchema:
      - AttributeName: id
        KeyType: HASH
      ProvisionedThroughput:
        ReadCapacityUnits: 5
        WriteCapacityUnits: 5

  GetJobList:
    Type: AWS::Serverless::Function
    Properties:
      FunctionName: !Sub "getJobs-${targetStage}"
      Handler: com.airliquide.aries.lambda.handlers.JobHandler::getList
      CodeUri: ./target/functions-1.0-SNAPSHOT.jar
      Policies: AmazonDynamoDBReadOnlyAccess
      Runtime: java8
      Timeout: 100
      Events:
        Api:
          Type: Api
          Properties:
            Path: /job
            Method: get

  CreateJob:
    Type: AWS::Serverless::Function
    Properties:
      FunctionName: !Sub "createJobs-${targetStage}"
      Handler: com.airliquide.aries.lambda.handlers.JobHandler::createJob
      CodeUri: ./target/functions-1.0-SNAPSHOT.jar
      Policies: AmazonDynamoDBFullAccess
      Runtime: java8
      Timeout: 100
      Events:
        Api:
          Type: Api
          Properties:
            Path: /job
            Method: put

  UpdateJob:
    Type: AWS::Serverless::Function
    Properties:
      FunctionName: !Sub "updateJobs-${targetStage}"
      Handler: com.airliquide.aries.lambda.handlers.JobHandler::updateJob
      CodeUri: ./target/functions-1.0-SNAPSHOT.jar
      Policies: AmazonDynamoDBFullAccess
      Runtime: java8
      Timeout: 100
      Events:
        Api:
          Type: Api
          Properties:
            Path: /job
            Method: post

I'm having the same problem.

Fixed in the latest release - https://github.com/awslabs/aws-sam-cli/releases/tag/v0.3.0