runtime/metrics: add /gc/heap/live:bytes

There appears to be no way to monitor what is called the Live Heap in the GC Guide.

The closest runtime/metric seems to be /memory/classes/heap/objects:bytes which is Live Heap + New Heap (i.e. includes potentially dead but unswept allocations).
Monitoring Live Heap seems to be useful when running with GOGC=off and a GOMEMLIMIT to understand if the live heap of a program is approaching the GOMEMLIMIT which could lead to high CPU usage or OOM kills.
Therefor I'm proposing to add /memory/classes/heap/live_objects:bytes and /gc/heap/live_objects:objects to the runtime/metrics package. They should return similar values as what is currently returned by the InUse* values of the runtime.MemProfile() profile. It's probably worth to discuss whether the term InUse is preferable over Live.
I created a very hacky prototype of this here, but there is probably a much better way to do this.

cc @mknyszek

A similar proposal was rejected previously (#51966). Part of the problem was inclusion in MemStats, but runtime/metrics also existed and I made some other arguments against it. Mainly, I think relying on it solely as an indicator can be somewhat fragile if you put it in context of the Go 1.18 GOGC changes. I wasn't ever fundamentally opposed to this, but at the time for the use-case presented, it just didn't seem worth it. I also wanted to think more carefully about what implications it could have (and why we didn't do it already).
However, I do think your argument that it's useful for identifying issues with a memory limit is "new information" (now that the memory limit has landed) as far as the proposal processes goes. I think that might be enough to push this over the edge into more serious consideration for me personally.
I think to start with I'd be inclined at this point to call this /gc/heap/live:bytes. This way it lives next to /gc/heap/goal:bytes. /memory/classes to me is more about a breakdown of memory at a higher level and all the metrics inside there are supposed to be orthogonal. We'd have to break up some of the other metrics and I don't think it's worth it for this.
If we do this, we should probably also export scannable stack bytes and scannable globals bytes, at least as far as they go into the GOGC calculation. We should also really export GOGC and the memory limit as well, since they can change during application execution.
I don't really see enough utility from exporting the count of live objects though. I think a memory profile is better for that sort of thing anyway. (Also, we don't currently track that (mod memory profiling) and it's more work.)

I created a very hacky prototype of this here, but there is probably a much better way to do this.

Yeah, there's a simpler way for bytes. :) You want gcController.heapMarked. You can just read that directly in the metrics callback without any synchronization since it only gets updated during a STW.

Sounds great. I'll update my patch based on your suggestions.

I also wanted to think more carefully about what implications it could have (and why we didn't do it already).

Yeah, that makes sense. Seems like this is missing the code freeze anyway.

I updated the CL to use gcController.heapMarked, but found one odd thing.

If we do this, we should probably also export scannable stack bytes and scannable globals bytes, at least as far as they go into the GOGC calculation. We should also really export GOGC and the memory limit as well, since they can change during application execution.

I can prepare CLs for those as well. Should I submit them separately?

I updated the CL to use gcController.heapMarked, but found one odd thing.

It's an accounting artifact currently required for good performance. It would be nice to fix, but I'm not yet sure how without either (1) impacting the allocation fast path or (2) slowing down metrics.Read to ReadMemStats levels (this path would avoid the global latency impact that ReadMemStats has because it's not necessary to actually, fully stop the world, but the worst-case latency of metrics.Read becomes effectively the same as stopping the world).
For users that really need this consistency (like the testing package), ReadMemStats provides it, so there is a workaround. Most of the time though, the skew doesn't make a very meaningful difference, and the memory stats (in /memory/classes) are all at least guaranteed to be consistent with each other. It might be worth mentioning in the docs that this stat specifically isn't necessarily consistent with the /memory/classes stats. We can lift this restriction in the future if we can figure out a way to flush cached efficiently (I'll keep thinking about it).

If we do this, we should probably also export scannable stack bytes and scannable globals bytes, at least as far as they go into the GOGC calculation. We should also really export GOGC and the memory limit as well, since they can change during application execution.

I can prepare CLs for those as well. Should I submit them separately?

Either or. Your current CL is not very big.

This proposal has been added to the active column of the proposals project
and will now be reviewed at the weekly proposal review meetings.
— rsc for the proposal review group

Based on the discussion above, this proposal seems like a likely accept.
— rsc for the proposal review group

No change in consensus, so accepted. 🎉
This issue now tracks the work of implementing the proposal.
— rsc for the proposal review group

@felixge were you planning to implement this?

@felixge were you planning to implement this?

Yes, I'll try to submit a good patch for /gc/heap/live:byte 👍.

If we do this, we should probably also export scannable stack bytes and scannable globals bytes, at least as far as they go into the GOGC calculation. We should also really export GOGC and the memory limit as well, since they can change during application execution.

If time allows I could also try to look into these, but I'm not sure if they'd be covered by the accepted proposal or would need separate proposals?

That's a good question. I personally feel we should just include those in the proposal.

All of the metrics discussed in this proposal have landed, see patches above. Closing this 🥳
Thanks @mknyszek et al. for reviews and discussion 🙇


[Docs] Remove Swift Version from Jazzy

The incorrect Swift version was causing Jazzy to fail generating docs. We should leave the setting to always use the latest Swift version implicitly.

Codecov Report

Merging #247 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #247   +/-   ##
=======================================
  Coverage   95.56%   95.56%
=======================================
  Files          16       16
  Lines         766      766
=======================================
  Hits          732      732
  Misses         34       34

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 ea98256...047e7ca. Read the comment docs.

@ReSwift/reswift Anyone mind signing off on this so we can test if it works out when building master?

I added a Rakefile to automate doc generation so folks like me don't have to look this up.
After running jazzy, I get a list of files that are not part of the repo:
	Docs/Classes.html
	Docs/Classes/
	Docs/Protocols.html
	Docs/Protocols/
	Docs/Structs.html
	Docs/Structs/
	Docs/badge.svg
	Docs/css/
	Docs/docsets/
	Docs/img/carat.png
	Docs/img/dash.png
	Docs/img/gh.png
	Docs/index.html
	Docs/js/
	Docs/search.json
	Docs/undocumented.json

Should we add them to .gitignore since the CI tool will probably build that locally?

@DivineDominion Actually the doc generation is already automated as part of the .travis.yml file: https://github.com/ReSwift/ReSwift/blob/master/.travis.yml#L43-L46
The deployment was just breaking due to the incorrect Swift version specified in the jazzy config file. So I think we should look into getting feedback on failed automatic generation of the docs. I don't think we need the Rakefile in this case. Does that make sense? 😃

Oh no, @DivineDominion, I didn't notice that the Rakefile was commited on top of this PR. Will remove it with a separate PR. In general I think it would be best to open separate PRs instead of working off of existing ones 😄

Oh, sorry for the trouble. Figured this belonged to "fix docs generation" to test docs locally without much hassle, but it did sure feel a bit intrusive.

No big deal :) I also could have taken another look at the PR before merging 😉
If you think the Rakefile is still useful for testing locally, we should just open a separate PR :)

Right, done! #250


UVVM page description

Your UVVM page description says, "Open Source VHDL Verification Library and Methodology - ..."
Please note that "Open Source VHDL Verification Methodology" is a trademark of SynthWorks Design Inc.   What you have used is a too close to matching our trademark.
Perhaps you can update it to match what is in your readme file.

Page decsription has been updated. Closing.
Br,
Marius


[W6.1d][T15-B3] Dickson Chan

added isMutating method in Command class, and revised the Command codes to override isMutating methods, to show polymorphism behaviour

@itsdickson
Some comments added. Please ack & close the PR after reading comments.


Accessing a helm chart repo running on a port other than 80

Output of helm version:
version.BuildInfo{Version:"v3.5.0+6.el8", GitCommit:"77fb4bd2415712e8bfebe943389c404893ad53ce", GitTreeState:"clean", GoVersion:"go1.14.12"}
I'm seeing some unexpected behaviour when trying to access a helm chart repository running on a port other than 80.  It appears that this capability is almost all there, but isn't complete. (For what it's worth, I've tried this on x86_64, ppc64le and s390x and the behaviour is the same)
$ helm repo add local-repo http://<IP_ADDRESS>:8000
"local-repo" has been added to your repositories
$ helm repo update
Hang tight while we grab the latest from your chart repositories...
...Successfully got an update from the "local-repo" chart repository
Update Complete. ⎈Happy Helming!⎈
$ helm repo list
NAME      	URL
local-repo	http://<IP_ADDRESS>:8000
$ helm pull local-repo/moo-chop
Error: Get "http://<IP_ADDRESS>/moo-chop-0.1.0.tgz": dial tcp <IP_ADDRESS>:80: connect: connection refused
Note the port number here.  It's trying 80 instead of 8000
$ helm pull http://<IP_ADDRESS>:8000/moo-chop-0.1.0.tgz
$ ls -1 moo-chop-0.1.0.tgz
moo-chop-0.1.0.tgz
As you can see, it's possible to add a repository running on a high port (in this case, we're running python3 -m http.server), but when we try and do a pull (or a show) it seems to be ignoring the port number and defaulting back to port 80.
Is this a bug? (i.e. it should honour the port number specified in the repo list?) or is this a use case that isn't supported? (i.e. some input checking should be done on repo add to disallow specifying a port number?)

I just tried to reproduce this on the latest helm v3.6.0 and was not able to. I see you are using a version that's a commit that's not part of the Helm source tree.
Could you try reproducing on the latest Helm release?

Thanks @mattfarina on further investigation this issue occurred because on an incorrectly generated helm repo index file.  There's no issue.  Thanks for investigating.


Educational outcomes: math proficient

Description: percent of 3rd-8th grade students that are math proficient by where students live
Logic:
Source(s): DOE; HPD 2017-2018 WWL data
WWL Figure 5.75. Use WWL data for EDDT launch. DOE will supply annual data for future years.Cannot report last two years because of COVID.  Waiting on link/folder
Year(s):

[ ]

Geographies:

[ ] PUMA
[ ] Borough
[ ] City

Race and ethnicity breakdown:
Denominator:
SME:
Questions:
Roadblocks: Waiting for source data

Years: 2018 (nothing newer because of COVID)
Race/Ethnicity: confirming with DOE
Denominator: We will get percent values directly from DOE and won't be reporting total values (confirming as well)
We also need to add ELA scores (this was a late addition to the data dictionary), but it uses the same data source from DOE.

Columns to be divided -- from 5_StudentPerformance tab in NTA_data_prepared_for_ArcMap_wCodebook spreadsheet in linked folder:
AR/AL(All Students);
AS/AM(Asian);
AT/AN(Black);
AU/AO(Hispanic);
AV/AP(Other);
AW/AQ(White)
https://nyco365.sharepoint.com/:f:/r/sites/NYCPLANNING/civic-engagement-studio/Shared Documents/03. Working Materials/03.  Equity and Displacement Vulnerability/HPD-DCP EDDT Coordination/02. Indicators/EDDT QOL Indicators/DOE WWL Data/DOE Access to Opportunity Files/DOE Clean Data/Request 4 and 5 - Location of Proficient Schools and NTA Outcomes?csf=1&web=1&e=rV6lvK


Do generated views and locales have to be renamed?

I'm really confused about this Can I use them as is or do I have to rename them? Also I ran rails g devise:views and couldn't find "Invalid Email or password." in the newly generated devise.en.yml file Why is that?

@mices you can keep them where they are generated, i.e. under app/views/devise, or move them to specific places if you override the controllers... for example if you create your own SessionsController < Devise::SessionsController, you can technically put the sessions views under app/views/sessions instead of app/views/devise/sessions. Both should work with the Rails template lookup.
As for the devise.en.yml, that contains all text translations for validations errors and flash messages. Views themselves aren't using that translation, you can update the views directly yourself, or use some 3rd-party like https://github.com/tigrish/devise-i18n that does use locale files.
The "Invalid Email or password" in particular is this one: https://github.com/heartcombo/devise/blob/7f419bf99adb9463617be7ccb0852d5f8f7b0317/config/locales/en.yml#L12, but Email is dynamic there... (email is your "authentication key", but that could be a username field, or something else entirely, so we interpolate that dynamically.... but nothing stops you from changing that to anything else)
Hope that helps!


Add Data Converter to support custom serialization/deserialization

Users now can implement encoded.DataConverter and let Cadence use it to serialize/deserialize parameters that need to be sent over the wire.
DataConverter should be set in 2 places:

Worker.
When create worker, one can provide data converter in WorkerOptions. Then everything processed by that worker will use provided data converter.
Client.
When create client to perform operations, one can provide data converter in clientOptions. Then all operation will use provided converter to encode arguments.

Coverage increased (+0.8%) to 66.359% when pulling d4cacf378794cc8f4c216a2e2bce6b13cc11a44a on dc2 into bbc380ff417578fa74466f9d5bd76fd32e003859 on master.


react to removal of PlatformAbstractions

This PR removes all references to PlatformAbstractions.
Part of aspnet/PlatformAbstractions#50

@anurse,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

Ping @JunTaoLuo . This change is just to remove platform abstractions from the metapackage.


add python 3.11 in ci

blocked by https://github.com/mapbox/rio-color/pull/82

I've forked rio-color and remove rasterio/rio-mucho requirement to create a lightweight package: https://github.com/vincentsarago/color-ops
I'm going to replace rio-color with color-ops as soon the package will be on pypi


Cassandra example and other minor updates

[Current coverage][cc-pull] is 82.67%

Merging [#471][cc-pull] into [master][cc-base-branch] will decrease coverage by 0.32%

File ...ark.CSharp/Worker.cs (not in diff) was modified. more

Misses +17
Partials -1
Hits -16

@@             master       #471   diff @@
==========================================
  Files            47         47
  Lines          4952       4952
  Methods           0          0
  Messages          0          0
  Branches        517        517
==========================================
- Hits           4110       4094    -16
- Misses          600        617    +17
+ Partials        242        241     -1

Powered by Codecov. Last updated by [18668c3...46db916][cc-compare]
[cc-base-branch]: https://codecov.io/gh/Microsoft/Mobius/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/Microsoft/Mobius/compare/18668c3c58e25278d8d2d9017d1774ec3ab4efd4...46db916dceda549723073ccfb34cb4652585f09c
[cc-pull]: https://codecov.io/gh/Microsoft/Mobius/pull/471?src=pr

ping...

Added a few minor comments above. VS version consistency is important. Other than that, LGTM.


externs/browser/w3c_encoding.js deviates from spec

First off, the spec is a WHATWG publication, not a W3C publication:
 * @fileoverview Definitions for W3C's Encoding specification
 *     https://encoding.spec.whatwg.org

Right there in the URL. I don't know if the W3C has bothered to snapshot/republish the spec for giggles, but may want to fix that, rename the file, etc.
Second, the decode() method is annotated to take a non-nullable required Uint8Array as the first argument. Per spec, the first argument is an optional BufferSource.
Third, the decode() method is annotated to take a required string as the second argument. Per spec, it's an optional string.
extern:
/**
 * @param {!Uint8Array} input
 * @param {Object=} options
 * @return {string}
 */
TextDecoder.prototype.decode = function decode(input, options) {};

spec:
USVString decode(optional BufferSource input, optional TextDecodeOptions options);

Fourth, the TextEncoder constructor is given an options argument, but one is not present in the spec.
extern:
/**
 * @constructor
 * @param {string=} encoding
 * @param {Object=} options
 */
function TextEncoder(encoding, options) {}

spec:
[Constructor(optional DOMString utfLabel = "utf-8"),
 Exposed=Window,Worker]
interface TextEncoder { ... };

Fifthly (sorry), the encode() method is annotated to take a required string as the first argument. Per spec, it's an optional string:
extern:
/**
 * @param {string} input
 * @return {!Uint8Array}
 */
TextEncoder.prototype.encode = function(input) {};

spec:
 [NewObject] Uint8Array encode(optional USVString input = "");

Want to send a pull request? At least the required->optional changes should easy to land.

Can do, but as I normally just rely on the downloaded .jar it may take a bit.

Yay, thanks @phistuck !

@inexorabletash - after https://code.google.com/p/chromium/issues/detail?id=435727 - I think I owe you! :)


sponsor information

rough draft. Current issues for @patrick91

[x] on localhost, getting unhandled runtime error: Hydration failed because the initial UI does not match what was rendered on the server. Not sure if it is a problem only for local.
[ ] accordion style doesn't work, targeting the look of https://ep2022.europython.eu/sponsor/information

@patrick91 as requested, mentioning you to prompt a wee reminder. Hoping to publish this by end of next week :)

@patrick91 build seems to have failed? (Not sure I am reading this right on the phone).

@hypha yes, the lockfile wasn't updated, should be ok now ;)

@Kislovskiy could you check if the wording I added in https://github.com/EuroPython/website/pull/259/commits/e0bcac7675c4f505617ff805ac809dc224aa563e looks OK? Not sure if it is overly formal.


(PDB-3099) implicit facts->inventory relationship

We had the other half of this covered already.

Test PASSed

CLA signed by all contributors.

:+1: will merge once the release goes out


Error when trying to read CNES STPI2201.dbc

Transforming STPI2201.dbc
Traceback (most recent call last):
File "/mnt/c/Users/gcmas/OneDrive/Área de Trabalho/S3 Biotech/Desenvolvimento/ETL Growth/main.py", line 156, in
cnes_etl.run()
File "/mnt/c/Users/gcmas/OneDrive/Área de Trabalho/S3 Biotech/Desenvolvimento/ETL Growth/CNES/etl.py", line 80, in run
self.transform()
File "/mnt/c/Users/gcmas/OneDrive/Área de Trabalho/S3 Biotech/Desenvolvimento/ETL Growth/CNES/etl.py", line 64, in transform
transform.transform_dbc_files(self.path_temp, PATH_UNZIPED, self.target_files)
File "/mnt/c/Users/gcmas/OneDrive/Área de Trabalho/S3 Biotech/Desenvolvimento/ETL Growth/CNES/transform.py", line 27, in transform_dbc_files
df = pd.DataFrame(read_dbc(path_temp + file))
File "/usr/local/lib/python3.10/dist-packages/dbfread-2.0.7-py3.10.egg/dbfread/dbf.py", line 122, in init
self._read_field_headers(infile)
File "/usr/local/lib/python3.10/dist-packages/dbfread-2.0.7-py3.10.egg/dbfread/dbf.py", line 235, in _read_field_headers
field.name = self._decode_text(field.name.split(b'\0')[0])
File "/usr/local/lib/python3.10/dist-packages/dbfread-2.0.7-py3.10.egg/dbfread/dbf.py", line 215, in _decode_text
return data.decode(self.encoding, errors=self.char_decode_errors)
UnicodeDecodeError: 'ascii' codec can't decode byte 0xba in position 3: ordinal not in range(128)

I don't know what I am seeing here. If you want my help I need you to help me help you. What is the smallest snippet of code that reproduces this error? What were you trying to do? What were you expecting to see?
I need something like this https://github.com/stevemao/github-issue-templates/blob/master/simple/ISSUE_TEMPLATE.md


Differential Calculator

Issue Id you have worked upon - #343
Briefly explain your program logic -
Made a differentiation calculator that takes equation and order, and calculate differentiation of equation order times.
Screenshots(Attach 2 screenshots of your own input and output) -

By raising this PR I affirm that -

[x] My code follows the guidelines of this project.

[x] I have performed a self-review of my own code.

[x] I have commented my code.

[x] My code gives the correct output.

[x] I affirm that I strictly follow contributing guidelines and code of conduct.

Congratulations 🎉 @nipun221 on your contribution.
Thanks for adding value to this repository. 😊
Let's celebrate this fest 😃 by providing more developers 👨💻👩💻 an opportunity to contribute. Help us make this repository reach far and more by sharing this to your friends and colleagues.
Your efforts can make difference.

Hey 👋, may you also encourage us by clicking the star icon on the repository main page. ⭐

@nipun221,
Really impressive work. Very organized code and UI display. Adding admin-appreciation points for the work presented. 👏👏


Extend options to support passthrough

Description of problem
Today, if a workload does not match any of the options, we explode. We are suggesting that it is possible that a workload purposely does not want to match any of the options and would instead like the ability to move on to the next step.
Proposed solution
https://gist.github.com/squeedee/ed833b036ecc8f983d5aaf5801cc5fa8

Do we have any thoughts what the shape of status.resource[x] should be for the "skipped" resource?
Possibly:
    - name: source-tester
      inputs:
        - name: source-provider
       outputs:
        - name: url
          preview: |
           ...
        - name: revision
          preview: |
            ...


Add docs about cross compiler setup

One large barrier towards people helping the project (and contributing to osdev in general) is having to set up a cross compiler. If there were docs pages about setting up a cross compiler, it would make Carbon more accessible.

Also, move kernel/docs to docs.


Missing "ver" query argument on stylesheets

I use WordPress' asset versioning system to bust cache on the dynamic version of my site, and noticed that the ver query arg appears to be removed (it's in the source of the dynamically-generated page, but not in the source of the WP2Static generated site).
Note: the ver argument does get includes for external JavaScript files.

@crstauf shouldn't be an issue in V7. Moved to this repo to track V6 issues, so we can start fixing and maintaining for users unable to move to V7 yet

Thanks, @leonstafford!


How to remove or hide the Remove tab bar option and copy current URL options

How to remove or hide the Remove tab bar option and copy current URL options

Closing as dup of https://github.com/jiahaog/nativefier/issues/1044


Orbital elements

Changed ephermis function to ephemeris_vector
Added the ability to fetch keplarian orbital elements from JPL's Horizons API

Will be a breaking change, due to ephemeris being renamed to ephemeris`_vector

Cool, thanks!


Shipping Address Support

Hi y'all,
Any plans to implement support for shipping addresses in the near future? If not, I'd be happy to submit a PR along those lines.
Thanks!

Please do go ahead and submit a PR!


Thaw/fluent demo fixes

Fixed the search.
Light/dark theme selection is now stored in local storage. It looks like it was supposed to be stored in the URL but it wasn't fully implemented.
Added discord button.
Fixed issue with message bar markdown.
Removed options param from auto complete.
I didn't add keyboard functionality to the auto complete. I did some testing and it can be done using AutoCompleteInjection as a proxy between AutoComplete and children but I'm not sure if it's the right way to do it.

I didn't add keyboard functionality to the auto complete. I did some testing and it can be done using AutoCompleteInjection as a proxy between AutoComplete and children but I'm not sure if it's the right way to do it.

We can investigate how FluentUI does it.

https://github.com/microsoft/autocomplete-search-box - that's all I found, I don't see auto complete component on the react.fluentui.dev
So they pass options as a list and use optional custom renderers. What's the purpose of changing this to accept children? What additional features would this allow us to add?

Refer to the Combobox component.
Every time a new field is added to the previous AutoComplete structure, it is destructive.

https://github.com/microsoft/fluentui/blob/master/packages/react/src/components/ComboBox/ComboBox.tsx - no idea how it works. I see the components receiving 'props.options' but  I don't know how the  children are transformed into an array or IComboBoxOption type.  I'm leaving it for now.
I will fix the table issue.
Next I will move the dropdown component to fluent branch.

Ok, I added "attr:style" to the table generated from markdown but this branch is still using leptos 0.6 so I can't see the effect. I've seen the fluent branch with leptos 0.7 so I'm assuming it will work there and leave it for now.

no idea how it works. I see the components receiving 'props.options' but I don't know how the children are transformed into an array or IComboBoxOption type.

We can discuss this in Discord.

Thank you for working on this.


add missing sqlparse dependency

Checklist

[x] Used a fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

see:

https://github.com/django/django/commit/f82be9ebc79e30fef796bbc61fec4b9694cb779f
https://github.com/conda-forge/django-feedstock/pull/75

Thanks for catching this @nrbgt, not sure how I missed this when syncing with conda-forge.

Thanks!

not sure how I missed this when syncing with conda-forge.

No worries: they added their second(!) dep on a patch release, so it caught us by surprise on CF, too.
I've considered PR'ing the tests to catch these kinds of things, as the complexity might start ramping up as they start towards async stuff.


Release v6.0.0

Changes

NPM release
Remove Zepto support in favor of jQuery
Migrate all dependencies to NPM, remove Bower
Migrate JS linting to eslint
Migrate SCSS compilation to libsass
Lint SCSS with sass-lint
Update circleCI config with modern testing steps
Add some GitHub templates for contributing, issues, and PRs

How to Test

Checkout code
Review package.json
Review Changelog
npm install
grunt test or grunt test:browser
Check examples with grunt examples

Todos:

[ ] +1
[x] Add an .npmignore
[ ] Post merge: Publish to NPM

👍 Tests pass, examples work as expected.


Unable to export

Import: Asset collection has versions probably incompatible with each other. Here they are:
Import: 4.7.0f1
Import: 4.6.4p4
Export: Can't export

 'Roboto-Black_copy(Font)' because it isn't valid
General: System.IO.FileNotFoundException: Could not find file 'Ripped\mainData\Shader'.
File name: 'Ripped\mainData\Shader'
   at System.IO.__Error.WinIOError(Int32 errorCode, String maybeFullPath)
   at System.IO.Directory.InternalCreateDirectory(String fullPath, String path, Object dirSecurityObj, Boolean checkHost)
   at System.IO.Directory.InternalCreateDirectoryHelper(String path, Boolean checkHost)
   at UtinyRipper.AssetExporters.AssetExportCollection.Export(ProjectAssetContainer container, String dirPath)
   at UtinyRipper.AssetExporters.ProjectExporter.Export(String path, IEnumerable`1 objects)
   at UtinyRipper.Program.Load(IReadOnlyList`1 args)

https://mega.nz/#!MPxRzQRY!zTRa9A0yzGD2sFUUrGMvg-CeeArN8HUH2wwmA-y2QSI

There is no any issues with this file. Problem is with some of its dependencies

How to resolve then ?

Find out which file has this issue and send it. Or send all files and I'll find it myself.


Windows 10 bug [System.ComponentModel.Win32Exception]

System.ComponentModel.Win32Exception
The system cannot find the file specified
Boolean StartWithCreateProcess(System.Diagnostics.ProcessStartInfo) @ System.Diagnostics.Process
   at System.Diagnostics.Process.StartWithCreateProcess(ProcessStartInfo startInfo)
   at System.Diagnostics.Process.Start()
   at System.Diagnostics.Process.Start(ProcessStartInfo startInfo)
   at UninstallTools.Factory.ScoopFactory.RunScoopCommand(String scoopArgs)
   at UninstallTools.Factory.ScoopFactory.GetUninstallerEntries(ListGenerationCallback progressCallback)
   at UninstallTools.Factory.ApplicationUninstallerFactory.GetMiscUninstallerEntries(ListGenerationCallback progressCallback)
BCUninstaller.dll [<IP_ADDRESS>]
<IP_ADDRESS>
24/02/2022 10:52:47 AM
5.0.13

NativeErrorCode: 2
ErrorCode: -2147467259
HResult: -2147467259

This is a fresh install. This happens on launch.

IS it possible that you might have leftovers from previously installed Scoop? Because if you get this error, it means the code path has found something in the places in the GetScoopInfo() method and then return true for ScoopIsAvailable. But when it comes to running it, BCU could not find the scoop in the path.
It is also possible to add a File.Exists(_scoopUserPath) but to me, it is an edge case to handle for Scoop, not for BCU.
private static bool ScoopIsAvailable
        {
            get
            {
                if (!_scoopIsAvailable.HasValue)
                {
                    _scoopIsAvailable = false;
                    GetScoopInfo();
                }
                return _scoopIsAvailable.Value;
            }
        }

        private static void GetScoopInfo()
        {
            try
            {
                _scoopUserPath = Environment.GetEnvironmentVariable("SCOOP");
                if (string.IsNullOrEmpty(_scoopUserPath))
                    _scoopUserPath = Path.Combine(WindowsTools.GetEnvironmentPath(CSIDL.CSIDL_PROFILE), "scoop");

                _scoopGlobalPath = Environment.GetEnvironmentVariable("SCOOP_GLOBAL");
                if (string.IsNullOrEmpty(_scoopGlobalPath))
                    _scoopGlobalPath = Path.Combine(WindowsTools.GetEnvironmentPath(CSIDL.CSIDL_COMMON_APPDATA), "scoop");

                _scriptPath = Path.Combine(_scoopUserPath, "shims\\scoop.ps1");

                if (File.Exists(_scriptPath))
                {
                    if (!File.Exists(PathTools.GetFullPathOfExecutable("powershell.exe")))
                        throw new InvalidOperationException(@"Detected Scoop program installer, but failed to detect PowerShell");

                    _scoopIsAvailable = true;
                }
            }
            catch (SystemException ex)
            {
                Console.WriteLine(ex);
            }
        }

I have scoop installed. It exists.
The environment variable is set: SCOOP                          C:\Users\Emily\scoop
The shims\\scoop.ps1 file exists. I don't know why this is erroring.

This looks like a Heisenbug. I installed scoop for this. Installed yara with scoop for testing and uninstalled it via BCU. No error. Can you provide steps for reproducing -that might be important?

I have reinstalled scoop. I have reinstalled BCUninstaller. I have installed BCU through scoop itself, and manually. No improvement.
I don't know how to reproduce this. I have set both environment variables, just incase.
I'm happy to try anything to help debug this, but I don't know where to start.

@purple-emily Can you check if running powershell.exe works from the Run dialog (Win+R)?

Can you try using this build? If the issue is with powershell not being in PATH then it should fix it. UninstallTools.zip

Can you try using this build? If the issue is with powershell not being in PATH then it should fix it. UninstallTools.zip

Perfect!! It works!
I dont know what you did but that solved it.


Add missing appearance transition forwarding

Forward appearance transition methods to old center view controller
during animated "Quick View Change".

Great catch. But I'm getting a little concerned with how hard the appearance logic is to follow across the various close methods... I need to think about how this can be refactored.


feat: action redesign, UQI upgrade AWS plugin config to dual zone format

Description
Upgrade AWS lambda plugin config to new format using SECTION_V2, SINGLE_COLUMN_ZONE, and DOUBLE_COLUMN_ZONE.
Fixes #35488
Automation
/ok-to-test<EMAIL_ADDRESS>:mag: Cypress test results

[!CAUTION]
If you modify the content in this section, you are likely to disrupt the CI result for your PR.

Communication
Should the DevRel and Marketing teams inform users about this change?

[ ] Yes
[x] No

/build-deploy-preview skip-tests=true

This PR has been tested on CE & EE (modules, workflows).


Trace: unit tests are non-deterministically broken on Windows

e.g.:
================================== FAILURES ===================================
__________________ Test__TraceAPI.test_list_traces_no_paging __________________
Traceback (most recent call last):
  File "C:\projects\google-cloud-python\trace\tests\unit\test__gax.py", line 215, in test_list_traces_no_paging
    _rfc3339_to_datetime(span['startTime']),
  File "c:\projects\google-cloud-python\.nox\default\lib\site-packages\google\cloud\_helpers.py", line 274, in _rfc3339_to_datetime
    dt_str, _RFC3339_MICROS).replace(tzinfo=UTC)
  File "c:\python27\Lib\_strptime.py", line 332, in _strptime
    (data_string, format))
ValueError: time data '2017-11-08T21:41:48Z' does not match format '%Y-%m-%dT%H:%M:%S.%fZ'
1 failed, 15 passed in 0.48 seconds

I think this is related to #4323. I had to make changes there because of .isoformat() quirks on Windows. It looks like the .00000 seems to be truncated in .isoformat()?

isoformat() docs say: "Return a string representing the date and time in ISO 8601 format, YYYY-MM-DDTHH:MM:SS.mmmmmm or, if microsecond is 0, YYYY-MM-DDTHH:MM:SS"
So probably need:
try:
    return datetime.datetime.strptime(
        dt_str, _RFC3339_MICROS).replace(tzinfo=UTC)
except ValueError:
    return datetime.datetime.strptime(
        dt_str, _RFC3339_NO_FRACTION + 'Z').replace(tzinfo=UTC)

The unit test for Test__rfc3339_to_datetime should also manually construct some of these troublesome dates and test various round-trips.
It might not hurt to check leap-seconds either, since they will almost certainly fail.

Since this is from the gax library, and I do not see any test_list_traces_no_pages in the current library, I'm going to close this.  I have also looked at test_list_traces and test_list_traces_exception and see nothing similar.
Please open a new issue if this comes up again.


Remove usage of __TABLES__

resolves #113
rebase of #364

Key comments from description of https://github.com/dbt-labs/dbt-bigquery/pull/364 that we'll want to consider sufficiently prior to merging this PR.
Trade-offs and outstanding questions
The problem is that it doesn't contain row_count or size_bytes, so it is not a perfect drop-in replacement. This could be possibly overcome by using the INFORMATION_SCHEMA.TABLE_STORAGE view to get the number of rows + the size in logical bytes, but that has its own complications (see below).

Is it okay to go from reporting the row count and size in bytes and then degrade to 0 for both?

Is there a way to make this work without an environment variable? e.g., is there a way for the adapter to know the relevant region(s)?

Are there any region-specific effects that would could cause a table/view/external to not be reflected?
Is this method slower, faster, or the same?

Given the tradeoffs, I'm quite weary of removing displaying row count and size given how many folks have found that to be helpful.


User Images are rotated & too large.

When adding an image, they are rotated 90 degrees to the right, and images taken with a real device are too large.

Fixed


Get rid of 'moment' as a dependency

What about this issue ?
Back to backlog or closed ?

@cloderic more info ?

done by https://github.com/craft-ai/craft-ai-client-js/pull/205


makeStyles.js:3 Uncaught Error: MUI: makeStyles is no longer exported from @mui/material/styles. You have to import it from @mui/styles.

makeStyles.js:3 Uncaught Error: MUI: makeStyles is no longer exported from @mui/material/styles.
You have to import it from @mui/styles.

Just import mui/styles and it should work 🙂 . Please follow up


"Implicit braces are forbidden" triggered with constant in class body

Reproduction similar to #68
class ABC

  @CONST_1 = 'test'

  constructor: (abc)->
    @def = abc * 2

  getDef: -> @def

{
    "no_implicit_braces": {
        "level": "error",
        "strict": false
    }
}

output:
[~]$ coffee --version
CoffeeScript version 1.6.3
[~]$ coffeelint --version
1.5.5
[~]$ coffeelint -f coffeelint.cfg badclass.coffee
  ✗ badclass.coffee
     ✗ #5: Implicit braces are forbidden.

✗ Lint! » 1 error and 0 warnings in 1 file

+1 this - it is causing some erroneous linting errors on our build we can't get rid of as the code is valid, so we've had to turn the rule off which is a pain.  As a note we're using strict: false.

:+1:

i'm not entirely sure if this is fixed because this code which is in a class body gives me the same error "Implicit braces are forbidden.".
`OPTIONS =
type            : 'type'
autoDelete  : yes
constructor: (@owner, options = {}) ->`

This is already addressed in #484

awesome :+1:


community-operators [N] [CI] assisted-service-operator (0.0.1)

The bot told me in #3603 that I needed to update the ci.yaml and do so in a separate PR. So here we are :sunglasses:


[Android]Working with validation of an email address and password client side, by using regular expressions

Proposed title of article
Working with validation of an email address and password client side, by using regular expressions
Proposed article introduction
We will learn, how to validate an email address and password client side, by using regular expressions (regex).
This way we can check, if the inserted email address is formed like an email address, and if the password contains certain characters, like upper and lower case letters, digits and special characters.
For this we will use a class, with which we can compile a regex from a string and then use the matcher method to compare it with the input.
Key takeaways
1.Creating a class that is going to handle regular expression
2.How to validate the email
3.Checking  if the inserted email address is formed like an email address
Article quality
Its gives a complete guide step by guideline of validation of email and password
References
N/A

Good afternoon and thank you for submitting your topic to the EngEd program.
After some careful consideration, it struck us that this topic may be a bit over-saturated throughout other blog sites and official documentations.
We typically refrain from publishing content that is covered widely on the net or other blogs. We're more interested in original, practitioner-focused content that takes a deeper dive into programming-centric concepts.
But in order to approve the topic, it has to serve value to the larger developer community at large.
An option and a great way to write this as an in-depth article and make it more add value to the greater developer community at large would be to walk the reader through the USE of methods and functions by building a unique, different, useful project.

We would love to learn your thought process behind the solution you arrived at using these concepts and topics.

That way a developer could see them in action.
As mentioned above - we believe this topic is widely covered on other blog sites.
The best way for students to build a great portfolio is by building what does not exist and what can provide the most value.


Support OS X

Only thing breaking OS X compatibility was UIImage use. Easy fix.

Thats a good way to fix that. There are a couple of issues though.

The podspec does not pass linting you can try it out by doing pod lib lint
There should probably be a demo target for os x
Tests should also be added for os x

Cool! I can handle updating the tests. What's the lint error?

It had to do with Alamofire and deployment targets. Just pushed a fix.

Awesome! I'll take a look at tests when I get home. Thanks again!

Going to merge this as-is; I've opened https://github.com/Moya/Moya/issues/265 to address the CI. Thanks!

Yayyy, thanks

Out of curiosity, what's the flow from tag necessitating change, to new CocoaPods version? Would be cool to see OS X support on trunk, but not sure if there's a lot to do (for example, if addressing the CI issue is pre-req) before making a push?

Hey, so that's a great question. Normally, tags are created with the automated release task. I just forgot to run the task after merging this >.<

Okeedoke, good to know

New version is up: https://raw.githubusercontent.com/CocoaPods/Specs/87704665c95c9c49e1324991ee94142c3504ded6/Specs/Moya/4.1.0/Moya.podspec.json


[state sync] redo upstream logic in peer manager

Motivation
Update state sync peer manager:

use PeerNetworkID instead of PeerID to identify peers in different networks
use UpstreamConfig instead of state_sync.upstream_peers config
move out network senders from peer manager
clean up unnecessary API and internal peer management logic (e.g. set_peers)

Test Plan
Existing tests

:umbrella: The latest upstream changes (presumably 9f69011556ba313919065645e1e62a9ee69e2b78) made this pull request unmergeable. Please resolve the merge conflicts.

@bors-libra r=phoenix-antigravity

:pushpin: Commit a2f323c has been approved by phoenix-antigravity

:hourglass: Testing commit a2f323cc5d51fd8013b32d635d065909f20d37e1 with merge 6a896cb2ac681e81046f10123d1747f3118e8f1b...

:sunny: Test successful - checks-actions_land_blocking_test, checks-circle_commit_workflow
Approved by: phoenix-antigravity
Pushing 6a896cb2ac681e81046f10123d1747f3118e8f1b to master...


Problem start linux

Existed vmlinux26.bin when in jslinux.js (after patch) vmlinux-2.6.20.bin
Line 77 in jslinux.js

I have recently updated the code with new jslinux version. This should solve the issue. Please reopen it as a new issue if it did not worked for you.


[Feature request] “issues” feature

Description
After switching from Omi to Jellyseerr I was really impressed! But I also realised there is no “issues” feature to report issues with existing media. This would be a very helpful feature.
Overseerr implemented such a feature in v1.28.0 (https://github.com/sct/overseerr/releases/tag/v1.28.0). So hopefully this makes the whole implementation process a lot easier.
Desired Behavior
Being able to report issues with existing media trough Jellyseerr.
Additional Context

Issues feature is present in the develop tag. We will merge develop branch with latest very soon. For now please use develop tag for the latest features of overseerr

Issues feature is present in the develop tag. We will merge develop branch with latest very soon. For now please use develop tag for the latest features of overseerr


[minor] sitemap https://www.evelynandarthur.com/sitemap_pages_1.xml is returning empty sites array

i know this relates to #34, but this xml doesn't have multiple <sitemap> tags.
https://www.evelynandarthur.com/sitemap_pages_1.xml
what could be wrong? we are using your package to gather urls before testing (a11y).

I’m assuming you already tried increasing the timeout? Seems like a pretty small file, so I don’t think that would have issues, but I’m not able to debug the issue at this exact moment.

yup. tried everything from the mentioned issue.
it would be awesome to help us with this problem, when you have some time to tackle this.

It seems as though this request is forbidden by everlynandarthur.com:
Forbidden Request
The server understood the request,
but is refusing to fulfill it because access is
forbidden to the requested resource.

"<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">

    <meta http-equiv="content-type" content="text/html;charset=UTF-8" />
    <title>Forbidden</title>

403Forbidden Request

The server understood the request, but is refusing to fulfill it because access is forbidden to the requested resource.

Back to previous page | shopify.com

"

thanks @seantomburke


windows dual compiler cmake build & test

windows build and test

LGTM. Do we have to add the build instructions in the README.md?

Eventually, but for now just getting it all set up.


[Bios 906] Bios Update tutorial is not precise enought.

The BIOS update 906 doesn't come with an EFI folder, so we need to copy the EFI folder from the 603 and proceed by typing commands, you should add that to the tutorial !
Do you want me to give you the commands ?

As you can see in my BIOS update section I have the following:

UPDATE: For the latest XMAKB5R0P0906 BIOS update, just unzip the folder and execute the H2OFFT-Wx64.exe executable, in a Windows installation.

It does not need anything else, it is a different process :)

Hello @Cr0wTom, but this doesn't work for those who are using Mac or Linux, but we can proceed as follow:

Download the 906 and 603 BIOS updates.
Unzip the 603 update to a FAT32 formatted drive, with the EFI directory at the root of the USB drive.
Copy the BIOS update file XMAKB5R0P0906.fd from 906 and paste it at the root of the USB Stick.
Reboot and mash  until you have the boot menu and boot to the USB Stick.
When you have a shell, type the following command:

H2OFFT-Sx64.efi XMAKB5R0P0906.fd

Wait for it to reboot, flash with a progress bar, and reboot again.

Hey, has anybody tried this? I would like to update from 502 to 906 without bricking using this method (I am using a linux distro).
Or is this safe?

Yes, I did, it works great :)

I think it's safe, but you take your own risks :)

@Nurrl Thanks for the response, I'll try this method somewhere next week, I'll keep you up-to-date

Hello @Cr0wTom, but this doesn't work for those who are using Mac or Linux, but we can proceed as follow:

Download the 906 and 603 BIOS updates.
Unzip the 603 update to a FAT32 formatted drive, with the EFI directory at the root of the USB drive.
Copy the BIOS update file XMAKB5R0P0906.fd from 906 and paste it at the root of the USB Stick.
Reboot and mash <F12> until you have the boot menu and boot to the USB Stick.
When you have a UEFI Shell, type the following command:

H2OFFT-Sx64.efi XMAKB5R0P0906.fd

Wait for it to reboot, flash with a progress bar, and reboot again.

Tried this But getting an error that "Permission denied by user". Anyone faced this? and How can I fix this?

Hello @Cr0wTom, but this doesn't work for those who are using Mac or Linux, but we can proceed as follow:

Download the 906 and 603 BIOS updates.
Unzip the 603 update to a FAT32 formatted drive, with the EFI directory at the root of the USB drive.
Copy the BIOS update file XMAKB5R0P0906.fd from 906 and paste it at the root of the USB Stick.
Reboot and mash <F12> until you have the boot menu and boot to the USB Stick.
When you have a UEFI Shell, type the following command:

H2OFFT-Sx64.efi XMAKB5R0P0906.fd

Wait for it to reboot, flash with a progress bar, and reboot again.

Tried this But getting an error that "Permission denied by user". Anyone faced this? and How can I fix this?

I have the same problem……

Closing as the issue is fixed. If you see any discrepancy in the repo please issue a pull request and I will try to merge it asap.


Add info grant to get in work with non-TM software

Hello, there is a problem with grants in the script
See more
https://github.com/AdguardTeam/AdguardForWindows/issues/3300#issuecomment-636026007

I took a look at the original issue, and it looks like its Adguard blocking the script. Adding the grant looks like a workaround that adds a sandbox layer so Adguard won't think it's something originally in the page. I'll change that.


Error: cannot import name 'PartialMessageable' from 'discord.channel'

Hi, I have an error:
Traceback (most recent call last):
  File "site.py", line 2, in <module>
    from quart_discord import DiscordOAuth2Session, requires_authorization, Unauthorized
  File "/home/jourdelune/.local/lib/python3.8/site-packages/quart_discord/__init__.py", line 4, in <module>
    from .client import DiscordOAuth2Session
  File "/home/jourdelune/.local/lib/python3.8/site-packages/quart_discord/client.py", line 3, in <module>
    import discord
  File "/home/jourdelune/.local/lib/python3.8/site-packages/discord/__init__.py", line 25, in <module>
    from .client import Client
  File "/home/jourdelune/.local/lib/python3.8/site-packages/discord/client.py", line 53, in <module>
    from .webhook import Webhook
  File "/home/jourdelune/.local/lib/python3.8/site-packages/discord/webhook/__init__.py", line 12, in <module>
    from .async_ import *
  File "/home/jourdelune/.local/lib/python3.8/site-packages/discord/webhook/async_.py", line 46, in <module>
    from ..channel import PartialMessageable
ImportError: cannot import name 'PartialMessageable' from 'discord.channel' (/home/jourdelune/.local/lib/python3.8/site-packages/discord/channel.py)

Finnaly, after re install PyCord, that work


bug:  MainController: nvim_execute_lua: Wrong type for argument 1, when using example from docs

Check the following:

[X] I have checked the behavior after setting "vscode-neovim.neovimClean" in VSCode settings.
[X] I have read the vscode-neovim docs.
[X] I have searched existing issues.

Neovim version (nvim -v)
NVIM v0.9.4
Operating system/version
Windows 11 + Ubuntu 22.04 WSL
Describe the bug
When using the example for jk from the docs
 "vscode-neovim.compositeKeys": {
        "jk": {
            // Use lua to execute any logic
            "command": "vscode-neovim.lua",
            "args": [
                [
                    "local code = require('vscode-neovim')",
                    "code.action('vscode-neovim.escape')",
                    "code.action('workbench.action.files.save')"
                ]
            ]
        }
    },

I am getting this error when i jk
nvim_execute_lua: Wrong type for argument 1 when calling nvim_execute_lua, expecting String
Steps To Reproduce
Copy the code from the docs into settings.json and try using jk, clean neovim installation with empty lua.init
Expected Behavior
Would work as intended

Wait for the new version. Please check the README on the vscode extension detail page.

Any eta on the new release?


Fix for #95  RM with mailer block syntax leads to errors (symbolize keys)

Did not touch tests as this can only be tested with an integration test?

Current unit tests are failing (via Travis). Can you take a look?

Sorry. New PR with passing unit tests


Jlisowskyy/setup host site test framework

Things done in this PR:

Added python code to parse output of alkos and process tests correctly
Implemented stub code on kernel site to test the python code
Added stub conf.bash file at this stage will be expanded in future PRs
Added logging to python code
Added nice CLI to python code

With the introduction of python code, i just wanna ask. Should we now use pylint / pep8?


changed snmpwalk command for faster overall plugin response, replaced…

… walk for get in storageSize, storageSizeUsed and storageAllocationUnits. Changed regex in storageID since it wasn't returning anything in a DS415play nas

Checked and confirmed. Still works for DS713+. Thanks!

;)
Le lundi 6 juin 2016, Roland Rickborn<EMAIL_ADDRESS>a écrit :

Merged #2 https://github.com/exensio/synology-nagios-plugin/pull/2.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/exensio/synology-nagios-plugin/pull/2#event-682532332,
or mute the thread
https://github.com/notifications/unsubscribe/AFlG_Y66cN3zdVbCWuf7VLAVsMFhEbsGks5qI9ibgaJpZM4IbAgX
.

--
Att.
Luiz de Andrade


Discussion about allowing HTML

I want to discuss here of what should we do about trusting html.
I saw various ways, and I am not sure of any of them:
As today, you can add html onto it, $sce will trust your html but I am unsure how it works vs XSS. Using ngSanitize is also an option, but it doesn't allow to use form items on the toast.
The problem with $sce is that you can't put directives into the trusted HTML (that is intended). That doesn't mean I can create a directive where you can put any kind of html on your toast (even forms that will work with your scope), but that is highly insecure.
I want to discuss what are your thoughts about this.

i got $sce warning with

AngularJS v1.5.7
toastr 2.1.1 (not validate)

my intention enabling html just using
it work on http success callback with toastr.success
and it's throw $sce error when on fail callback, even with no html


make import clearer

The current readme calls the guidance module directly which is not possible given that it is a module and not a callable.

Thanks. However, in this case the module is callable (for convenience). You can ignore this if you like and just use guidance.Program instead.
Below is what makes the module callable:
https://github.com/microsoft/guidance/blob/c63b59774f83e9a83fe69a8931c46cc2b6458a54/guidance/__init__.py#L18-L22


a solution found for search issue

hello
please do this way: [add &exact=true ]
https://www.imdb.com/find/?q=spider 2002&s=tt&exact=true&ref_=fn_tt_ex
compare with default search:
https://www.imdb.com/find/?s=tt&q=spider 2002&ref_=nv_sr_sm
and see why we have many mistakes
thanks for your amazing brains [both of you]
I did NOT find this solution
a user found it and asked me to send you in site

You can close this issue, already post it in issues.


Add evaluate to trainer

Added Evaluate to classical RL trainer. Thoughts on this?

Still working on this?

Is this fine or should I add something else?

Looks good to me, have you tested this on all algos?

Maybe we should think about renaming Trainer to Runner in the future, if we want to have functions like evaluate.

Yup i tested this on both QLearning and SARSA.

Maybe we should think about renaming Trainer to Runner in the future, if we want to have functions like evaluate.

Seems like a better idea to me because right now for evaluating we need to run agent.evaluate, which seems odd to me

Should I add evaluate to trainer in deepRL as well in this PR only?

I am going to add evaluate for DeepRL trainers as well.

Wow. Sounds good!

I have tested every deeprl agent on CartPole-v0, got a mean reward of >120 for every agent after 2 epochs. Let me know if you are not able to replicate.

Ok cool. What about DDPG, TD3 and SAC?

DDPG, TD3 and SAC?

Yup these too, made some changes to them.

Oh wait SAC is left. I'll add tests as well. Sorry.

How but? DDPG, TD3 and SAC are not supposed to work for discrete envs 🤣

Maybe check Pendulum-v0 for them

Can you save the agents for these? And record a video for them? (We could add these in readme)
Recording just requires  monitoring the env.

How but? DDPG, TD3 and SAC are not supposed to work for discrete envs

I don't know. It ran successfully :P

Ok. I think CartPole-v0 should reach 200, as its the easiest environment.
I dont know how but this was an error with Vecenvs as well (It was only reaching a reward of 120). We should probably debug this.
Pendulum-v0 reward should reach around -120
CartPole-v0 should be 200
CartPole-v1 should be 500

I think CartPole-v0 should reach 200

Yup i tested this for DQN without vectored env it reaches 200.

I dont know how but this was an error with Vecenvs as well

Regarding this I'll check the subprocess code again, maybe there is an issue there.

Can you save the agents for these? And record a video for them?

Cool, I'll add these in another PR. Let's not add it here.

Ok sure, sounds good to me. But just save the agents and video (or hyperparams) to avoid doing the hardworking

On 12-May-2020, at 7:34 PM, Het Shah<EMAIL_ADDRESS>wrote:
I think CartPole-v0 should reach 200
Yup i tested this for DQN without vectored env it reaches 200.
I dont know how but this was an error with Vecenvs as well
Regarding this I'll check the subprocess code again, maybe there is an issue there.
Can you save the agents for these? And record a video for them?
Cool, I'll add these in another PR. Let's not add it here.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/SforAiDl/genrl/pull/116#issuecomment-627366335, or unsubscribe https://github.com/notifications/unsubscribe-auth/AH72FJ27RC5GHPL6HZQ5AGLRRFJNXANCNFSM4M5FQDLA.

Yea sure.

Also, why are there buffer errors now. I did not make any change to them.

Not sure, you could try running the tests locally?

Yea now even locally they are showing up.  What changed?

I don't know, I restarted everything now. Pushing every agent one by one.

I am done now I guess with every agent. Review this lol.

I think changes are minimal so we can merge. Probably does not need another review from @ajaysub110

Totally upto you.

Aah more work. Nice :P

I'll fix them
On Wed, 13 May 2020, 19:21 Het Shah<EMAIL_ADDRESS>wrote:

Aah more work. Nice :P
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/SforAiDl/genrl/pull/116#issuecomment-628000506, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AGQWIYZZ3MBK7XS2ZIXFOK3RRKQWLANCNFSM4M5FQDLA
.

wait i pushed too lol

oh i am cancelling the push lol

Lol. Sorry boss. I invaded your PR

Sorry from my side. You had to do it :(


Lambda execution: Why use now-bridge with local http-server instead of just calling the render function?

First of all, I'm just here because tf-next build enables me to actually use my next-app in a manually configured environment using CloudFormation/SAM. Good job 👍 .
I want to understand why you made the decision to implement the launcher and bridge (the runtime of the lambda proxy) in a way that you create a local http server and call it directly from inside the code. It seems to me that this just creates a massive unnecessary overhead.
The only thing that is needed is the routing stuff from the generated now__launcher.js.
The render function can be called directly with mocked req/res objects:
const mod = require('./.next/serverless/pages/index.js')
mod.render(
    { headers: {}, url: 'http://<IP_ADDRESS>/' },
    {
        setHeader: v => { console.log('next:setHeader', v)},
        getHeader: v => { console.log('next:getHeader', v); return undefined; },
        hasHeader: v => { console.log('next:hasHeader', v); return false; },
        end: console.dir
    }
);

Hi,
the launcher & bridge was something we simply copied from the Vercel repo when their deployment process for Next.js was still open source.
Vercel has chosen this approach for their platform to achieve a maximum of compatibility with traditional Node.js server environments.
While there are existing mocking solutions for request & response around (e.g. node-mocks-http), the objects that Node.js creates for a request in practice are quite complex and difficult to mock.
I would guess that starting a Node.js server and making a request is just good enough in terms of performance instead of using a mocking solution.
Since the server is frozen by AWS Lambda between calls, the server is also only generated and started on a Lambda cold start.
So for every following request to the same Lambda the performance is equal to a running Node.js server.
The render function that is currently packed into the launcher is also something that is already deprecated in Next.js 12.
Future deployments to AWS Lambda will use a full Next.js server, instead of creating an abstraction for serverless environments.

Not exactly the Next.js custom server but the core of it (called next-server), yes.
I have an ongoing PR that implements it: #89
The main part of it is a new launcher that uses the next-server package: https://github.com/milliHQ/terraform-aws-next-js/blob/63d6547c85e5d8e381b4072c63607d22c11ae4fc/packages/runtime/src/server-launcher.ts
Yes, sls-next is currently in the process moving its Lambdas from Lambda@Edge to regional Lambdas to follow a similar architecture as ours (and Vercel's).
However it seems that they try to implement the routing logic into API-Gateway while we use Lambda@Edge for it.
It's kind of crazy that each community Next.js serverless project creates their own builder (this project, sls-next, @netlify, Amplify, Flightcontrol.dev) given that we all deploy to AWS Lambda eventually 😅

I have an ongoing PR that implements it: #89

👍 , any roadmap/eta?

It's kind of crazy that each community Next.js serverless project creates their own builder

You're absolutely right. Kind of a bummer that vercel doesn't provide the full build tools for us. Leaves a bittersweet taste given the fact that Next.js is promoted as "open-source". Nevertheless, I also understand that it's their business model to kind of force people to use their hosting platform.

It also seems that @dphang from the serverless-next.js repo is working on a simple compat layer for normal lambdas right now: https://github.com/serverless-nextjs/serverless-next.js/tree/master/packages/libs/lambda (last commit 30 mins ago)

Sorry, just chiming in since I was mentioned - definitely interesting projects coming along. Yes, that is right, the build/packaging logic is actually mostly done (thanks in large part to Jan Varho for the initial work) but I'm currently working on testing deployment logic using CDK for Terraform and ironing bugs out.

It's kind of crazy that each community Next.js serverless project creates their own builder (this project, sls-next, @netlify, Amplify, Flightcontrol.dev) given that we all deploy to AWS Lambda eventually 😅

Yeah, that is definitely a lot of duplication, though for serverless-next.js, I guess the original author made it very coupled/custom logic for Lambda@Edge to optimize for serverless platforms instead of trying to use parts of next-server. I took over and am trying to make the core routing logic more generic and able to support multiple platforms in the future (Lambda@Edge, Lambda, Azure, GCP etc.) while optimizing for serverless platforms (cold starts especially). Unfortunately it's also hard to keep up with Next.js parity since we are emulating the routing logic nearly from scratch (except parts like reusing image optimizer and some build tools) and it's not the cleanest code (and especially since I'm the only one working on it now haha. I've been working on cleaning things up and making it easier to update the core logic - just been a slow process).
In fact https://github.com/serverless-stack/serverless-stack does use our @sls-next/lambda-at-edge builder with their own CDK constructs...

You're absolutely right. Kind of a bummer that vercel doesn't provide the full build tools for us. Leaves a bittersweet taste given the fact that Next.js is promoted as "open-source". Nevertheless, I also understand that it's their business model to kind of force people to use their hosting platform.

Yeah, I definitely understand too since their business model is hosting on Vercel, but considering it's open source I believe we should also have bunch of self-hosted options (AWS, etc.), hence why I continued on maintaining serverless-next.js.

👍 , any roadmap/eta?

Plan is to ship it together with ISR mode in the 0.11 milestone by end of November.

Hey all, just chiming in since I was mentioned - definitely an interesting project here that I didn't know about yet.

Always welcome! Nice to hear your thoughts on this topic, thanks for sharing! 👍
It think Vercel has learned some lessons from the Gatsby story: They invested a lot of time (of their then small team) to improve the open source Markdown parsing / MDX projects and then Gatsby came in, put a GraphQL API on top of MDX and scored their first big VC money from it.
So their engagement in open source there ultimately created the first real competitor to Next.js, which gave them a hard time.
However since they pulled the deployment code from the public repo they always worked towards removing special Vercel code from Next.js (which resulted in the deprecation of the serverless target mode in v12).
However it's still hard to keep up-to-date with the latest Next.js features since most of the changes of internal Next.js behaviors are not public (or hidden in patch releases), so reverse engineering for new features takes a lot of time.

Unfortunately it's also hard to keep up with Next.js parity since we are emulating the routing/handling logic nearly from scratch (except parts like reusing image optimizer and some build tools)

That might be some points where working together would make sense:

For the image optimization we developed a small wrapper around Next.js that separates the image optimizer from the rest of Next.js.
There is already a request (https://github.com/milliHQ/terraform-aws-next-js-image-optimization/issues/58) to make the wrapper available as Express middleware, so we are currently thinking about publishing the wrapper as dedicated npm module with additional adapters (For Lambda, Express, etc).
Our router (called proxy) is also an unopinionated module that simply mimics the routing  of Vercel's edge network and could be reused in multiple projects.

I'm going to close this issue because it has been inactive for 30 days ⏳. This helps to find and focus on the active issues.


Helm chart not yet updated for 'annotation' typo fix

Bugs should be filed for issues encountered whilst operating cert-manager.
You should first attempt to resolve your issues through the community support
channels, e.g. Slack, in order to rule out individual configuration errors.
Please provide as much detail as possible.

Describe the bug:
Latest helm chart still includes the typo fixed in https://github.com/jetstack/cert-manager/commit/be0b865522e9d876b81155790996a2ea1e37a893
Expected behaviour:
Now that the typo is fixed, the helm chart would be updated
Steps to reproduce the bug:
Pull the latest helm chart, examine templates/psp.yaml. Note the use of annotation instead of annotations
/kind bug

We version our Helm chart alongside the actual application, so this will appear once we cut v0.11.1 (as this patch has already been back-ported) 😄


How to represent to-many relationship in objects in primary data

In JSON API I need to represent a collection of primary data where the primary data objects have to-many relationship to themselves.
ie. data will look like below
account1 -> canTransferTo (relationship) -account3
account2 -> canTransferTo (relationship) -account1, account3
account3 -> canTransferTo (relationship) -account1
how will I do that so that I follow the JSO API specifications. Can it be like this below in the compoind document?
{
"data": [{
    "type": "accounts",
    "id": "1",
    "attributes": {
        "name": "account1"
    },
    "relationships": {
        "canTransferTo": {
            "links": {
                "self": "http://example.com/accounts/1/relationships/canTransferTo",
                "related": "http://example.com/accounts/1/canTransferTo"
            },
            "data": {
                "type": "accounts",
                "id": "3"
            }
        }
    },
    "links": {
        "self": "http://example.com/accounts/1"
    }
}, {
    "type": "accounts",
    "id": "2",
    "attributes": {
        "name": "account2"
    },
    "relationships": {
        "canTransferTo": {
            "links": {
                "self": "http://example.com/accounts/2/relationships/canTransferTo",
                "related": "http://example.com/accounts/2/canTransferTo"
            },
            "data": [{
                "type": "accounts",
                "id": "1"
            }, {
                "type": "accounts",
                "id": "3"
            }]
        }
    },
    "links": {
        "self": "http://example.com/accounts/2"
    }
}, {
    "type": "accounts",
    "id": "3",
    "attributes": {
        "name": "account3"
    },
    "relationships": {
        "canTransferTo": {
            "links": {
                "self": "http://example.com/accounts/3/relationships/canTransferTo",
                "related": "http://example.com/accounts/3/canTransferTo"
            },
            "data": {
                "type": "accounts",
                "id": "1"
            }
        }
    },
    "links": {
        "self": "http://example.com/accounts/3"
    }
}]
}

Yes, that's how you would do it!
In the future, though, please ask questions like this on our discussion forum, discuss.jsonapi.org.
Thanks!


IOS using Google Map instead

Hi
Thank you for your effort in creating this library, amazing stuff!
What if i need to use google maps sdk instead of native apple on ios. It's just a requirement.
Is that possible to do through some sort of configuration?
Thank you

Duplicate #35


Installing react-reveal

PS C:\Users\astro\Desktop\invis> npm install react-reveal --save

npm ERR! code ERESOLVE
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR!
npm ERR! While resolving<EMAIL_ADDRESS>npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/react
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>|| ^16.0.0" from<EMAIL_ADDRESS>npm ERR! node_modules/react-reveal
npm ERR!   react-reveal@"*" from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force, or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.
npm ERR!
npm ERR! See C:\Users\astro\AppData\Local\npm-cache\eresolve-report.txt for a full report.
npm ERR!      C:\Users\astro\AppData\Local\npm-cache_logs\2023-01-25T13_44_53_012Z-debug-0.log
this is a new react app I made just to install this, as it wasnt working on my other project, does anyone know why this wont install ?

https://github.com/rnosov/react-reveal/issues/135#issuecomment-1229422979

Ah, thank you, this will definitely work.

npm install react-reveal --save --legacy-peer-deps
This will make it work. Easy pissy lemon squeezy

I had to remove react-reveal from package.json before running npm install, after that I reinstalled react-reveal using @hamzapaskingakhtar1999's solution.

Hey guys I have upgraded this package  and created a new packagethat supports react 18 . So can we move on to that package please?  Syntaxes are same andall features are same withnew name and upgraded to latest standards.  I will need contributors as I can't give it full time. Please check https://www.npmjs.com/package/easy-reveal


Valkyria Revolution [PCSE01003]

App summary

App name: Valkyria Revolution
App serial: PCSE01003
App version: 1.01

Vita3K summary

Version: v0.1.5
Build number: 2658
Commit hash: https://github.com/vita3k/vita3k/commit/48f3e7ce
CPU backend: Dynarmic

Test environment summary

Tested by: gymzatan
OS: Windows 11
CPU: AMD Ryzen 7 5800H
GPU: AMD Radeon(TM) Graphics
RAM: 16 GB

Issues
Game crashes right after booting.
Screenshots

Log
vita3k.log
Recommended labels

Bootable
crash

Would like to add an update to this.
App Summary

App Name: Valkyria Revolution
App Serial: PCSE01003
App version: 1.01

Vita3K Summary

Version: 0.1.6
Build: 2917
CPU backend: Dynarmic

Test environment

Tested by: ValliteKing
OS: Windows 11
CPU: Intel Core i5-10300H
GPU: Nvidia Geforce 1650GTX
RAM: 16GB

Issues
Game loads into gameplay but crashes on results screen after finishing the first mission. Has major graphical issues in certain cutscenes.
Screenshots

Log
vita3k.zip
(Had to compress as the log was over 25MB)
Recommended Labels

Ingame -
Crashes
Graphics bug

as of https://github.com/Vita3K/Vita3K/commit/c541d8eb45e445d8741e5344d2cb3f9017607212, the game got a bit further with 4 cutscenes in OpenGL but crashed during loading. On Vulkan, it crashed before the tank mini-boss with a out-of-memory error. There's still graphical bugs in menu and ingame.
Screenshots

Log
vita3k.log
Recommended Labels

Ingame -
graphical bug
vulkan crash


Add an API to return Volume Types for StoragePool

Add an API to return Volume Types for StoragePool

Coverage increased (+0.03%) to 43.889% when pulling 490e69079513819d027c3227aec64c4c4c63cd7b on lijuncloud:flow03 into 0b5f91dccc57c015360a815db3a6bb6ed822609c on opensds:development.

Coverage increased (+0.03%) to 43.889% when pulling 490e69079513819d027c3227aec64c4c4c63cd7b on lijuncloud:flow03 into 0b5f91dccc57c015360a815db3a6bb6ed822609c on opensds:development.


Can't add or remove events programmatically on v4

When trying to add new event or remove a current event from calendar using array's push or splice method Im getting the following error:
TypeError: this.calendar.removeEventSources is not a function
    at FullCalendarComponent.push../node_modules/ngx-fullcalendar/fesm5/ngx-fullcalendar.js.FullCalendarComponent.ngDoCheck (ngx-fullcalendar.js:230)
    at checkAndUpdateDirectiveInline (core.js:22004)
    at checkAndUpdateNodeInline (core.js:23265)
    at checkAndUpdateNode (core.js:23227)
    at debugCheckAndUpdateNode (core.js:23861)
    at debugCheckDirectivesFn (core.js:23821)
    at Object.eval [as updateDirectives] (SchedulersComponent.html:2)
    at Object.debugUpdateDirectives [as updateDirectives] (core.js:23813)
    at checkAndUpdateView (core.js:23209)
    at callViewAction (core.js:23450)

Steps to reproduce the behavior:
 this.events.push({
                id: 'e',
                title: 'Friends coming round 2',
                start: '2018-07-29T18:00:00',
                end: '2018-07-29T23:00:00'
            }
       );

That's because that method has been removed in v4 of Fullcalendar. If you look at this page: https://fullcalendar.io/docs/v4/release-notes it clearly states that:

removeEventSources | Removed. Instead, use multiple EventSource::remove calls. Group together for performance with batchRendering.

So you should fetch all events that the calendar currently has in memory, and then call remove on each of them. Once that's done, you can add new events using the addEvent method on fullcalendar.

Try something like this:
`@ViewChild('theCalendar') theCalendar: FullCalendarComponent;
// Grab all events
let allevents = this.theCalendar.calendar.getEvents();
// Loop all events
allevents.forEach(el => {

  // And remove them
  el.remove();

});

// Loop that array
events.forEach(el => {
  // And add them to the calendar again
  this.theCalendar.calendar.addEvent(el);

});`

Fixed in v5.0.0-alpha.1

@martinsiden please check https://github.com/StickNitro/ngx-fullcalendar/issues/21


Discrepency in AEP UI access for non AEP licensed customers

Issue in ./help/edge/web-sdk-faq.md
The documentation state
**Do customers need to buy Adobe Experience Platform to use the Web SDK?
No. Any Adobe Digital Experience customer can use it. Totally free. Any customer wanting to use the Web SDK will be given access to create schemas and datasets in the Adobe Experience Platform UI.**
Given that customers without the AEP license can use Web SDK and leverage ID Sync for "Visitor Attributes", they should also have access to the UI to create Identity namespace.
The document should explicitly call this out.
https://experienceleague.adobe.com/docs/experience-platform/edge/web-sdk-faq.html?lang=en#do-customers-need-to-buy-adobe-experience-platform-to-use-the-web-sdk%3F

Thank you for submitting this issue. We will review the documentation and make the necessary updates as quickly as possible. Tracking internally in PLAT-117510.

The documentation was updated with the suggested information.


add IAM SaaS config

/retest

So IAM will check if SaaS environment on its own now?

We can add IAM SaaS configuration into the OperandConfig template, so we can remove the logic for updating OperandConfig. I will update this PR after #463 is merged.


Adding Institute of Medical Biology of Polish Academy of Sciences

The email addresses are<EMAIL_ADDRESS>(e.g. my email address: rbachorz@cbm.pan.pl), the website is: www.ibmpan.pl
The Institute of Medical Biology is a part of National Polish Academy of Sciences.

@rafalbachorz This institute is primarily a research organization. I believe that there are some students who take their PhD in the institute, however we do not add such institutions to this repositrory in general. Please email<EMAIL_ADDRESS>to check if you are eligible for another forms of support. This request remains open until the final decision is made. Thank you for the understanding.

@philipto - thank you for feedback. As you suggest I have contacted the community-support.


Snippet expansion using TAB doesn't work in R markdown

For example, in a R code section, type sql + TAB, TAB and it doesn't expand like in R editor.  Insert code snippet command works ok though.

https://github.com/Microsoft/RTVS/commit/7f6b400551d23ea0a9c392ac34dfa0492c74708d


Chapter 14.3 Django version

I believe when using pip to get the django on PythonAnywhere,
we would need to do:
pip install -U django==1.9
not
pip install -U django==1.7

Thanks! We are working on updating these chapters :-)
On Wed, Jul 13, 2016 at 12:45 PM, koviusesGitHub<EMAIL_ADDRESS>wrote:

I believe when using pip to get the django on PythonAnywhere,
we would need to do:
pip install -U django==1.9
not
pip install -U django==1.7
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/leifos/tango_with_django_19/issues/26, or mute the
thread
https://github.com/notifications/unsubscribe/AB02B1Fkt5oJyaxAh9oV-yQUChXxRxYKks5qVM_QgaJpZM4JLVRz
.

Thanks! All sorted now, I'll close this.


feat(dapp-console-api): createApp endpoint

#249  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @tremarkley and the rest of your teammates on  Graphite


Load called twice on page reload?

Describe the bug
Hello,
We have a root route (A) with a children root route (B) which contains by itself other children. Not sure if that matters however as this bug occurs during reloading the page I can not provide a stackblitz sample so trying to explain the circumstances as best as I can.
When we use a simple loader on the child root route (B) and simply log out then on initial page load when hitting the route it calls that loader twice. However on regular navigation using Links it correctly is called only once. We also checked if the abortController is maybe called on second load (like with the useEffect issue in dev mode) but it is not.
Your Example Website or App
https://stackblitz.com/github/tanstack/router/tree/beta/examples/react/quickstart?file=src%2Fmain.tsx
Steps to Reproduce the Bug or Issue
See description
Expected behavior
Loader when entering a route is only called once not twice
Screenshots or Videos
No response
Platform

OS:macOS
Browser: Chrome
Version: 120
Tanstack Router: 0.0.1-beta.279

Additional context
No response

Is it possible that its just due to <StrictMode />? If you remove that temporarily, does it only get called once?

Yes indeed that solves it. I already feared its due useEffect but isn't that something the router should handle correctly also in strict / dev mode like ie tanstack query does?

Ya that was a lovely thing React Query did to smooth over that. I'd actually bet it's more of a result of the recent switch to be React dominant in its implementation and it was possibly overlooked. Good call out.

Bug. I’ll get it fixed

You're the best @tannerlinsley!

I can't seem to replicate. Will you please add a link to a sandbox set up with the bug?

@tannerlinsley Not sure why I can't share my fork on stackblitz. However I simply modified the sample code and post it here so you could just make copy+paste on main.tsx. The relevant change is the loader on the index route, on initial load it'll show a load counter of 2 so it got called twice. On subsequent navigation it works as expected its only on initial load:
import React, { StrictMode } from 'react'
import ReactDOM from 'react-dom/client'
import {
  Outlet,
  RouterProvider,
  Link,
  Router,
  Route,
  RootRoute,
} from '@tanstack/react-router'
import { TanStackRouterDevtools } from '@tanstack/router-devtools'

const rootRoute = new RootRoute({
  component: () => (
    <>

        <Link to="/" className="[&.active]:font-bold">
          Home
        </Link>{' '}
        <Link to="/about" className="[&.active]:font-bold">
          About
        </Link>

      <Outlet />
      <TanStackRouterDevtools />
    </>
  ),
})

let loaderCounter = 0

const indexRoute = new Route({
  getParentRoute: () => rootRoute,
  path: '/',
  loader: async () => {
    loaderCounter+= 1
    console.log('Index loader')
  },
  component: function Index() {
    return (

Welcome Home! #{loaderCounter}

    )
  },
})

const aboutRoute = new Route({
  getParentRoute: () => rootRoute,
  path: '/about',
  component: function About() {
    return
Hello from About!

  },
})

const routeTree = rootRoute.addChildren([indexRoute, aboutRoute])

const router = new Router({ routeTree, defaultPreload: 'intent' })

declare module '@tanstack/react-router' {
  interface Register {
    router: typeof router
  }
}

const rootElement = document.getElementById('app')!
if (!rootElement.innerHTML) {
  const root = ReactDOM.createRoot(rootElement)
  root.render(
    <StrictMode>
      <RouterProvider router={router} />
    </StrictMode>,
  )
}

Thanks!
On Dec 24, 2023 at 4:44 AM -0700, Alexander Adam @.***>, wrote:

@tannerlinsley Not sure why I can't share my fork on stackblitz. However I simply modified the sample code and post it here so you could just make copy+paste on main.tsx. The relevant change is the loader on the index route, on initial load it'll show a load counter of 2 so it got called twice. On subsequent navigation it works as expected its only on initial load:
import React, { StrictMode } from 'react'
import ReactDOM from 'react-dom/client'
import {
Outlet,
RouterProvider,
Link,
Router,
Route,
RootRoute,
} from @./react-router'
import { TanStackRouterDevtools } from @./router-devtools'
const rootRoute = new RootRoute({
component: () => (
<>

Home
{' '}

About

</>
),
})
let loaderCounter = 0
const indexRoute = new Route({
getParentRoute: () => rootRoute,
path: '/',
loader: async () => {
loaderCounter+= 1
console.log('Index loader')
},
component: function Index() {
return (

Welcome Home! #{loaderCounter}

)
},
})
const aboutRoute = new Route({
getParentRoute: () => rootRoute,
path: '/about',
component: function About() {
return Hello from About!
},
})
const routeTree = rootRoute.addChildren([indexRoute, aboutRoute])
const router = new Router({ routeTree, defaultPreload: 'intent' })
declare module @.***/react-router' {
interface Register {
router: typeof router
}
}
const rootElement = document.getElementById('app')!
if (!rootElement.innerHTML) {
const root = ReactDOM.createRoot(rootElement)
root.render(

,
)
}
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>


there is no moving event for map in ol5

in my case (like this d3 Integration ), i use Image for my layer and ImageCanvas for source, i use canvasFunction to draw my own data, while the zoom or center is changed, canvasFunction is used to recalculate data position in canvas. But during the moving, the data will get scale with the map, I expect is there something like moving event, i can use to render my ImageCanvas so that i won't be larger or smaller.......

http://openlayers.org/en/latest/apidoc/module-ol_MapEvent-MapEvent.html#event:moveend

@raiyni is there any showcase?

https://openlayers.org/en/latest/examples/moveend.html?q=moveend

@bartvde this case is about moveend, while i want to call canvasFunction when moving.

@mysisi The events for that would be change:center and change:resolution.

@ahocevar thx, i see ol canceled all map fires related to change

Can we close this @mysisi?

@ahocevar i still wonder when will you add change:resolution or change:center events to ol?


URL malformed on RHEL

Hi!
I am trying to install the BMW Training GUI on a RHEL machine --> AC922, IBM Power architecture (ppc64le).
I reach the landing page with http://<IP_ADDRESS>:4200/training, but then I get a notification:
'The URI is malformed' in Firefox.

Failed to execute 'open' on 'XMLHttpRequest': Invalid URL in chromium:

The requesting machine is in the same network and has the operating system: Ubuntu 18.04.
I installed the setup also on the requesting machine Ubuntu 18.04. and it worked (i don´t think it´s network related). I´ve been searching in the code for anything which could cause the error, but I cannot find it.
Does anyone have an idea what causes the problem or even a solution on how the problem can be solved?
I appreciate any hints. Thanks.

It seems like an issue with the URL format. It would be helpful if you could open the network tab of the browser and try to refresh the page so you could see the URL that app is trying to reach.
Another helpful thing that you could do is to provide screenshots of your environment.ts and environment.prod.ts files.

It seems like an issue with the URL format. It would be helpful if you could open the network tab of the browser and try to refresh the page so you could see the URL that app is trying to reach.
Another helpful thing that you could do is to provide screenshots of your environment.ts and environment.prod.ts files.

Hi Joe,
sorry for the late reply.
I took a screenshot of the environment.ts (the environment.prod.ts looks the same)

I also took a screenshot of the network tab:

I installed the setup on an Ubuntu machine and if I look at the network tab, it looks like this:

Actually, I don´t see which URL is requested in the network tab or at least nothing which is conspicuous.

Hi Joe,
sorry for the late reply.
I took a screenshot of the environment.ts (the environment.prod.ts looks the same)

I also took a screenshot of the network tab:

I installed the setup on an Ubuntu machine and if I look at the network tab, it looks like this:

Actually, I don´t see which URL is requested in the network tab or at least nothing which is conspicuous.

Hello,
when I enter http://<IP_ADDRESS>:4200/training, i get to the start page

I can also ping the machine (it is in the same network). So, it´s reachable.
Unfortunately, it is not requesting the get_finished_jobs as i see in the network tab:

But when I enter it manually (http://http://<IP_ADDRESS>:2222/get_finished_jobs), I get an empty array back with the http message 200 OK.
I guess there might be something wrong with the routing or building the correct URL. What do you think?

Hello,
when I enter http://<IP_ADDRESS>:4200/training, i get to the start page

I can also ping the machine (it is in the same network). So, it´s reachable.
Unfortunately, it is not requesting the get_finished_jobs as i see in the network tab:

But when I enter it manually (http://http://<IP_ADDRESS>:2222/get_finished_jobs), I get an empty array back with the http message 200 OK.
I guess there might be something wrong with the routing or building the correct URL. What do you think?

any updates on this?

It might be a browser related issue. Could you try to use regular chrome instead of chromium and trying again?

Yes, I tried. It is the same issue coming up.


x/website: contribution guide: make it clear a Gerrit account is required even when sending a GitHub PR

Background
It's possible to read the current "Sending a change via GitHub" section of the Contribution Guide as saying a Gerrit account is optional when sending a GitHub PR. The closest it comes to explicitly telling someone to use Gerrit might be:

Remember you can always visit Gerrit to see the fine-grained review.

Perhaps avoiding the requirement of a Gerrit account was an aspirational goal when that section was written, or perhaps that is an unintended reading.
Outside of the "Sending a change via GitHub" section, the contribution guide does say a Gerrit account is required, but that seems to be intermixed with various other steps that are not required for a GitHub-based workflow, and hence can be read as not applying to the GitHub-based workflow.
I think experience suggests that a Gerrit account is almost always needed, including to write a reply to reviewers, mark feedback as done, and so on. Some of the friction listed by Bryan in https://github.com/golang/go/discussions/61182 seems to be the result of GitHub users not registering for Gerrit.
Suggestion
We should make it clear that a Gerrit account is considered a requirement even if sending a PR via GitHub.
The potential contributor must use a Gmail or other Google account to sign the CLA, so registering for Gerrit is hopefully a very small additional step at that point. A Gerrit account of course also serves as an on-ramp to more use of Gerrit, including possibly helping review other CLs or using the Gerrit-based contribution workflow.
To help make this suggestion more concrete, I will send a draft CL shortly.

Rough draft for discussion purposes in https://go.dev/cl/509019.
CC @heschi, @cagedmantis


Add support for bitvector sort and operations

See flux-tests/tests/pos/surface/bitvec00.rs for example.

It looks good in general but I haven't had time to sit down for a detailed review. Will probably look at it tomorrow.

👍
On Tue, Mar 28, 2023 at 6:47 PM Nico Lehmann @.***>
wrote:

It looks good in general but I haven't had time to sit down for a detailed
review. Will probably look at it tomorrow.
—
Reply to this email directly, view it on GitHub
https://urldefense.com/v3/__https://github.com/flux-rs/flux/pull/405*issuecomment-1487831981__;Iw!!Mih3wA!FYGd1Ddr4-BvOcA9EhvrHfurkDnkkh_zTTn3Y8EST0C_SWAszbN_Tqgy3ATYfq87k-GVMyBiqqyHstnLkEXLq9u6$,
or unsubscribe
https://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/AAMS4OE3MSKYGASLDSLSNHTW6OICPANCNFSM6AAAAAAWHTRIVI__;!!Mih3wA!FYGd1Ddr4-BvOcA9EhvrHfurkDnkkh_zTTn3Y8EST0C_SWAszbN_Tqgy3ATYfq87k-GVMyBiqqyHstnLkHr_NITW$
.
You are receiving this because you authored the thread.Message ID:
@.***>

--

Ranjit.

The reason I elected to reuse the existing function/symbol machinery is to
not have to duplicate the name resolution / sort checking logic.
Does your suggestion avoid that?
On Wed, Mar 29, 2023 at 10:22 AM Nico Lehmann @.***>
wrote:

@.**** requested changes on this pull request.
The way function symbols are disambiguated between the different kinds
feels wrong to me. Left a suggestion on how I think it should be
implemented.
In flux-tests/tests/lib/rbitvec.rs
https://urldefense.com/v3/__https://github.com/flux-rs/flux/pull/405*discussion_r1152217586__;Iw!!Mih3wA!B--0loNKoJb7rfscmWIAAAYZLWlnGC39uCr4HRXDCKIoVvoHjH1PA5mxenqaZ5PdAtbmo86Nq4bHnvqaCUzYc3qb$
:

+#[flux::refined_by(value: bitvec<32>)]
+pub struct UsizeBv(usize);

This is wrong, the width of usize depends on the architecture and it's
most defenitely not 32 these days. Can we change this to u32?
Also this should be marked as #[flux::opaque]. The defintion should be
rejected without the opaque annotation because the index is not being used,
but it hasn't being implemented (tracked here #212
https://urldefense.com/v3/__https://github.com/flux-rs/flux/issues/212__;!!Mih3wA!B--0loNKoJb7rfscmWIAAAYZLWlnGC39uCr4HRXDCKIoVvoHjH1PA5mxenqaZ5PdAtbmo86Nq4bHnvqaCZsCKBia$
)
In flux-tests/tests/pos/surface/bitvec00.rs
https://urldefense.com/v3/__https://github.com/flux-rs/flux/pull/405*discussion_r1152219863__;Iw!!Mih3wA!B--0loNKoJb7rfscmWIAAAYZLWlnGC39uCr4HRXDCKIoVvoHjH1PA5mxenqaZ5PdAtbmo86Nq4bHnvqaCUiVfsSd$
:

+#[flux::refined_by(value: bitvec<32>)]
+struct UsizeBv(usize);

Same here, this should be a u32 and be marked as opaque
In flux-middle/src/fhir.rs
https://urldefense.com/v3/__https://github.com/flux-rs/flux/pull/405*discussion_r1152222394__;Iw!!Mih3wA!B--0loNKoJb7rfscmWIAAAYZLWlnGC39uCr4HRXDCKIoVvoHjH1PA5mxenqaZ5PdAtbmo86Nq4bHnvqaCfQfINwe$
:

@@ -739,17 +748,38 @@ impl Map {
self.consts.get(name.borrow())
}

// Theory Symbols
fn insert_interp_func(&mut self, name: Symbol, inputs: Vec, output: Sort) {

   let sort = FuncSort::new(inputs, output);

   self.uifs.insert(name, FuncDef { name, sort, interp: true });

}

pub fn insert_interpreted_funcs(&mut self) {

⬇ Suggested change

pub fn insert_interpreted_funcs(&mut self) {

fn insert_interpreted_funcs(&mut self) {

In flux-middle/src/fhir.rs
https://urldefense.com/v3/__https://github.com/flux-rs/flux/pull/405*discussion_r1152226372__;Iw!!Mih3wA!B--0loNKoJb7rfscmWIAAAYZLWlnGC39uCr4HRXDCKIoVvoHjH1PA5mxenqaZ5PdAtbmo86Nq4bHnvqaCc_v8i5W$
:

+pub struct FuncDef {
pub name: Symbol,
pub sort: FuncSort,

// Is this an interpreted theory symbol
pub interp: bool,

A better name for this could be FuncDecl. We currently use this struct
for interpreted theory functions, uninterpreted functions and functions
with a body, so maybe we should also have a FuncKind enum instead of a
bool.
In flux-refineck/src/fixpoint_encoding.rs
https://urldefense.com/v3/__https://github.com/flux-rs/flux/pull/405*discussion_r1152259716__;Iw!!Mih3wA!B--0loNKoJb7rfscmWIAAAYZLWlnGC39uCr4HRXDCKIoVvoHjH1PA5mxenqaZ5PdAtbmo86Nq4bHnvqaCX1k0SN-$
:

@@ -92,7 +92,7 @@ struct ExprCtxt<'a> {

#[derive(Debug)]
struct ConstInfo {

name: fixpoint::ConstName,

name: fixpoint::Func,

This feels really wrong, we should only be tracking constants here. I'd do
the following

Make sure that in fhir symbols used as functions are fully
disambiguated between the three kinds of functions (interpreted,
uninterpreted and with body). We can either have one case per kind in
fhir::Func or attach a FuncKind to disambiguate (related to the
FuncDecl vs FuncDef comment).
Carry the disambiguation info into rty.
Since we have all cases disambiguated, we should only lookup for
the correponding constant in the uninterpreted case. For the interpreted
case we just carry the symbol into fixpoint. Finally, for the function with
body case we just panic because they must have been normalized away already.

—
Reply to this email directly, view it on GitHub
https://urldefense.com/v3/__https://github.com/flux-rs/flux/pull/405*pullrequestreview-1363539865__;Iw!!Mih3wA!B--0loNKoJb7rfscmWIAAAYZLWlnGC39uCr4HRXDCKIoVvoHjH1PA5mxenqaZ5PdAtbmo86Nq4bHnvqaCdJ1gDdN$,
or unsubscribe
https://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/AAMS4ODEYR5WX3NAL7NFZUDW6RVVXANCNFSM6AAAAAAWHTRIVI__;!!Mih3wA!B--0loNKoJb7rfscmWIAAAYZLWlnGC39uCr4HRXDCKIoVvoHjH1PA5mxenqaZ5PdAtbmo86Nq4bHnvqaCTsDqMwG$
.
You are receiving this because you authored the thread.Message ID:
@.***>

--

Ranjit.

The reason I elected to reuse the existing function/symbol machinery is to not have to duplicate the name resolution / sort checking logic. Does your suggestion avoid that?

I think it does, if we add a FuncKind field to fhir::Func::Uif then everything else remains the same.

👍
On Wed, Mar 29, 2023 at 10:36 AM Nico Lehmann @.***>
wrote:

The reason I elected to reuse the existing function/symbol machinery is to
not have to duplicate the name resolution / sort checking logic. Does your
suggestion avoid that?
I think it does, if we add a FuncKind field to fhir::Func::Uif then
everything else remains the same.
—
Reply to this email directly, view it on GitHub
https://urldefense.com/v3/__https://github.com/flux-rs/flux/pull/405*issuecomment-1489026939__;Iw!!Mih3wA!GMpnAbVZXcJokYskyIwAcPLs_qIQd96HyCGlDBgeLG7d9ukotRk8bEVHGsf_rVNGGCzAh706thQzZRhzSrCZHGOg$,
or unsubscribe
https://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/AAMS4OD63QUFE65S6GDUKXDW6RXJBANCNFSM6AAAAAAWHTRIVI__;!!Mih3wA!GMpnAbVZXcJokYskyIwAcPLs_qIQd96HyCGlDBgeLG7d9ukotRk8bEVHGsf_rVNGGCzAh706thQzZRhzShqs1ict$
.
You are receiving this because you authored the thread.Message ID:
@.***>

--

Ranjit.

Have made the relevant edits (as I understood them!) Was trying to avoid creating a thy_map ; i guess its not really doing much -- we could have just translated Symbols not in the const_map directly into fixpoint -- but I thought it may be useful to carry it around. LMK if we should delete it.

There's something odd going on with pos/surface/bitvec01.rs, the CI seems stuck on that test. I tried it locally and tests are passing.

Maybe the z3 version? Seems the CI is using 4.10.2, I have 4.11.2 on my laptop.

Ugh I was wondering what's going on. I have z3 4.12 ...
On Wed, Mar 29, 2023 at 5:14 PM Nico Lehmann @.***>
wrote:

Maybe the z3 version? Seems the CI is using 4.10.2
https://urldefense.com/v3/__https://github.com/pavpanchekha/setup-z3/blob/master/action.yml*L16__;Iw!!Mih3wA!HIjnQMw92pLCxUCmIE59Ev8dU7OCFseK3d0Ge7ibuFU9UUaMcEPaCCoiJZUU9sLoXAp-Ka197STMNAYYAbe0W0gw$,
I have 4.11.2 on my laptop.
—
Reply to this email directly, view it on GitHub
https://urldefense.com/v3/__https://github.com/flux-rs/flux/pull/405*issuecomment-1489509537__;Iw!!Mih3wA!HIjnQMw92pLCxUCmIE59Ev8dU7OCFseK3d0Ge7ibuFU9UUaMcEPaCCoiJZUU9sLoXAp-Ka197STMNAYYAd7Sog7C$,
or unsubscribe
https://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/AAMS4ODXOMZ6VFM5TCDXPC3W6TF6NANCNFSM6AAAAAAWHTRIVI__;!!Mih3wA!HIjnQMw92pLCxUCmIE59Ev8dU7OCFseK3d0Ge7ibuFU9UUaMcEPaCCoiJZUU9sLoXAp-Ka197STMNAYYAVTrza2G$
.
You are receiving this because you authored the thread.Message ID:
@.***>

--

Ranjit.

Ok, apparently it was z3. 🚢 it


DocsArchive-<IP_ADDRESS>Auto archive content

Auto archive content from https://github.com/OPS-E2E-PPE/docs-archive-test

Docs Build status updates of commit f2dfe76:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error]

Cannot sync git repo to specified commit because branch Archive_master_2019-09-25-11-47-34 has been deleted or has been force pushed: fatal: Couldn't find remote ref Archive_master_2019-09-25-11-47-34

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.