Sentry responded with an error: 'utf8' codec can't decode byte 0x9c

Expected Behavior
Bot starts as usual
Actual Behavior
After pulling the latest changes, the bot crashed on start and output some errors.
Your FULL config.json (remove your username, password, gmapkey and any other private info)
{
            "auth_service": "ptc",
            "username": "",
            "password": "",
            "location": "",
            "gmapkey": "",
            "libencrypt_location": "",
            "tasks": [{
                "type": "HandleSoftBan"
            }, {
                "type": "SleepSchedule",
                "config": {
                    "enabled": false,
                    "time": "22:54",
                    "duration": "7:46",
                    "time_random_offset": "00:24",
                    "duration_random_offset": "00:43"
                }
            }, {
                "type": "CollectLevelUpReward"
            }, {
                "type": "IncubateEggs",
                "config": {
                    "longer_eggs_first": true
                }
            }, {
                "type": "UpdateLiveStats",
                "config": {
                    "enabled": false,
                    "min_interval": 10,
                    "stats": ["uptime", "stardust_earned", "xp_earned", "xp_per_hour", "stops_visited"],
                    "terminal_log": true,
                    "terminal_title": true
                }
            }, {
                "type": "TransferPokemon"
            }, {
                "type": "NicknamePokemon",
                "config": {
                    "enabled": true,
                    "nickname_template": "{name}_{ivcp_pct2}"
                }
            }, {
                "type": "EvolvePokemon",
                "config": {
                    "evolve_all": "Pidgey,Weedle,Caterpie,Rattata,Zubat,Kakuna,Metapod",
                    "first_evolve_by": "cp",
                    "evolve_above_cp": 500,
                    "evolve_above_iv": 0.8,
                    "logic": "or",
                    "evolve_speed": 20,
                    "use_lucky_egg": false
                }
            }, {
                "type": "RecycleItems",
                "config": {
                    "min_empty_space": 15,
                    "item_filter": {
                        "Pokeball": {
                            "keep": 60
                        },
                        "Greatball": {
                            "keep": 50
                        },
                        "Ultraball": {
                            "keep": 30
                        },
                        "Potion": {
                            "keep": 70
                        },
                        "Super Potion": {
                            "keep": 20
                        },
                        "Hyper Potion": {
                            "keep": 30
                        },
                        "Revive": {
                            "keep": 30
                        },
                        "Razz Berry": {
                            "keep": 50
                        }
                    }
                }
            }, {
                "type": "CatchVisiblePokemon"
            }, {
                "type": "CatchLuredPokemon"
            }, {
                "type": "SpinFort"
            }, {
                "type": "MoveToFort",
                "config": {
                    "lure_attraction": true,
                    "lure_max_distance": 2000
                }
            }, {
                "type": "FollowSpiral",
                "config": {
                    "diameter": 4,
                    "step_size": 70
                }
            }],
            "map_object_cache_time": 5,
            "forts": {
                "avoid_circles": true,
                "max_circle_size": 50
            },
            "websocket_server": false,
            "walk": 4.16,
            "action_wait_min": 1,
            "action_wait_max": 4,
            "debug": false,
            "test": false,
            "health_record": true,
            "location_cache": true,
            "distance_unit": "km",
            "reconnecting_timeout": 15,
            "catch_randomize_reticle_factor": 1.0,
            "catch_randomize_spin_factor": 1.0,
            "logging_color": true,
            "catch": {
                "any": {
                    "catch_above_cp": 0,
                    "catch_above_iv": 0,
                    "logic": "or"
                },
                "// Example of always catching Rattata:": {},
                "// Rattata": {
                    "always_catch": true
                }
            },
            "release": {
                "any": {
                    "release_below_cp": 0,
                    "release_below_iv": 0,
                    "logic": "or"
                },
                "any": {
                    "keep_best_cp": 6,
                    "keep_best_iv": 1
                },
                "// Example of always releasing Rattata:": {},
                "//Rattata": {
                    "keep_best_cp": 1,
                    "keep_best_iv": 1
                },
                "// Example of keeping 2 stronger (based on IV) Zubat:": {},
                "//Zubat": {
                    "keep_best_cp": 1,
                    "keep_best_iv": 1
                },
                "//Golbat": {
                    "keep_best_cp": 2,
                    "keep_best_iv": 2
                },
                "//Metapod": {
                    "keep_best_cp": 1,
                    "keep_best_iv": 1
                },
                "//Kakuna": {
                    "keep_best_cp": 1,
                    "keep_best_iv": 1
                },
                "// Example of keeping 3 stronger (based on CP) Pidgey:": {},
                "// Pidgey": {
                    "keep_best_cp": 3
                },
                "// Also, it is working with any": {},
                "// any": {
                    "keep_best_iv": 3
                },
                "// Example of keeping the 2 strongest (based on CP) and 3 best (based on IV) Zubat:": {},
                "// Zubat": {
                    "keep_best_cp": 2,
                    "keep_best_iv": 3
                }
            },
            "vips": {
                "Any pokemon put here directly force to use Berry & Best Ball to capture, to secure the capture rate!": {},
                "any": {
                    "catch_above_cp": 1200,
                    "catch_above_iv": 0.9,
                    "logic": "or"
                },
                "Lapras": {},
                "Moltres": {},
                "Zapdos": {},
                "Articuno": {},

                "// S-Tier pokemons (if pokemon can be evolved into tier, list the representative)": {},
                "Mewtwo": {},
                "Dragonite": {},
                "Snorlax": {},
                "// Mew evolves to Mewtwo": {},
                "Mew": {},
                "Arcanine": {},
                "Vaporeon": {},
                "Gyarados": {},
                "Exeggutor": {},
                "Muk": {},
                "Weezing": {},
                "Flareon": {}

            }
        }

Output when issue occurred
2016-09-01 12:11:33,272 [sentry.errors] [ERROR] Sentry responded with an error: 'utf8' codec can't decode byte 0x9c in position 1: invalid start byte (url: https://app.getsentry.com/api/90254/store/)
Traceback (most recent call last):
File "/usr/local/lib/python2.7/site-packages/raven/transport/threaded.py", line 174, in send_sync
super(ThreadedHTTPTransport, self).send(data, headers)
File "/usr/local/lib/python2.7/site-packages/raven/transport/http.py", line 47, in send
ca_certs=self.ca_certs,
File "/usr/local/lib/python2.7/site-packages/raven/utils/http.py", line 66, in urlopen
return opener.open(url, data, timeout)
File "/usr/local/lib/python2.7/site-packages/future/backports/urllib/request.py", line 494, in open
response = self._open(req, data)
File "/usr/local/lib/python2.7/site-packages/future/backports/urllib/request.py", line 512, in _open
'_open', req)
File "/usr/local/lib/python2.7/site-packages/future/backports/urllib/request.py", line 466, in _call_chain
result = func(*args)
File "/usr/local/lib/python2.7/site-packages/raven/utils/http.py", line 46, in https_open
return self.do_open(ValidHTTPSConnection, req)
File "/usr/local/lib/python2.7/site-packages/future/backports/urllib/request.py", line 1284, in do_open
h.request(req.get_method(), req.selector, req.data, headers)
File "/usr/local/Cellar/python/2.7.11/Frameworks/Python.framework/Versions/2.7/lib/python2.7/httplib.py", line 1057, in request
self._send_request(method, url, body, headers)
File "/usr/local/Cellar/python/2.7.11/Frameworks/Python.framework/Versions/2.7/lib/python2.7/httplib.py", line 1097, in _send_request
self.endheaders(body)
File "/usr/local/Cellar/python/2.7.11/Frameworks/Python.framework/Versions/2.7/lib/python2.7/httplib.py", line 1053, in endheaders
self._send_output(message_body)
File "/usr/local/Cellar/python/2.7.11/Frameworks/Python.framework/Versions/2.7/lib/python2.7/httplib.py", line 895, in _send_output
msg += message_body
File "/usr/local/Cellar/python/2.7.11/Frameworks/Python.framework/Versions/2.7/lib/python2.7/encodings/utf_8.py", line 16, in decode
return codecs.utf_8_decode(input, errors, True)
UnicodeDecodeError: 'utf8' codec can't decode byte 0x9c in position 1: invalid start byte
2016-09-01 12:11:33,631 [sentry.errors.uncaught] [ERROR] [u'TypeError: call() takes exactly 1 argument (2 given)', u'  File "pokecli.py", line 781, in ', u'  File "pokecli.py", line 217, in main', u'  File "pokecli.py", line 265, in report_summary', u'  File "pokemongo_bot/metrics.py", line 108, in capture_stats', u'  File "pokemongo_bot/api_wrapper.py", line 239, in call', u'  File "pokemongo_bot/api_wrapper.py", line 188, in _call']
Steps to Reproduce
switch to master branch
do a git pull on master branch
run pip install -r requirements.txt
run ./run.sh
Other Information
OS:
OSX Yosemite
Branch:
master
Python Version:
2.7.11

Short Description

Possible solution

How it would help others

Which program did you use to edit the config file?

I'm using Sublime

According to this thread, your IP address may be banned : https://github.com/PokemonGoF/PokemonGo-Bot/issues/4908
Do you have the possibility to test with another IP@ (through VPN, proxy or by tethering your mobile phone) ?

Hmm don't think so, I've changed my IP and still got the error

@gusterwoei try to pip install -r requirements.txt --upgrade

Have you tried loggin in on the app with this account? It feels like a perma ban tbh.

Try to separate single config.json file  into  auth.json and config.json

@sohje  yup tried that too, doesn't work :(
@Darkayer I can login the app. In fact I can login via PokemonMap also, only the bot got this issue

pip freeze and show output

Solved it.
The problem is when I run pip install -r requirements.txt --upgrade
It's still using pgoapi-1.1.6
So later I tried to reset everything using ./setup.sh -r
Now it's using pgoapi-1.1.8, and the problem gone.

@mjmadsen Seems we have to check pgoapi version on bot start. That's causing lot of issues.


Trading Fee

Hi again,
Is a market fee actually taken (from markets.yml) I cannot see it logged anywhere or is this to be implemented still?
I understand it is taken in a percentage, but I cannot see in the logs or database where it takes the fee?

Thank you for ask, we'll fix it in next major release, it would be not to let you wait too lang.

Thanks for your reply, my next question is;
Are the fees actually taken at the moment? I know they aren't shown but are they still taken if I set it in markets.yml?

Read documentation and post
All this questions were asked million times.
Getting ridicules.
From: CryptNZ<EMAIL_ADDRESS>Sent: Saturday, February 07, 2015 10:46 PM
To: peatio/peatio
Subject: Re: [peatio] Trading Fee (#420)
Thanks for your reply, my next question is;
Are the fees actually taken at the moment? I know they aren't shown but are they still taken if I set it in markets.yml?
—
Reply to this email directly or view it on GitHub https://github.com/peatio/peatio/issues/420#issuecomment-73382231 .  https://github.com/notifications/beacon/AIx21m4NG54oJZDeXkc5dTLq7JsDmCjcks5npnEVgaJpZM4Dc_wx.gif

I have read the documentation, but it doesn't specify if the fee is actually taken. I know it is not shown anywhere but nowhere does it say if the fee is actually taken from the trade.

No fees taken. System works fees free
You need  write your own module
From: CryptNZ<EMAIL_ADDRESS>Sent: Monday, February 09, 2015 7:28 PM
To: peatio/peatio
Cc: ValueAd
Subject: Re: [peatio] Trading Fee (#420)
I have read the documentation, but it doesn't specify if the fee is actually taken. I know it is not shown anywhere but nowhere does it say if the fee is actually taken from the trade.
—
Reply to this email directly or view it on GitHub https://github.com/peatio/peatio/issues/420#issuecomment-73550909 .  https://github.com/notifications/beacon/AIx21l8vLyEDTR_8gMDTkId0Ob9DhYVGks5nqOWggaJpZM4Dc_wx.gif

CryptNZ did you solve this issue?

It requires some customization, if you know how to code its not too hard otherwise you will have to wait for an update from the dev's to incorporate it.

@CryptNZ its considered polite to share changes to open source code with others, even if it was on your own fork.

@MrDunne Yes but even we haven't fully implemented it yet, we don't see the point sharing half finished/working work.


Syntax highlight does not work properly

Since the latest update I now have the problem that the syntax highlighting no longer works properly.
The syntax highlightning only works if you import 'highlight.js/lib/core' and the corresponding language:

This is the result of working with 'highlight.js/lib/common':

Is there some error in console? Need more specifics.

Since the latest update

Clarify what is the "latest update" and what version you are running?

I have now set up a completely new project and implemented everything as described. I also get no error displayed in the console:

With correct style import*


 To me it seems like we have half of a solution for each version of BS. This became evident to me when the styling of my default inputs in BS2 weren't really fixed to look like BS.  If people agree, I'd be happy to help clean this up, more-so on the BS2 side of things.  Maybe 2 files, select2-bootstrap2.css and select2-bootstrap3.css

Closing this off because of the lack of activity. If this is not fixed in 4.0, and you are interested in seeing it fixed, add a :+1: to the ticket to bring it to our attention.
If you can add additional details, that would also be useful.


Draft BM1 extension and move rotate with carry into it

The only instruction missing from x86's BMI1 is BEXTR, but that requires 3 arguments. Instead of BEXTR, I've included BZHI which from BMI2

Also discussed here: https://discord.com/channels/828292123936948244/946806826756882552/1208923377763360769


Cleanup of Splat/ply

Goal to automate it after initial manual editing

In this issue we are researching to find ways to edit existing splats. When we will have a dataset of the KIMS we will need to make another set of issues for putting all of this into practice.
A raw Splat usually has varying amount of Gaussians we dont want or need, outside or inside the scene.
Cutting down a  Splat is easy and quick, no need to automate that part.
Removing artifacts inside the Scene however can be very challenging and time consuming.
The Supersplat Webeditor is a great tool for this process, and I havent found a decent alternative that doesnt cost money.
I've found its feature of selecting gaussians by size/opacity, combined with its options of further modifying a selection incredibly useful.
To to this in the Editor:

Use Splat Size/Opacity in the Selection part
set the Slider to your cutoff point (for example with Splat Size 0: everything, 0.5: anything half as big as the largest)
use Add to select by that filter and Invert if needed before removing
It can get rid of mostly transparent artifacts like from lensflares quickly and in my tests can reduce the filesize by a third while improving visuals.

Also very useful is using the circle cameramode to select gaussians by where they extend to instead of where they are centered.
What this means is that we can effectively "trim" clusters into shape. (For example getting rid of gaussians that vertically stick out of a floor.)
As its an open source project we might be able to build on features, like filtering gaussians by size and texture, to add more specific functionality. In my tests with an organic scene, there were a lot of white artifacts left over to trim manually around small details like leaves. Filtering by color like we can already filter by opacity could be possible to implement.


Replace a link where googlers ask for /experimental permissions with a short go one

Currently provisioning script asks Googlers to request access for /experimental folder using a large URL.
We should replace that with go/experimental-folder-access

Fixed.


Brazil data guide

the csv file on github here
has 2 columns that are not in the current brazil data guide.
how should microcephaly_indicative_of_infection and microcephaly_zika_positive be coded?

The corresponding table has not been included in the most recent reports which we started with, it appears to have been discontinued. Your names are good, but as the variables come from microcephaly_confirmed, they could alternatively be coded as:
BR0009, microcephaly_confirmed_infection_signs
BR0010, microcephaly_confirmed_zikv_positive


GatewayClass spec.controller non-empty requirement is not enforced

The GatewayClass spec.controller field is documented as "cannot be empty", but there's no minimum length validation, and the spec field itself is optional. This means that it is possible to create a GatewayClass with an empty controller name.

/assign


Allow passing a document node or nodelist

Hi, great module! thanks for your effort!
It would be great if we can also pass an element or array of elements to in-view... something like this:
inView(document.querySelectorAll('img')).on('enter', function(img) {
   console.log(img.height);
});

what do you think?

Ultimately I would not even accept string based selectors. For such a low level library, no magic is fine. Accept only Node or Node Lists :)

@vvo @bertez I agree, it would be nice to just pass a Node or NodeList.
One nice thing about using string selectors is that we can use it as a key to keep track of registered handlers for that selector.
inView('.selector').on('enter', el =>
    el.classList.add('in-view'));

inView('.selector') // returns the same registry as above

If we add the ability to use Node & NodeList, it will be up to the user to maintain a reference to that registry.
const registry = inView(document.querySelectorAll('.selector'));

Admittedly, that's not the worst thing in the world. It wouldn't break any existing functionality.

Thanks for this great library!
I'd also like to see an option to pass an Array of nodes as I'm using jQuery and it returns an Array of nodes, not NodeList. This would make jQuery interop much simpler.

+1

I vote for only one DOM node to pass as @vvo mentioned. The utility size should be kept at the minimum.


introducing blog and static mapping plus code indention and typo fixing

Hi, this fixes three lines where the automatic indention did not work and one comma.
thanks

ok,
(still surprising moments with git and github)
in aboves pull request are minor changes combined with changes in config.js and cms.js for letting the user handle his site with a listing of posts on the frontpage or with the contentn of an existing pagename and letting the listing of posts be on another url.
as issued in https://github.com/cdmedia/cms.js/issues/16 by @ali4ever4 and tagged as enhancement


AnnData.concatenate generates -inf values

I'm using anndata to store some facs data. After reading in the files (each file one sample) and storing the values in anndata format, I concatenate all the samples with ann.AnnData.concatenate(*anndata_sample_training). However, after concatenation, there are some -inf values introduced to the X data matrix. I checked the data before and after concatenation and before I have no -inf values but after concatenation I suddenly have around 30 -inf values in my dataset (values stored in X).

I haven't been able to reproduce this behaviour locally. Could you provide a minimal reproducible example of this? Something like what's described here. Information on what versions of packages you are using would be helpful too (like the output of sinfo.sinfo(dependencies=True)).


to master

Coverage remained the same at 100.0% when pulling b1ca64c12f13300f9ac86e8b0dcbe0f9e4fae7b4 on next into 294114430cdf54da6e13b260bb55ca1aab4dccc1 on master.


v0.2.0

Looks like if you updated main package-lock.json, you also need to update the one from samples, otherwise samples' npm install fails for some reason. We're still learning how to live with that :)
Can you try running
rm -rf node_modules samples/node_modules package-lock.json samples/package-lock.json
npm install && npm link && cd samples && npm link ../ && npm install && cd ..

and then add both package-lock.json and samples/package-lock.json to this PR?

Codecov Report

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

@@          Coverage Diff          @@
##           master    #15   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           1      1
  Lines           4      4
=====================================
  Hits            4      4

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 a672630...e6105a4. Read the comment docs.

@alexander-fenster seems to have done the trick!

@callmehiphop Dave, may I ask you to do the same trick once more to regenerate lock files? I pushed one more change to master (#18) and now this PR cannot be merged. When you update the lock files, feel free to draft a release of the latest master. Thanks!

Nevermind, I was able to resolve conflicts using GitHub web editor :)

Nope, it's still bad. Looks like I cannot fix it properly in your repo - can you still regen the lock files please? :) @callmehiphop

OK - I found out I can update your repo, so here we go! :)


Add check for module item visibility

closes #598
What was wrong?
Item visibility is not checked, so the ingot below is not rejected properly.
# foo.fe
const FOO: i32 = 1

# main.fe
use foo::FOO
contract Main:
    fn foo() -> i32:
       return FOO

How was it fixed?
Add check for visibility in resolve_name and resolve_path.
To-Do

[ ] OPTIONAL: Update Spec if applicable

[x] Add entry to the release notes (may forgo for trivial changes)

[x] Clean up commit history

Codecov Report

Merging #677 (c3c2fac) into master (90d5162) will increase coverage by 0.40%.
The diff coverage is 90.34%.

:exclamation: Current head c3c2fac differs from pull request most recent head 73f67c5. Consider uploading reports for the commit 73f67c5 to get more accurate results

@@            Coverage Diff             @@
##           master     #677      +/-   ##
==========================================
+ Coverage   81.71%   82.11%   +0.40%
==========================================
  Files         125      125
  Lines       10760    10849      +89
==========================================
+ Hits         8792     8909     +117
+ Misses       1968     1940      -28

Impacted Files
Coverage Δ

crates/analyzer/src/namespace/types.rs
72.03% <0.00%> (-1.04%)
:arrow_down:

crates/analyzer/src/traversal/call_args.rs
94.66% <ø> (ø)

crates/analyzer/src/traversal/declarations.rs
92.72% <ø> (ø)

crates/parser/src/ast.rs
88.47% <ø> (ø)

crates/yulc/src/lib.rs
88.46% <ø> (ø)

crates/analyzer/src/context.rs
61.34% <44.44%> (-1.05%)
:arrow_down:

crates/analyzer/src/db/queries/module.rs
78.96% <66.66%> (ø)

crates/analyzer/src/namespace/scopes.rs
79.13% <94.54%> (+7.26%)
:arrow_up:

crates/analyzer/src/traversal/expressions.rs
80.94% <95.45%> (+0.31%)
:arrow_up:

crates/analyzer/src/namespace/items.rs
92.52% <96.87%> (+4.01%)
:arrow_up:

... and 18 more

Continue to review full report at Codecov.

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

Yeah, ideally it would be best.
Some difficulty here is that items can be referenced even if they don't appear in use statements. i.e., through wildcard or path.
In that case, a visibility check must be performed on a ref point. I guess that's why rustc and rust-analyzer emit the error at all reference points.
This error is easy to understand, so one option would be to reduce the error message to improve the user experience a little.

Anyway, I'll merge as is, this would fix the broken CI.


Jobs Form: Add a form field for OSS projects to paste a link to their active License for their projects

https://opensourcedesign.net/jobs/job-form/ (the public form page)
https://github.com/opensourcedesign/jobs/blob/master/job-form.html (The code that needs changing)
So if an OSS project doesn't have a 'easy' to access License or License's (if a project has multiple licenses to address different 'types' of work) then we reject the PR for a job posting - it's become a sort of unofficial criteria for a job post - There are some exceptions where a project might not have a license or an appropriate license for the type of content may not exist. These project can be approved by having a conversation with OSD maintainers to assure them they are following open source principles.
I'd like to suggest we add a mandatory (so with the *) free text field to the job form specifically asking for the job poster to share any public license documents there so we can review and approve. These can be text files in repos or they can be webpage etc. we should be careful not to discriminate towards a repo license.
Suggested copy:
Form label: Project License/s *
Form help text: Project must have some kind of open source license publicly viewable to be accepted. You can share any kind of document e.g. link to the license in your public repo, a webpage, an online cloud document etc. we only want to check that you are following open source principles.
(Large free text box)

I think the help wording could be improved but that's my best go right now at wording 😄

we only want to check that…

suggested wording: "…be sure that…"

we only want to check that…

suggested wording: "…be sure that…"

Edited the main issue text to change to this wording - thank you


how to get metadata

hi,
wondering if its possible to read a model and get the metadata?
thx

Not implemented sorry. You would need to read the related ir.model.data record if any and the create or update date eventually.


How to get highscore data from a song

What is your question?
Is there a way to get certain data from a song's highscore? Such as the score, rating, or ratingFC string? I ask because I already released my own mod about a week ago, and the current system I have means that people would have to replay the songs that they've already completed for the save data to update.

Dunno why you'd only need to grab those, but you can parse the data saved in the user's .sol files.
user name > AppData(hidden file) > Roaming > ShadowMario > PsychEngine > ninjamuffin99
The save data should carry over to following versions if the game's/mod's directory doesn't change.

I know where it is. And I assume that I can use the lua save data functions to grab it, but how would I know what exactly to grab? (How do I read the .sol files?)
FYI, I want to use this to recreate an entire menu.

If your source mod has a new directory, I recommend duplicating and modifying Highscore.hx that runs after Main.hx. It'l grab save data from another location and flush it before the titlescreen.

For lua, that system mostly stores external vars other than save data, the game's system should handle it for you. If you want it like that though, use the same functions you use to write to the file as the ones you use to read them.

Turns out the wiki wasn't updated, check for getTextFromFile()'s source callback in funkinLua.hx check highscore.hx to see what variables to load.

initSaveData() and flushSaveData()

So I'm able to get simple variables from the funkin.sol file like noteOffset and framerate, but the main problem is getting certain variables from maps.

I dunno how to get array stuffs with lua, I'm currently working on a source mod. Best advice I got is converting source code from Highscore.hx to lua.

Well, then I think I already have that covered. I just wanted to make sure if there was another way to do it via lua.


Completion of Utility PV example

This PR contains Correction and completion of Utility PV Example.
The major changes are -

Removed np.prod with multiplication signs as they were giving erroneous results.
Added new values to the investment scenarios of Low R&D, High R&D, Mid R&D but the parameters and design numbers are all the same.
Added plots for metrics.
Added investment calculations and optimization.

Maybe add a comment to the notebook that the input datasets don't capture technology changes with investment so users know the graphs will just be flat across all scenarios.
A tip for future analysis notebook changes - when committing/pushing notebook changes, first go to Kernel -> Restart and Clear Output. That way only changes to the code and text cells will be recorded, not changes in the output.

Thank you I will do this in the next commit.


Feature request: Configurable typesetting engine (like xelatex or lualatex)

When I try to use Living Papers with the PLATEAU LaTeX class, LaTeX tells me:
! Fatal Package fontspec Error: The fontspec package requires either XeTeX or
(fontspec)                      LuaTeX.
(fontspec)
(fontspec)                      You must change your typesetting engine to,
(fontspec)                      e.g., "xelatex" or "lualatex" instead of
(fontspec)                      "latex" or "pdflatex".

Switching from pdflatex to xelatex in the LP code seems to fix things. Would be great to have this as an option, if that doesn't open too many cans containing too many worms.

Random note for posterity: The PLATEAU LaTeX class also relies upon biber as a BibTeX backend, and biber doesn't like citation keys with parentheses in them, making this the hilarious flip side to #23.


Input out of bounds in a model exported from onnx function

Error:
Traceback (most recent call last):
  File "/home/titaiwang/onnx-script/bug_onnxscript.py", line 11, in <module>
    expected = onnx.shape_inference.infer_shapes(expected)
  File "/home/titaiwang/anaconda3/envs/pytorch-dev/lib/python3.9/site-packages/onnx/shape_inference.py", line 40, in infer_shapes
    inferred_model_str = C.infer_shapes(
RuntimeError: Input 1 is out of bounds.

Repro (modified from @justinchuby comment below)
import onnxscript
from onnxscript import FLOAT, opset18 as op
from onnxscript.function_libs.torch_aten.ops.core import aten_amax
import onnx

@onnxscript.script()
def expected_model(x: FLOAT[2, 3, 4]):
    return aten_amax(x, op.Constant(value_int=0), keepdim=False)

proto = expected_model.to_model_proto()
with_shape = onnx.shape_inference.infer_shapes(proto)
onnx.checker.check_model(with_shape, full_check=True)

cc @gramalingam @justinchuby

The previous was a wrong call since dim is an input. The new function
import onnxscript
from onnxscript import FLOAT, opset18 as op
from onnxscript.function_libs.torch_aten.ops.core import aten_amax
import onnx

@onnxscript.script()
def expected_model(x: FLOAT[2, 3, 4]):
    return aten_amax(x, op.Constant(value_int=0), keepdim=False)

proto = expected_model.to_model_proto()
with_shape = onnx.shape_inference.infer_shapes(proto)
onnx.checker.check_model(with_shape, full_check=True)

gets an error
Traceback (most recent call last):
  File "/home/justinchu/dev/onnx-script/testtest.py", line 12, in <module>
    onnx.checker.check_model(with_shape, full_check=True)
  File "/home/justinchu/anaconda3/envs/onnx/lib/python3.10/site-packages/onnx/checker.py", line 117, in check_model
    C.check_model(protobuf_string, full_check)
onnx.onnx_cpp2py_export.checker.ValidationError: Field 'shape' of 'type' is required but missing.

<
   ir_version: 8,
   opset_import: ["" : 18, "onnxscript.atenlib" : 1]
>
expected_model (float[2,3,4] x) => (float[] return_val) {
   tmp = Constant <value_int = 0> ()
   return_val = onnxscript.atenlib.aten_amax <keepdim = 0> (x, tmp)
}
<
  domain: "onnxscript.atenlib",
  opset_import: ["" : 18]
>
aten_amax <keepdim>(self, dim) => (return_val)
{
   return_val = ReduceMax <keepdims: int = @keepdim> (self, dim)
}

Hi,
I suspect this is due to this issue in ONNX: https://github.com/onnx/onnx/issues/3985 ... will test it more, and will work on the fix for that issue.

I think the above is correct. For example, changing tmp = Constant <value_int = 0> () to tmp = Constant <value_ints = [0]> () works. I would recommend this anyway, because it is better to keep the sequence of dimensions to be reduced as a 1D tensor anyway. (But there is a bug/incompleteness in ONNX as mentioned above that needs to be fixed too, but you can work around that as above for now ... alternatively, using just 0 instead of tmp may also work, assuming a 0D-tensor is ok.


Add AppliedTo struct separately instead of sharing NetworkPolicyPeer

This PR addresses issue 2686.
Currently AppliedTo shares the NetworkPolicyPeer with To/From fields, yet some members such as Namespaces and FQDN would not be set in AppliedTo. So, it is beneficial to use a separate struct for distinction, as well as future extensions.
Change:

v1alpha1.types, added AppliedTo struct, changed AppliedTo in Rule and ClusterNetworkPolicySpec.
Corresponding tests and codebase that uses this AppliedTo.

Signed-off-by: Qiyue Yao<EMAIL_ADDRESS>

Codecov Report

Merging #3678 (02feaa9) into main (2ab80d0) will decrease coverage by 19.80%.
The diff coverage is 93.33%.

@@             Coverage Diff             @@
##             main    #3678       +/-   ##
===========================================
- Coverage   63.62%   43.81%   -19.81%
===========================================
  Files         278      210       -68
  Lines       39360    35658     -3702
===========================================
- Hits        25041    15624     -9417
- Misses      12381    18898     +6517
+ Partials     1938     1136      -802

Flag
Coverage Δ

kind-e2e-tests
?

unit-tests
43.81% <93.33%> (-0.04%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

pkg/controller/networkpolicy/validate.go
46.91% <92.30%> (-3.72%)
:arrow_down:

...g/controller/networkpolicy/clusternetworkpolicy.go
64.11% <100.00%> (-5.10%)
:arrow_down:

pkg/ovs/openflow/default.go
0.00% <0.00%> (-100.00%)
:arrow_down:

pkg/util/runtime/runtime.go
0.00% <0.00%> (-100.00%)
:arrow_down:

pkg/agent/openflow/groups.go
0.00% <0.00%> (-100.00%)
:arrow_down:

pkg/agent/nodeportlocal/k8s/annotations.go
0.00% <0.00%> (-100.00%)
:arrow_down:

pkg/ovs/openflow/ofctrl_pipeline.go
0.00% <0.00%> (-95.84%)
:arrow_down:

pkg/ovs/openflow/logs.go
10.34% <0.00%> (-89.66%)
:arrow_down:

pkg/agent/controller/networkpolicy/reject.go
0.00% <0.00%> (-87.06%)
:arrow_down:

pkg/apis/controlplane/register.go
0.00% <0.00%> (-83.34%)
:arrow_down:

... and 200 more

I see that Egress is current using AppliedTo struct of v1alpha2 crd. Maybe Antrea-native policies can use that too? Not sure if the added option of externalEntitySelector and serviceAccountSelector would affect the Egress feature. /cc @jianjuns I know Jianjun had the idea of unioning the AppliedTo struct from Antrea-native policies and Egress for a while now, maybe he can provide more context here.
Also, maybe we should come up with another struct so that ANP can stop using the NetworkPolicyPeer in appliedTo as well

I see that Egress is current using AppliedTo struct of v1alpha2 crd. Maybe Antrea-native policies can use that too? Not sure if the added option of externalEntitySelector and serviceAccountSelector would affect the Egress feature. /cc @jianjuns I know Jianjun had the idea of unioning the AppliedTo struct from Antrea-native policies and Egress for a while now, maybe he can provide more context here. Also, we should come up with another struct so that ANP can stop using the NetworkPolicyPeer in appliedTo as well

Union of the two AppliedTo structs sounds good for clarity. I also found v1alpha2.crd, just had one concern, since all the others such as spec builders use v1alpha1 types, including Rule, would it be confusing to use a v1alpha2 AppliedTo? Thanks.

/test-all


time slider view?

I love this! Since I visited for the first time today, I'd love to be able to view the history going back to when it started with an interactive time-slider (or a video, or a gallery). Does this exist anywhere, or is it in the plan?  (Yep - put it out there and we're just going to ask for more features.)

Thanks @hsu000001 ! I've added a crude animation of the prior 120,000 years. It's just a .gif at this point, and I haven't figured out how to control the playback speed, but at least there's something.

Thanks for adding the animation!  I now have a lot more to talk about!


graphicsdriver/metal: Confirm that context-lost actually does NOT happen

This is an investigation work. I need more information.

Unfortunately I could not find any resources about the possibility of context-lost in Metal. However, as Metal API doesn't have any notion of context-lost, and as a relatively new graphics API DirectX 11 doesn't have device lost that happened in DirectX 9 [1], we probably don't have to care about context lost.
[1] https://docs.microsoft.com/en-us/previous-versions/windows/apps/dn458383(v=win.10)


add homepage screenshot testing

this will need electron to take a (base-homepage) screenshot that this can be tested against.

Seems like everything is passing except for the screenshot code. I'll fix this.


test for eager loading

when there are 3 tables between the relation using via() to link it

this is a similar problem as described in #5341

thanks for the test case! reproduced the issue and found the cause. closing this in favor of #11898 which contains more failing tests including yours.


dataServer EADDRINUSE Error

Hi there!
Using ftp-srv on a project and I am getting this error:
ftpServer.on('client-error', (data) => {

    // data.context
    context:"dataServer"

    // data.error
    address:"<IP_ADDRESS>"
    code:"EADDRINUSE"
    errno:"EADDRINUSE"
    port:8080
    syscall:"listen"

});

I have 2 CCTV cameras that FTP snapshots of CCTV Footage to ftp-srv every second. This is not a persistent error, it just happens every now and then, maybe once every few seconds? But every time it happens, there is a 10 second gap for ftp-srv to recover. I really can't pinpoint when or why this is happening though. Tried googling, failed.
Here is the bulk of the code, if it helps:
const ftpServer = new FtpSrv('ftp://<IP_ADDRESS>:9876', {
  pasv_range:8080
});

ftpServer.on('login', (data, resolve, reject) => {

  resolve({root: 'F:/path/to/ftp/root'});

  data.connection.on('STOR', (error, fileName) => {
    var filePath = data.connection.fs.cwd;
    if (error === null) {
      mainWindow.webContents.send('cctvSnapshot', {
        filePath: filePath,
        fileName: fileName
      });
    }
  });

});

Any idea how to fix this?

I tried to dig deeper here and I found out that the cause of this issue is because of simultaneous connections? To test I turned off 1 of the 2 CCTV Cameras and then poof the error went away.

I think this might be caused by how I'm choosing which port to use in a passive connection.
It always starts from the min pasv range and works it's way up.
If 2 connections attempt to get a passive connection at the same time, I can see them both being assigned the same port.
I'm working on some major changes which includes a generator for pasv ports so they are always incrementing.
But some more testing will be needed to see if this is the cause.


brew prune deleting empty dir from /usr/local/etc that I created

Bug reports:
It appears brew prune is searching for and deleting empty directories everywhere in <homebrew_prefix>.
I thought it only is supposed to clean out Cellar.
For ex. I'm creating an empty dir in apache settings for later Ansible use:
mkdir -p /usr/local/etc/apache2/2.4/other

Here's what brew prune does:
$ brew prune --dry-run
Would remove (empty directory): /usr/local/etc/apache2/2.4/other

Suggestion (maybe naive - is there a reason to clean everywhere?)

brew prune should not clean in areas outside of the Cellar.

brew prune never cleans the Cellar, it's specifically for cleaning the prefix.


Explain "Add to Xcode Project"

Please excuse my rookie question, but can you describe the steps for:

Then add RNGeocoder folder to your xcode project.

in more detail?
Thanks!

I would appreciate this as well. The standard instructions from the React docs indicate that an Xcode project is required for native libraries, but I'm not seeing one here. Thanks!

sorry for the late answer. To add RNGeocoder, you can drag and drop RNGeocoder folder (located inside ./node_modules/react-native-geocoder) directly to your xcode project.
you can also right-click on 'Libraries' from the project-navigator, then 'Add files to', and select RNGeocoder.
hope this helps

I don't believe this is adequate, you also need to add RNGeocoder.m to the Compile Sources section of the Build Phases section in xCode, that should absolutely be documented. Had to figure this out by pluggin away
http://cl.ly/0V2w2c0d3s0R

I'm having problem w/ iOS simulator saying that all locations are "not found".  I tried adding the RNGeocoder.m but then get Build Failed with duplicate fields:
Ld /Users/barton/Library/Developer/Xcode/DerivedData/snowflake-ekizixumavllepfmlesfcgrdfsow/Build/Products/Debug-iphonesimulator/snowflake.app/snowflake normal x86_64
    cd /Users/barton/projects/stargazers/client/ios
    export IPHONEOS_DEPLOYMENT_TARGET=9.1
    export PATH="/Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/usr/bin:/Applications/Xcode.app/Contents/Developer/usr/bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin"
    /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/clang -arch x86_64 -isysroot /Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/SDKs/iPhoneSimulator9.1.sdk -L/Users/barton/Library/Developer/Xcode/DerivedData/snowflake-ekizixumavllepfmlesfcgrdfsow/Build/Products/Debug-iphonesimulator -F/Users/barton/Library/Developer/Xcode/DerivedData/snowflake-ekizixumavllepfmlesfcgrdfsow/Build/Products/Debug-iphonesimulator -filelist /Users/barton/Library/Developer/Xcode/DerivedData/snowflake-ekizixumavllepfmlesfcgrdfsow/Build/Intermediates/snowflake.build/Debug-iphonesimulator/snowflake.build/Objects-normal/x86_64/snowflake.LinkFileList -Xlinker -rpath -Xlinker @executable_path/Frameworks -mios-simulator-version-min=9.1 -Xlinker -objc_abi_version -Xlinker 2 -ObjC -fobjc-arc -fobjc-link-runtime /Users/barton/Library/Developer/Xcode/DerivedData/snowflake-ekizixumavllepfmlesfcgrdfsow/Build/Products/Debug-iphonesimulator/libRNGeocoder.a /Users/barton/Library/Developer/Xcode/DerivedData/snowflake-ekizixumavllepfmlesfcgrdfsow/Build/Products/Debug-iphonesimulator/libRCTMapboxGL.a -lz -lsqlite3 -lc++ -framework SystemConfiguration -framework QuartzCore -framework MobileCoreServices -framework ImageIO -framework GLKit -framework CoreTelephony /Users/barton/Library/Developer/Xcode/DerivedData/snowflake-ekizixumavllepfmlesfcgrdfsow/Build/Products/Debug-iphonesimulator/libReact.a /Users/barton/Library/Developer/Xcode/DerivedData/snowflake-ekizixumavllepfmlesfcgrdfsow/Build/Products/Debug-iphonesimulator/libRCTActionSheet.a /Users/barton/Library/Developer/Xcode/DerivedData/snowflake-ekizixumavllepfmlesfcgrdfsow/Build/Products/Debug-iphonesimulator/libRCTGeolocation.a /Users/barton/Library/Developer/Xcode/DerivedData/snowflake-ekizixumavllepfmlesfcgrdfsow/Build/Products/Debug-iphonesimulator/libRCTImage.a /Users/barton/Library/Developer/Xcode/DerivedData/snowflake-ekizixumavllepfmlesfcgrdfsow/Build/Products/Debug-iphonesimulator/libRCTLinking.a /Users/barton/Library/Developer/Xcode/DerivedData/snowflake-ekizixumavllepfmlesfcgrdfsow/Build/Products/Debug-iphonesimulator/libRCTNetwork.a /Users/barton/Library/Developer/Xcode/DerivedData/snowflake-ekizixumavllepfmlesfcgrdfsow/Build/Products/Debug-iphonesimulator/libRCTSettings.a /Users/barton/Library/Developer/Xcode/DerivedData/snowflake-ekizixumavllepfmlesfcgrdfsow/Build/Products/Debug-iphonesimulator/libRCTText.a /Users/barton/Library/Developer/Xcode/DerivedData/snowflake-ekizixumavllepfmlesfcgrdfsow/Build/Products/Debug-iphonesimulator/libRCTVibration.a /Users/barton/Library/Developer/Xcode/DerivedData/snowflake-ekizixumavllepfmlesfcgrdfsow/Build/Products/Debug-iphonesimulator/libRCTWebSocket.a -Xlinker -dependency_info -Xlinker /Users/barton/Library/Developer/Xcode/DerivedData/snowflake-ekizixumavllepfmlesfcgrdfsow/Build/Intermediates/snowflake.build/Debug-iphonesimulator/snowflake.build/Objects-normal/x86_64/snowflake_dependency_info.dat -o /Users/barton/Library/Developer/Xcode/DerivedData/snowflake-ekizixumavllepfmlesfcgrdfsow/Build/Products/Debug-iphonesimulator/snowflake.app/snowflake

duplicate symbol _OBJC_IVAR_$_RNGeocoder._geocoder in:
    /Users/barton/Library/Developer/Xcode/DerivedData/snowflake-ekizixumavllepfmlesfcgrdfsow/Build/Intermediates/snowflake.build/Debug-iphonesimulator/snowflake.build/Objects-normal/x86_64/RNGeocoder.o
    /Users/barton/Library/Developer/Xcode/DerivedData/snowflake-ekizixumavllepfmlesfcgrdfsow/Build/Products/Debug-iphonesimulator/libRNGeocoder.a(RNGeocoder.o)
duplicate symbol _OBJC_CLASS_$_RNGeocoder in:
    /Users/barton/Library/Developer/Xcode/DerivedData/snowflake-ekizixumavllepfmlesfcgrdfsow/Build/Intermediates/snowflake.build/Debug-iphonesimulator/snowflake.build/Objects-normal/x86_64/RNGeocoder.o
    /Users/barton/Library/Developer/Xcode/DerivedData/snowflake-ekizixumavllepfmlesfcgrdfsow/Build/Products/Debug-iphonesimulator/libRNGeocoder.a(RNGeocoder.o)
duplicate symbol _OBJC_METACLASS_$_RNGeocoder in:
    /Users/barton/Library/Developer/Xcode/DerivedData/snowflake-ekizixumavllepfmlesfcgrdfsow/Build/Intermediates/snowflake.build/Debug-iphonesimulator/snowflake.build/Objects-normal/x86_64/RNGeocoder.o
    /Users/barton/Library/Developer/Xcode/DerivedData/snowflake-ekizixumavllepfmlesfcgrdfsow/Build/Products/Debug-iphonesimulator/libRNGeocoder.a(RNGeocoder.o)
ld: 3 duplicate symbols for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)

@bartonhammond how did you solve this error i am also having same


Update jsonschemanet version testing

@daxian-dbw, this is the same as #19610, but I want to use this for testing so I don't gunk up the commit history over there.  Please approve any builds if you need to.  Otherwise, feel free to ignore.
!!! NOT FOR MERGING !!!

Looks like this is the problem, which is what I suspected:
Microsoft.PowerShell.Commands.JsonSchemaReferenceResolutionException: Exception of type 'Microsoft.PowerShell.Commands.JsonSchemaReferenceResolutionException' was thrown.
 ---> System.IO.DirectoryNotFoundException: Could not find a part of the path '/vsts/work/1/s/test/powershell/Modules/Microsoft.PowerShell.Utility/assets/valid_schema_definitions.json'.

I don't understand this because the file is definitely there.
Adding more logs...

More detail:
Fetching: /vsts/work/1/s/test/powershell/Modules/Microsoft.PowerShell.Utility/assets/valid_schema_definitions.json
Reading file: /vsts/work/1/s/test/powershell/Modules/Microsoft.PowerShell.Utility/assets/valid_schema_definitions.json
Json.Schema.JsonSchemaException: Cannot resolve base schema from `file://home/vsts/work/1/s/test/powershell/Modules/Microsoft.PowerShell.Utility/assets/valid_schema_definitions.json#/definitions/name`

I'm unsure why this isn't able to find the file.  I'm open to ideas from someone who know linux better.

Okay.  It looks like it's a difference in how filename URIs are handled.
var windowsFilePath = "C:\\Folder\\target.json";
var unixFilePath = "/Folder/target.json";

var windowsFileUri = new Uri($"file:///{windowsFilePath}");
var unixFileUri = new Uri($"file:///{unixFilePath}");

Console.WriteLine(windowsFileUri);
Console.WriteLine(windowsFileUri.AbsoluteUri);
Console.WriteLine(windowsFileUri.AbsolutePath);
Console.WriteLine();
Console.WriteLine(unixFileUri);
Console.WriteLine(unixFileUri.AbsoluteUri);
Console.WriteLine(unixFileUri.AbsolutePath);
Console.WriteLine(unixFileUri.Host);

file:///C:/Folder/target.json
file:///C:/Folder/target.json
C:/Folder/target.json

file://folder/target.json
file://folder/target.json
/target.json
folder

That last line is missing the folder.  It's interpreted as the host and not included in the path.
Also interesting is the lower-casing.  Since *nix operating systems have case-sensitive file systems, I wonder if that's going to cause a problem.
@adityapatwardhan @daxian-dbw I would love to get your thoughts on this.

Looks like I might be able to just use .LocalPath.
Windows: C:\Folder\target.json
*nix: \\folder\target.json

Getting closer.  Not sure why it's changing to backslashes.
Fetching: file://home/vsts/work/1/s/test/powershell/Modules/Microsoft.PowerShell.Utility/assets/valid_schema_definitions.json
Reading file: \\home\vsts\work\1\s\test\powershell\Modules\Microsoft.PowerShell.Utility\assets\valid_schema_definitions.json

I wonder to see #/definitions/name tail:
JsonSchemaException: Cannot resolve base schema from `file://home/vsts/work/1/s/test/powershell/Modules/Microsoft.PowerShell.Utility/assets/valid_schema_definitions.json#/definitions/name

No, the fragment is removed before attempting to fetch the file.  Only the base URI is used in the fetching code.  The fragment only is resolved after the file content is retrieved.

Interesting file:///var/ vs file://var/:

Yep... Looks like the problem is in how I build the base URI from a *nix file path.  I'll push out a release and update here first.  If it works, then I'll apply changes to the other PR and close this.  Then we can all move on.

Not sure why more tests are failing, but the URI issue is resolved.  Probably my other error handling changes.
I'm going to just update the package version on the other PR and see what happens there.  The pester tests work locally on that branch with the new version.  Closing this for now.


sf login functions jwt: TypeError: Cannot read properties of undefined (reading 'token')

I'm able to login to my org using sf login org jwt -a vdallano -i CLIENT_ID -f ../jwt/server.key -l https://test.salesforce.com -u USERNAME --json, but when I try sf login functions jwt -a vdallano -i CLIENT_ID -f ../jwt/server.key -l https://test.salesforce.com -u USERNAME --json I get the following error TypeError: Cannot read properties of undefined (reading 'token')

I just changed the word org to functions and hit enter. So what could be the problem?
I tried to update sf to the latest version but it didn't work

It only works with Prod orgs that have functions enabled. You both probably tried to login with a sandbox.

The creds were for Production. The issue was that I was trying to log in to org and functions with an API-only user.
But it requires standard UI login for the first time to accept the Functions access terms.
After one-time UI login, I was able to do that with jwt.
I guess it is a corner case to be covered by Salesforce.


content doesn't contain custom fields

Previous to version 2, by executing gh projects item-list $PROJECT_ID --org $ORG --format json | jq I was able to get custom fields in the content, which I found useful to perform some queries against those. Please restore this.

Sorry, I was making a mistake and didn't see the value.

@eddumelendez can I close this issue?

I though it was closed. Yes!


$ sign in router path generates parse error

Describe the bug
Our API path includes $ signs, as they're widely used in the FHIR medical record format. Including the $ signs in the router comment leads to a swag error:
ParseComment error in file main.go :can not parse router comment "/objects$modifier [get]"

To Reproduce
Create a main.go file with the following content:
package swag_test

// @id specialObjectList
// @Summary Special objects listing
// @Router /objects$modifier [get]
// @Description A special listing method...
// @Tags Test
func specialObjectList() {
}

Run swag init
Expected behavior
To have the /objects$modifier route correctly registered.
Your swag version
e.g. 1.7.4
Your go version
go1.17.2 darwin/amd64
Desktop (please complete the following information):

OS: macOS 11.3.1

Additional context
I believe #551 was a similar issue; I'd be happy to make a similar PR to add a $ sign into the regex if it would be accepted.

try use version 1.6.7

@sam-hoffman I don't believe "/objects$modifier" is a valid route in OpenAPi spec. This may be the reason we are not accepting $ sign in route definitions.

@ubogdan is there a part of the OpenAPI spec that specifically prohibits objects$modifier? It would be super helpful to add, as FHIR is a widely used specification for medical data that calls for $modifier

@sam-hoffman it's not necessary about prohibiting then it's about you won't be able to use the generated definition with swagger-ui to test the API.
Extracted from paths-and-operations
Swagger supports path templating, meaning you can use curly braces {} to mark parts of a URL as path parameters:

/users/{id}
/organizations/{orgId}/members/{memberId}
/report.{format}

We actually do manually add $ signs in to our generated definitions in order to have our paths display correctly in swagger-ui. We were hoping to avoid having to manually add them.

I see. According to rfc3986, the provided path "/objects$modifier" contains a special character.
I feel sorry for you, but this is not a valid issue, and we won't go forward with it.

What about #551, which also added a special character?

What about it?

It looks like it was accepted and swag now allows a : in router paths

We accepted that pr because It's a well-known practice (used in google API) and we felt like it's nice to have.
I don't see any benefit to introduce another special character. Why can't you simply use ":"?

I can't use : because FHIR, a well known and widely used specification for medical data, uses $

Well, can you prove it?

Specifically, the FHIR spec defines operations which make use of the $ character.
https://www.hl7.org/fhir/operations.html

In the end, this is an OSS project , feel free to contribute and break it :smile:.

Nice, thank you :) Will send a PR soon


A report debug endpoint that collects all key metrics for troubleshooting  / support

Summary
a special debug endpoint for collecting key metrics for support / troubleshooting
Motivation
when node operators face issues, and come to discord for support - we ask them for their setup, os, what A endpoint output is what B endopint output is etc. It would be easier to have a single report endpoint where all important troubleshooting data from various other debug endpoints can be pulled and presented to streamline this process.
Implementation
some data that is often requested by tech support on discord to identify or troubleshoot issues are:

hasSufficientFunds
isFullySynced
depth
reachability
radius
storageRadius
... etc
from some of the more common endpoints:
/health
/topology
/status
/reservestate
/redistributionstate
etc

config file and data-dir locationd based on the type of setup would be useful too
Drawbacks

This can be done externally by writing a script which queries all the endpoints and generates a report. This is already done in the integration tests where we collect all the endpoint data to debug.
Can you share more info on why this is needed?

Yeah, a script would work too.
I have a custom script that does this which i use for troubleshooting - but it pulls just some of the key parameters from some debug api endpoints.  I could turn that into a npx script for this purpose.


Only FTDs handle Discovery Requests and send Discovery Responses.

Current coverage is 74.41% (diff: 79.10%)

Merging #998 into master will increase coverage by 14.28%

@@             master       #998   diff @@
==========================================
  Files           119        124      +5
  Lines         16073      17725   +1652
  Methods        2325       2486    +161
  Messages          0          0
  Branches       1930       2131    +201
==========================================
+ Hits           9664      13190   +3526
+ Misses         5867       3862   -2005
- Partials        542        673    +131

Powered by Codecov. Last update 9de7f12...3aced3a


How does pytorch handle backward pass in a multi-GPU setting?

Past works have often proposed remote GPU caching as a performance optimization. As an example, if data x originally stored on GPU0 is requested by GPU1, then x is cached in GPU1's L1 or L2 cache (there are pros and cons if it's cached in L1 or L2 depending on the workload).
While applying remote caching is trivial for inference, getting it to work for training seems to be more challenging. Primarily because we need to maintain coherence across all cached copies after the backward pass. I was curious to know how Pytorch actually handles all this under the hood. I've read that the gradients are calculated based on a computational graph.
Some questions I have:

in case of model parallel training, does the computational graph also store which device contains the required weights? (I think it is yes)
what does pytorch do if data is remotely cached (there are 2 options here: invalidate all cached copies and only update the original data or update both original and cached copies)

Any pointers to find the answers to these questions would be great. Thanks!

I think that this is a general question and will probably be more holistically answered on Pytorch forums. I would suggest just reposting it there.

ok, will do. Thanks @mnaumovfb


fix: enable alloc_layout_extra for nightly feature

The latest nightly builds have apparently put alloc_layout_extra behind a feature gate, causing compilation failures with the nightly feature enabled. This adds the feature.
error[E0658]: use of unstable library feature 'alloc_layout_extra' (see issue #55724)
   --> /Users/kyleclemens/.cargo/git/checkouts/hashbrown-fc08982fe9a23efa/4496e06/src/raw/mod.rs:147:16
    |
147 |     let data = Layout::array::<T>(buckets).ok()?;
    |                ^^^^^^^^^^^^^^^^^^
    |
    = help: add #![feature(alloc_layout_extra)] to the crate attributes to enable

bors r+
Thanks!

Published in v0.1.6.


indy.error.IndyError: ErrorCode.WalletNotFoundError

In your README you say "try incrementing the schema version and fully restarting the server" to resolve this error. However, where should the schema version be incremented?permitify/site_templates/bc_registries/schemas.json? That didn't seem to work when I tried it.
Also, why would that have any effect when the docker-compose.yml file sets the env var      PYTHON_ENV: development, which in the code appears to set all of the schema versions to "0.0.0":
https://github.com/bcgov/permitify/blob/fd912a89f29fc67c5074dfd69717bce5ffbd4c9d/src/von_connector/schema.py#L37-L39
Is there something else I'm supposed to do to get the wallet to work?

How about referring to the schema by its hash in development mode? Then if the schema is already present on the ledger, it doesn't need to be registered again, and we don't end up with lots of identical clones. If it does change, so will the hash, so they won't clobber each other.
In production, it may still be a good idea to have a version number; I don't think full semver is a good idea, but it might be useful to be more human readable and provide some indication of sequence. That or a date, so that it would have more meaning.


CLI: Remove now-unnecessary "enhanced"  Kubernetes object types

The "enhanced" types aren't necessary because now the Kuberentes API
implementation has the correct JSON annotations for the InitContainers
field.
Signed-off-by: Brian Smith<EMAIL_ADDRESS>

The base branch for this PR is #380. I will rebase this onto the master branch after #380 is merged.

Rebased onto master.


v6 outstanding work

[x] improve typing to prevent null or arbitrary types from being dispatched to actions
[x] include jsdoc in package output
[x] add real unit tests instead of a react project with manual tests
[x] add more jsdocs to exported functions, hooks, and members
[x] update existing examples in examples package
[x] add better examples to readme
[x] remove old migration docs from readme
[x] update API section in readme
[x] add v6 breaking changes section to readme
[x] remove all jsdoc types
[x] add prettier for formatting
[x] ensure react support is current
[x] update all deps
[x] export immer stuff to allow for additional configuration
[x] ~add engines config to package.json~
[x] update github actions
[x] renovate bot? dependabot?

🚀


I need help

I want to use form builder with invoice creator example can you help me how to do it

In the GitHub repository we only discuss about Symfony bugs and new features. For this kind of questions about using Symfony, please use any of the support alternatives shown in http://symfony.com/support Thanks!


Inconsistent removal of extra whitespace in header fields

In the course of looking at the feasibility of using the Mail Gem edge build I'm pretty sure I found a bug in the 2.5.4 release / 2-5-stable branch that was fixed by this commit: 72befdc4dab3e6e288ce226a7da2aa474cf5be83
I've added a regression spec to this branch:
https://github.com/edsinclair/mail/commit/6aad4919c4def83e939d1955b4f332ae8b2248f7
Not sure if you'd be interested in a patch against the 2-5-stable branch. Feel free to close this, if not.

Interested in a PR against master, for sure, but I don't follow the new specs or what was fixed/regressed :)  Could you explain?


how to run multi tenant deployment

I have a seperate db for every client . Ex : Client 1 has db1 and client 2 has db2 and the db username and password is different for each clinet which is maintained in the tenant db . How do i automate the migration process ex : when i want to create a table in all the client databased or insert a data in all client data based . How to automate them ?

You should ask those questions in some other forum. They are not really related to objection itself. Objection doesn't even handle migration, but delegates it to knex.


ngHint spec/successConfig.js depends on non-existent "./environment.js"

I'm trying to run npm test and see this failure:
running: node lib/cli.js plugins/timeline/spec/conf.js
>>> pass
running: node lib/cli.js plugins/ngHint/spec/successConfig.js
ERROR - failed loading configuration file plugins/ngHint/spec/successConfig.js

/home/tullmann/demeter/protractor/lib/configParser.js:195
    throw e;
          ^
Error: Cannot find module './environment.js'
    at Function.Module._resolveFilename (module.js:338:15)
    at Function.Module._load (module.js:280:25)
    at Module.require (module.js:364:17)
    at require (module.js:380:17)
    at Object.<anonymous> (/home/tullmann/demeter/protractor/plugins/ngHint/spec/successConfig.js:1:73)
    at Module._compile (module.js:456:26)
    at Object.Module._extensions..js (module.js:474:10)
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:312:12)
    at Module.require (module.js:364:17)
>>> fail: Error: expecting exit code: 0, actual: 8

Its pointing at this:
https://github.com/angular/protractor/blob/master/plugins/ngHint/spec/successConfig.js#L1
Its not clear if this directory should have its own "environment" file, or symlink the real one, or reference the real one (in ../../../spec/environment.js).
(I had to disable "spec/multiConf.js" because my Firefox (v35) has the wont-talk-to-webdriver bug.  I don't think that should break this, but you never know ...)

Fixed with 1.7.0


Switch pip3 to uv in Dockerfile

Ugh sorry for 3x PRs... that's what I get for trying a new workflow 👎 There was a PR from @lainedfles that's unfortunately not reflected here now, apologies for that as well.
Description
This PR introduces uv, a new package manager, as a partial replacement for pip3 in the Docker build process for the "Open WebUI" project. Uv is a mostly drop-in compatible substitute for pip3 with significant performance improvements. In our testing, uv has shown to notably reduce build times — by nearly 50% on GitHub Action runners and by approximately 25-30% when building locally across various platforms. While the resultant build sizes have varied, showing both increases and decreases depending on the build environment, this change aims to leverage uv's efficiency gains to enhance the development workflow and process efficiency. It's important to note that pip3 has not been completely removed; it remains necessary for installing PyTorch and its dependencies, as uv has not yet reliably resolved them across all build environments. As this is an evolving integration, more opinions and testing are highly encouraged to ensure a smooth transition. For further details on uv, please refer to the uv documentation and this introductory blog post.

Changelog Entry
Added

Introduced uv as the new package manager in the Docker build process for certain dependencies, aiming to improve build times and efficiency.

Fixed

N/A

Changed

Partially replaced pip3 with uv in the Docker build

Using pip3:

Using uv:

Changelog Entry
Added

Introduced uv as the new package manager in the Docker build process for certain dependencies, aiming to improve build times and efficiency.

Fixed

N/A

Changed

Partially replaced pip3 with uv in the Docker build process. This does not yet cover PyTorch and its dependencies, which still rely on pip3 due to resolution issues in some environments.

Removed

N/A

Pull Request Checklist

[x] Description: This PR partially introduces uv as a replacement for pip3 in the Docker build process, aimed at improving build times and efficiency for certain dependencies. Pip3 remains in use for installing PyTorch and its dependencies, due to current limitations with uv.
[x] Changelog: A detailed changelog entry has been added, highlighting the introduction of uv, the partial replacement of pip3, and noting the continued use of pip3 for specific dependencies.

one way to find out if this works 🚀

Lol have that revert commit ready if it starts causing issues, I won't be offended. I've been using it for a couple weeks now on various platforms without problems so I'm mostly confident 😬


failed to cherry pick PR 4062 from cyclinder, to branch release-v0.8

commits 603c57e04da85655a6f7672612f2463fad5cffda of cyclinder conflict when merging to branch release-v0.8, please manually cherry pick it by yourself.
PR https://github.com/spidernet-io/spiderpool/pull/4062 ,  action https://github.com/spidernet-io/spiderpool/actions/runs/10933291536
Auto-merging pkg/coordinatormanager/coordinator_validate.go
Auto-merging pkg/multuscniconfig/multusconfig_mutate.go
Auto-merging test/doc/spidermultus.md
CONFLICT (content): Merge conflict in test/doc/spidermultus.md
Auto-merging test/e2e/spidermultus/spidermultus_test.go
CONFLICT (content): Merge conflict in test/e2e/spidermultus/spidermultus_test.go
error: could not apply 603c57e0... Merge pull request #4062 from cyclinder/spidermultusconfig/fix_panic
hint: After resolving the conflicts, mark them with
hint: "git add/rm <pathspec>", then run
hint: "git cherry-pick --continue".
hint: You can instead skip this commit with "git cherry-pick --skip".
hint: To abort and get back to the state before "git cherry-pick",
hint: run "git cherry-pick --abort".
hint: Disable this message with "git config advice.mergeConflict false"

@cyclinder

fixed #4099


[List] Add keyboard navigation

Discussion draft - applies the same logic to List as currently used in MenuList.
Not resolved:

Sublist navigation
Sibling list navigation

(Not that MenuList currently supports these either.)

@mbrookes Why should a list have keyboard support? I thought that the correct way to navigate within a list is to use the tab.

You might be right.
I was applying the principals described here:
"A primary keyboard navigation convention common across all platforms is that the tab and shift+tab keys move focus from one UI component to another while other keys, primarily the arrow keys, move focus inside of components that include multiple focusable elements."
And in 5.6 Keyboard Navigation Inside Components
"As described in section § 5.1 Fundamental Keyboard Navigation Conventions, the tab sequence should include only one focusable element of a composite UI component. Once a composite contains focus, keys other than Tab and Shift + Tab enable the user to move focus among its focusable elements."
This allows skipping past the nav with a single Tab, and has the added benefit that the active list-item (if any) can receive focus when tabbing onto the list. (See: 5.6.1 Managing Focus Within Components Using a Roving tabindex).
The question then is, should a nav be a composite component? It certainly seems better for accessibility, and yet that very page does not treat its TOC as a composite component...

Benchmarking:
https://material-components.github.io/material-components-web-catalog/#/component/list - composite
https://material.angular.io/components/list/overview#selection-lists - composite
https://ant.design/components/list/ - broken (no keyboard support of any kind)
https://vuetifyjs.com/en/components/lists - broken (only the list with checkboxes supports keyboard navigation)
https://getbootstrap.com/docs/4.3/components/list-group/#links-and-buttons - not composite
Others?

Thank you for sharing relevant sections of the accessibility guidelines.

Common examples of UI components that contain multiple focusable elements are radio groups, tables, menus, and grids.

There is no mention to a ul>li structure as a popular candidate. I don't think that we should provide this feature to the list by default. Now, they mention the tab. It's something I was talking about with @nathanmarks a long time ago. Maybe we can abstract the menu focus logic, use it in the tabs and possibly make it public to be used somewhere else?

There is no mention [of] a ul>li structure

HTML lists aren't inherently navigable, so it's understandable that they're not mentioned. However, the use of an unordered list for semantic "correctness" in a composite list component that is navigable changes that.

I don't understand this either. A basic list should not be navigateable since the items don't have an associated action. It might make sense for a nav or a ToDo list but just listing things does not require keyboard navigation. It's just text in the end.
In other words: If the items are not focusable there's no need for keyboard navigation.

I still think that this change could help people. Take Gmail, their left side menu implements this behavior (Google keep and Google Contacts don't). It's also related to #15107.

This behaviour is awesome if the list contains clickable elements, however how does the implementation handle lists without clickable elements. I don't see any special treatment for those?

@Pajn Please point me to an example in the spec of a non-interactive list item (other than a disabled list item, of course).
They are by definition "clickable":

They are composed of items containing primary and supplemental actions, which are represented by icons and text.

@Pajn Please point me to an example in the spec of a non-interactive list item (other than a disabled list item, of course).

I see we switched roles here :wink:
But I did misjudge the implementation. If the list contains no interactive items then you won't get keyboard navigation anyway. So this is actually super nice behavior that emerges.
One issue is that it doesn't work for https://deploy-preview-15334--material-ui.netlify.com/demos/lists/#checkbox. Maybe due to the nested action item? Maybe just needs some adjustments in the demo.

I see we switched roles here

Well, you did at least 😉

One issue is that it doesn't work for [...] #checkbox

Good catch! I'll take a look and see if there's an easy fix.


Envoyer Generate CSS and JS messages "sh: 1: mix: not found"

Laravel Mix Version: 6.0.43
Node Version (node -v): 12.18.3
NPM Version (npm -v): 6.14.6
OS: Ubuntu 18.04.2 LTS

Description:
We are using Envoyer for our Deployments. I have updated all our NPM packages in the last few days. Unfortunately, Envoyer now throws an error in the Generate CSS and JS Hook every time.
sh: 1: mix: not found
npm ERR! code ELIFECYCLE
npm ERR! syscall spawn
npm ERR! file sh
npm ERR! errno ENOENT
npm ERR! @ production: mix --mix-config=webpack.mix.js --production
npm ERR! spawn ENOENT
npm ERR!
npm ERR! Failed at the @ production script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
The strange thing is that I don't get any errors via console when I go into the directory and run e.g. an npm run production .
This error only comes up when I deploy using Envoyer.
I just don't know what else I can do to make it finally work again :-(
Steps To Reproduce:
Simply run a deployment process with Envoyer

Clone New Release
Install Composer Dependencies
Generate CSS and JS

here we do i simple:
cd {{release}}
npm run production

Thats all.

It looks like I solved this problem by simply removing all npm statements from composer.json and executing them directly in the deployment hook.
Generate CSS and JS:
cd {{release}}
npm install
npm run production