Don't include directive names automatically via GraphQLDescription?

https://github.com/ExpediaDotCom/graphql-kotlin/blob/861443970ae9859710967ae65c257e23bee47595/src/main/kotlin/com/expedia/graphql/schema/extensions/annotationExtensions.kt#L23-L32
Currently we include directives names automatically in schema descriptions, I think this should not be done.

Duplicate information: The graphQL fields already support deprecated, we don't need an additional Directive: deprecated in the description
Exposing internal implementation details: Something like @requiresPermision(...) is not necessarily something that should be exposed

I think the option is good, but it should not be the default, AFAIK if desired, a developer could use SchemaGeneratorHooks.onRewireGraphQLType to achieve the same. We could provide a helper method that appends directive names to the description.

deprecated directive is a kind of an oddball directive as that is the only one that is exposed through introspection. We envisioned use cases where exposing this information would be beneficial to the clients, one example would be to mark some queries experimental (opposite of deprecated) so from a single service you would be able to easily differentiate between the two. That being said there are also many use cases when exposing this data might not be desirable.
Currently we print out all the applicable directive names (it is just a comment) but I agree that this should be configurable. Ideally we should have some mechanism to limit the exposure of those directives. We'll look into this and get back to you.
See corresponding discussion around: https://github.com/facebook/graphql/issues/300 and https://github.com/graphql-java/graphql-java/issues/1017

Hm, I see. Theoretically this could also be done by adding an expose: Boolean = false to GraphQLDirective, though im not sure how we would best put the information into the schema then, for users of those pattern just the directive name would probably be not enough...
Seems the discussion on the spec in that regard is still on-going, I'd vote to remove this behavior until it is either in spec or someone presents an actual use-case for it.

I agree with @d4rken on this - I'd rather just wait for schema directives to be added to introspection


Better exports

Shouldn't this mixin also extend the @untool/core mixin and then accept config and options in the constructor and call super with them? Or just lose the constructor entirely as it isn't doing anything? https://github.com/untool/untool/blob/bd27b3fff70f7fcb85d87510db7b4d0c8d81c249/packages/cli/mixin.core.js#L8

@ZauberNerd I think the untool mixin should not inherit from @untool/core: untool is primarily meant to be installed globally, and we would have to jump through some hoops to reliably get a grip on @untool/core.
I have added the missing options argument, though.

Code-wise LGTM - what about documentation though?

@ZauberNerd docs will follow soonish - the previous exports were undocumented as well.


New Update

Hi, nice to see that you are updating the plugin.
Is it up on NPM now?
I'm a bit afraid of updating as there is no information, is this vesrion the same as the previous one?

I want to know what is the change to

nothing was changed


Fixes Issue [#9](Fixes Issue #9

Description
Fixes Issue #9

set chef_ingredient action to :install when an explicit version is found
added ChefSpec examples

Issues Resolved
#9
Check List

[X ] All tests pass. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/TESTING.MD
[X ] New functionality includes testing.
[X ] New functionality has been documented in the README if applicable
[X ] The CLA has been signed. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/CONTRIBUTING.MD

cancel this PR


Mark as library for ModMenu

ModMenu includes a filter to hide mods that are libraries, so that the mod list is less cluttered.
This just adds the required data in the fabric.mod.json so that ModMenu knows LangFilesPlus is a library.
Just so happened that I noticed it when going through the modlist to show the mods I used to someone.
For me it was the mod Mod-erate Loading Screen using LangFilesPlus.
Congrats on such a simple and useful tool, when using Github code search to find where the entry may come from, I noticed that (at least) 21 projects import your mod.

Thank you for your PR! If i remember i'll release it later today.
I've been meaning to archive this mod since you can replicate the exact behaviour with some gradle magic at build time
but i guess its still useful in some cases


[Todoist] Adds labels for in-focus tasks

Description
Adds a preference to set a specified label for the task in focus. The label will be removed when the task is not in focus.
This helps filter in-focus tasks on Todoist UI. The label will only be applied if the preference textfield is not empty.
Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

closes https://github.com/raycast/extensions/issues/12730

Great job! It's something I wanted to have/to do. And great job to allow the user to let them define the name of the label as well. I currently use Focus for instance.

Hey @kud , just changed the preference title in latest revision, can you please review again?

All good for me.


authWithOAuthRedirect does not return authData when calling $onAuth teh first time

When using a call to authWithOAuthRedirect, the authData is not returned on the initial $onAuth callback. No other consecutive callbacks are raised.   When reloading the page or attempting a second login with social networks, the authData is returned correctly.
This leads to a work around to track the initial session after a successful social network login to automate the second attempt thus saving the end user from having to click or reload the page.
similar case here:  http://stackoverflow.com/questions/26390027/firebase-authwithoauthredirect-woes

Please provide a minimal, complete, verifiable repro.

Closing due to inactivity. Feel free to re-open if you have a repro (JSFiddle, Plunkr, etc.) for us.


Enable checkers

resolves #2056
resolves #2383

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- analyzer/codechecker_analyzer/analyzers/config_handler.py  6

Clones added
============
- analyzer/tests/unit/test_checker_handling.py  7

Clones removed
==============
+ analyzer/codechecker_analyzer/analyzers/clangtidy/analyzer.py  -1
+ analyzer/codechecker_analyzer/analyzers/clangsa/analyzer.py  -1

See the complete overview on Codacy

This will be rebased after #2835 is merged.

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- analyzer/codechecker_analyzer/analyzers/config_handler.py  6

Clones added
============
- analyzer/tests/unit/test_checker_handling.py  7

Clones removed
==============
+ analyzer/codechecker_analyzer/analyzers/clangsa/analyzer.py  -1
+ analyzer/codechecker_analyzer/analyzers/clangtidy/analyzer.py  -1

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- analyzer/codechecker_analyzer/checkers.py  2
- analyzer/codechecker_analyzer/analyzers/config_handler.py  6

Clones added
============
- analyzer/tests/unit/test_checker_handling.py  7

Clones removed
==============
+ analyzer/codechecker_analyzer/analyzers/clangtidy/analyzer.py  -1
+ analyzer/codechecker_analyzer/analyzers/clangsa/analyzer.py  -1

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- analyzer/codechecker_analyzer/analyzers/config_handler.py  6
- analyzer/codechecker_analyzer/checkers.py  2

Clones added
============
- analyzer/tests/unit/test_checker_handling.py  7

Clones removed
==============
+ analyzer/codechecker_analyzer/analyzers/clangtidy/analyzer.py  -1
+ analyzer/codechecker_analyzer/analyzers/clangsa/analyzer.py  -1

See the complete overview on Codacy

I tried this out and noticed that
CodeChecker analyze -e profile:something
and there is no such profile we are not giving en error.
same for CodeChecker analyze -e guideline:something
I think we should give an error in these cases as for the profiles and guidelines we expect exact match.
About checker groups: It is not fully clear for me what is a checker group.
Maybe we could add a bit more clarification in the
https://codechecker.readthedocs.io/en/latest/analyzer/user_guide/#toggling-checkers
section.
As I see if the we match for checker names with a prefix match.
for example
CodeChecker analyze -e sec works.
as well as
CodeChecker analyze -e bugprone-
is this the intended behaviour?

So we agreed that
-e profile:no_such_profile
and
-e guideline:no_such_guideline
should give error.
Otherwise lgtm.

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- analyzer/codechecker_analyzer/analyzers/config_handler.py  8
- analyzer/codechecker_analyzer/checkers.py  2

Clones added
============
- analyzer/tests/unit/test_checker_handling.py  7

Clones removed
==============
+ analyzer/codechecker_analyzer/analyzers/clangsa/analyzer.py  -1
+ analyzer/codechecker_analyzer/analyzers/clangtidy/analyzer.py  -1

See the complete overview on Codacy


Max # of processed objects in state file probably won't work if # of backends are high

I notice that we download an object-per-backend for ELB.
If the ELB has, say, 4 backends, this scales perfectly fine with https://github.com/honeycombio/honeyelb/blob/master/main.go#L33, since we would get about (60 * 24 / 5) * 4 ((minutes per hour * hours per day / minute interval) * # backends) == 1152 objects per day.
cc @maplebed for thoughts when you get a chance.
Easy (hack) solution: Just kick maxProcessedObjects up to a high value like 100000, and/or make it configurable. It's plain text so it doesn't take up that much space on disk (approx. 0.2 kB for each entry). Each ELB gets its own state file.

To be clear: The problem here is that if the max processed objects limit is hit before the objects are expired time-wise (i.e., more than a day has passed), some objects which should be present in the state file will be rotated out, which could result in duplicate events.


Difference between time formats (extension window vs. hint) for PT-BR

What steps will reproduce the problem?

Have Chrome set to PT-BR;
Open Extension window by clicking in its button and see event time in 24h format;
Hover over Extension button and see next event's time in 12h format.

What is the expected result?
Time formats should be the same (particularly for PT-BR, it's 24h format).
What happens instead?
Time formats are different (24h x 12h). BTW, date translations are also wrong (in hint, they're in English; in window, they are in PT-BR translation but in wrong format - it's "dddd, mmm dd" but should be "dddd, dd 'de' mm"; example: "Segunda-feira, 14 de março").
Screenshot:

System Information:

Version: 2.6
Language: pt-BR
Timezone Offset: -03:00
User Agent: Mozilla/5.0 (Windows NT 10.0; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/49.0.2623.87 Safari/537.36

[X] I have read and performed all the steps listed under the Troubleshooting section:
https://github.com/manastungare/google-calendar-crx/wiki/Troubleshooting

I have the same issue. Tooltip always uses 12h time format. I have LC_TIME=en_DK.UTF-8


read-only power user

Is your feature request related to a problem? Please describe.
As far as user access in VinylDNS we have super users who can access and modify all data, or users who can modify groups and records based on group membership and ACL rules. Super user abilities are potentially destructive if not used correctly, so there's only a select few with that level of access. Unfortunately the lack of an in between makes it difficult to support users if you aren't a super user, and therefore cannot view their zones and groups.
Describe the solution you'd like
A read-only power user would have the ability to view all zones and groups but they would not be able to modify any of the data. We want users to make their own modifications but it's much easier to support them if we can see their configuration ourselves in VinylDNS when they need assistance.

I believe the only change that should be needed here is for GET endpoints.  All GET endpoints should allow access to the read-only power user.
There are a number of them, they all fall out in one of the Services somewhere.
The only "tricky" by-pass is when listing zones (which is not that tricky as there is already a path defined for super users).  I presume that the power user would be able to see all zones.

This needs a new attribute on users, an existing data migration on current vinyl installations, and a protobuf change right? Really like the concept, will make support easier and safer, and will allow for auditors if they are desired by whoever has a vinyl installation

@nimaeskandary we do not protobuf users (because they were born in the portal).  So, we can just add the flag to the user table.  If it is not present, then default it to false.
The issue that does come to light here is that we need a way to make someone a readOnlyPowerUser.  Not sure how to do that at all.  We also do not have a way to make someone a vinyldns admin either.  So we should discuss that here.

ah good points @pauljamescleary. I was thinking of using groups too. Like a super user group and a readOnly power user group to organize who has what abilities.

The simplest approach would be to use a group.  Since "super users" have access to all the things, they can manage that group.  The idea would be to have a "support user" group (I like support user rather than read-only power user for naming).  The support user group should be system defined / hard-coded (like named vinyldns-support).
We would have to do some tomfoolery in access validations however, to ask "isSupportUser" everywhere, that really just checks the user's groups.  In those validations, maybe a "flag" of some sort is preferable?

gotta think about the portal too btw
1st pass, this person probably cant see the full manage zone tab info because the logic for viewing is all or nothing on the ACL and connection info.
There will also have to be portal logic in there to show/hide the edit/delete buttons.
In general, assume both the api and portal have to know that this user is of that type, rather than checking a group, but the storage could potentially be by group

on the making, i do think it makes sense to give the site admins the power to give someone read only access. in the future, it may be nice to give them even more fine grained power (ACL!?!) so they can make someone a support user for a subset of zones, though i dont want to build out something like that til we need it.
the "how do we make admins in the 1st place" is still an open question for me

I was actually thinking something along the lines of @rebstar6's zone-inclusive ACL rules as well.
Regarding the actual implementation, I am a bit indifferent to having a user flag vs group.

Did we decide on using a group?
For the portal, I would think that everything is available to support users, we would just make everything read only (except of course for things the user has access to).
For "how do we make someone an admin", we need a separate card.  I think we have a single "admin" for the system.  The admin has the ability to manage support users and that is about it (but we can talk about that on the other card.)

my portal point is that we cant ignore it - we have logic around what to show to each person (for example, do you show the update or delete buttons on the recordsets). Im not positive what this logic would do with this type of person if we make no portal changes.

Where the discussion I believe ended on this was to postpone the issue.
We need to thing through allowing read-only "any" access to users.
On the surface, there should be no issue with allowing all users to discover / see (read only) everything in the system.  From a UX standpoint, this has UI implications that need to be thought through.
Questions include:

How does a user see zones they cannot access?  Right now, the zones view shows only "my zones" that I have access to.  How does a user access other zones?  For example, a separate tab, a search filter, etc.
Are there any bits that we do not want read-only users to see.  For example, ACL rules.
Is a blanket read-only solution tenable?  Or do we need some "vinyldns admin" role for privilege management?

All to be discussed later.  Taking this out of discussion for the time being.

Initial
boolean on the user
treat all GETs as okay for read-only
keep PUTs for super users only
Next

API: add ability to enable/disable read-only user access (only super users can do it)
update portal to determine what to show read-only users


Add stream related javadoc

Adding some java doc to the interfaces
Moving SimpleConsumerFactory to KafkaConsumerFactory, and deprecating SimpleConsumerFactory

Codecov Report

Merging #3436 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master    #3436      +/-   ##
============================================
+ Coverage     70.57%   70.57%   +<.01%
  Complexity        4        4
============================================
  Files           997      998       +1
  Lines         45611    45612       +1
  Branches       6108     6108
============================================
+ Hits          32189    32192       +3
  Misses        11311    11311
+ Partials       2111     2109       -2

Impacted Files
Coverage Δ
Complexity Δ

...t/core/realtime/stream/StreamConfigProperties.java
66.66% <ø> (ø)
0 <0> (ø)
:arrow_down:

...ore/realtime/impl/kafka/SimpleConsumerFactory.java
100% <100%> (ø)
0 <0> (ø)
:arrow_down:

...kedin/pinot/core/realtime/stream/StreamConfig.java
99.33% <100%> (ø)
0 <0> (ø)
:arrow_down:

...core/realtime/impl/kafka/KafkaConsumerFactory.java
100% <100%> (ø)
0 <0> (?)

...in/pinot/core/query/pruner/ValidSegmentPruner.java
57.14% <0%> (-28.58%)
0% <0%> (ø)

...a/manager/realtime/RealtimeSegmentDataManager.java
75% <0%> (-25%)
0% <0%> (ø)

...lix/EmptyBrokerOnlineOfflineStateModelFactory.java
86.66% <0%> (-13.34%)
0% <0%> (ø)

.../impl/dictionary/LongOffHeapMutableDictionary.java
81.81% <0%> (-10.91%)
0% <0%> (ø)

...e/operator/dociditerators/SortedDocIdIterator.java
54.05% <0%> (-10.82%)
0% <0%> (ø)

.../predicate/NotEqualsPredicateEvaluatorFactory.java
84.74% <0%> (-6.78%)
0% <0%> (ø)

... and 26 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 2e8e20e...2121e7a. Read the comment docs.

Removed all unused methods from StreamLevelConsumer

Addressed review comments @mcvsubbu


Fixed ambiguous indentations

If True:
    if True:
       pass
    else:
       pass

Was ambiguous, as the alignment of the else was not enforced. This is fixed now for if, while and for

Wrong base


removed unused node power apis

Error: Cannon find module 'ad-block' is real - suppressing the DCHECKS around it still results in crash, or rather, the browser builds but doesn't load

that error is typically related to how node_modules are built and is likely not related to this change. I'm also not seeing it.


JS callback didn't completely worked

Bonjour,
Maybe it was related to my Bootstrap + ElevateZoom theme ? Anyway there was an issue : the "more-views" links then the Switcher thumbnails worked to display the good picture with zoom. But when using the more-views links after the Switcher, it was not possible to get the Switcher related picture and zoom anymore.
I just replaced the JS callback in the admin.
Easylife Switcher :
ProductMediaManager.destroyZoom();
ProductMediaManager.createZoom(jQuery('#image-main'));
My version :
ProductMediaManager.swapImage($j('#image-main'));
Hope it helps.

@MichaelLFR Bonjour.
The js callback is there just for this purpose. It allows the developer/user to exectute any js code he/she needs after the image is changed because I cannot make it work for all possible themes and all possible zoomers.
The default value that comes with the extension works on the default magento theme.
I will close this for now because this is not actually a bug in the module, it has to do with your custom theme.
Thank you for this. I'm sure it will help other users that use the same settings you do.
Cheers.

Wow ! How fast you were ! :D
Yes, it was more a hint than a real issue.
Thank you for your work !


Fix loadin of table rows and ref from xlsx file

Summary
Only structure (columns) of tables are loaded when reading a xlsx file. The data is missing : rows and ref properties remain undefined.
Test plan
A test file is commited. It tests that table data is properly loaded.
Related to source code (for typings update)

@antoineacy Could you please move over here? I'd be happy to merge this feature.


Feature : support google places api ( AutoCompelete input opiton)

is there any plan to add this feature ?

No immediate plans maybe in the future. PRs welcome though.


update for latest image alias and ip sku

While going through this tutorial, the CLI returns some warnings that are easily fixed:
Warning:
WARNING: It is recommended to use parameter "--public-ip-sku Standard" to create new VM with Standard public IP. Please note that the default public IP used for VM creation will be changed from Basic to Standard in the future.

Fix:
Add --public-ip-sku Standard to the VM creation commands.
Warning:
WARNING: Consider using the "Ubuntu2204" alias. On April 30, 2023,the image deployed by the "UbuntuLTS" alias reaches its end of life. The "UbuntuLTS" will be removed with the breaking change release of Fall 2023.

Fix:
--image UbuntuLTS --> --image Ubuntu2204

can one of the admins verify this patch?


Refactor runner

Codecov Report

Merging #193 into master will decrease coverage by 1.2%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master    #193      +/-   ##
=========================================
- Coverage   67.61%   66.4%   -1.21%
=========================================
  Files          19      19
  Lines        1266    1289      +23
=========================================
  Hits          856     856
- Misses        410     433      +23

Impacted Files
Coverage Δ

dpemu/runner.py
0% <0%> (ø)
:arrow_up:

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 a641ac6...0446923. Read the comment docs.


Property: analysis, design, implementation

to define a new db schema
property/columns of the new entity
relationship with the entities of the geodata schema, i.e. address, country


Source of Public Summary in the webserver's display

Great product! I really enjoy using this to map out our IaaS footprint. My question is around the source IP listed as <IP_ADDRESS>/0 and defined as Public coming from the Cloud. Please see the image i have attached.
Why is cloudmapper defining some external connections coming from specific /32 or /16 addresses and some coming from <IP_ADDRESS>/0?

CloudMapper looks at your Security Groups to determine what is allowed access to your resources.  In cases where an unknown CIDR is used, such as a /32 or /16, CloudMapper will create a node for that CIDR.  <IP_ADDRESS>/0 is a special case that CloudMapper renames as "Public".

Thank you very much for  the clarification. Closing the issue


Add hashbang, fix CLI example in readme, add script for installing

Hello!  I am excited to be using taxumap.  These changes allow the tool to be run via the run_taxumap.py script in the repo by adding it to the package install.   I've also fixed the usage example for the CLI interface in the docs.

Thanks for your contribution. Merged.


Refactor domain page

We need to completely refactor the domain page for it to have a better UX/UI IMO.
1/ Being able to search even if I'm not connected to any wallet
2/ Being able to search for subdomains (even external subdomains).
3/ Having maybe a better style of Textfield and components in general (not only material UI basic style).
4/ When a domain is available you directly have a purchase page, when it's not you can check the identity of someone with all the relevant info using this component
Big work both on the design side and the dev side. The new ENS app is a very good example of a good UX in https://app.ens.domains/ for the UI let's make it fun/jungle themed as usual !

done


Escape unicode cookies

Checklist

[x] I'm asking a question and not reporting a bug/feature request
[X] I've looked through the README
[X] I've read the guidelines for opening an issue
[X] I've searched the bugtracker for similar questions including closed ones

Question
Encoding errors continue to occur each time I try to download from Cuetv.
What I have tried:
--encoding=UTF-8
--restrict-filenames
Replacing cookies.txt with new one
How can solve it? I only know a little about programming, so please give me instructions in detail.
+) I found that modifying line 1228 of client.py to values[i] = one_value.encode('UTF-8') works (In fact, another error occurs, but let's solve this problem first.), but I'm afraid it will cause other errors. Is it OK to change the line? If there's a way to solve this without modifying client.py, please let me know.
Verbose log
[debug] Command-line config: ['--cookies=cuetv.online_cookies.txt', '-F', 'https://www.cuetv.online/opera-ariane-et-barbe-bleue-p-dukas-theatre-du-capitole-2019/videos/opera-ariane-et-barbe-bleue-p-dukas-theatre-du-capitole-2019', '-vU']
[debug] Encodings: locale cp949, fs utf-8, out utf-8, err utf-8, pref cp949
[debug] yt-dlp version 2022.04.08 [7884ade65]
[debug] Python version 3.10.4 (CPython 64bit) - Windows-10-10.0.22000-SP0
[debug] Checking exe version: ffmpeg -bsfs
[debug] Checking exe version: ffprobe -bsfs
[debug] exe versions: ffmpeg 5.0.1-essentials_build-www.gyan.dev (setts), ffprobe 5.0.1-essentials_build-www.gyan.dev
[debug] Optional libraries: brotli, certifi, Cryptodome, mutagen, sqlite, websockets
[debug] Proxy map: {}
Latest version: 2022.04.08, Current version: 2022.04.08
yt-dlp is up to date (2022.04.08)
[debug] [generic] Extracting URL: https://www.cuetv.online/opera-ariane-et-barbe-bleue-p-dukas-theatre-du-capitole-2019/videos/opera-ariane-et-barbe-bleue-p-dukas-theatre-du-capitole-2019
[generic] opera-ariane-et-barbe-bleue-p-dukas-theatre-du-capitole-2019: Requesting header
ERROR: 'latin-1' codec can't encode characters in position 1714-1715: ordinal not in range(256)
Traceback (most recent call last):
  File "C:\Users\lg\AppData\Local\Programs\Python\Python310\lib\site-packages\yt_dlp\YoutubeDL.py", line 1408, in wrapper
    return func(self, *args, **kwargs)
  File "C:\Users\lg\AppData\Local\Programs\Python\Python310\lib\site-packages\yt_dlp\YoutubeDL.py", line 1478, in __extract_info
    ie_result = ie.extract(url)
  File "C:\Users\lg\AppData\Local\Programs\Python\Python310\lib\site-packages\yt_dlp\extractor\common.py", line 641, in extract
    ie_result = self._real_extract(url)
  File "C:\Users\lg\AppData\Local\Programs\Python\Python310\lib\site-packages\yt_dlp\extractor\generic.py", line 2728, in _real_extract
    head_response = self._request_webpage(
  File "C:\Users\lg\AppData\Local\Programs\Python\Python310\lib\site-packages\yt_dlp\extractor\common.py", line 767, in _request_webpage
    return self._downloader.urlopen(url_or_request)
  File "C:\Users\lg\AppData\Local\Programs\Python\Python310\lib\site-packages\yt_dlp\YoutubeDL.py", line 3601, in urlopen
    return self._opener.open(req, timeout=self._socket_timeout)
  File "C:\Users\lg\AppData\Local\Programs\Python\Python310\lib\urllib\request.py", line 519, in open
    response = self._open(req, data)
  File "C:\Users\lg\AppData\Local\Programs\Python\Python310\lib\urllib\request.py", line 536, in _open
    result = self._call_chain(self.handle_open, protocol, protocol +
  File "C:\Users\lg\AppData\Local\Programs\Python\Python310\lib\urllib\request.py", line 496, in _call_chain
    result = func(*args)
  File "C:\Users\lg\AppData\Local\Programs\Python\Python310\lib\site-packages\yt_dlp\utils.py", line 1543, in https_open
    return self.do_open(functools.partial(
  File "C:\Users\lg\AppData\Local\Programs\Python\Python310\lib\urllib\request.py", line 1348, in do_open
    h.request(req.get_method(), req.selector, req.data, headers,
  File "C:\Users\lg\AppData\Local\Programs\Python\Python310\lib\http\client.py", line 1282, in request
    self._send_request(method, url, body, headers, encode_chunked)
  File "C:\Users\lg\AppData\Local\Programs\Python\Python310\lib\http\client.py", line 1323, in _send_request
    self.putheader(hdr, value)
  File "C:\Users\lg\AppData\Local\Programs\Python\Python310\lib\http\client.py", line 1255, in putheader
    values[i] = one_value.encode('latin-1')
UnicodeEncodeError: 'latin-1' codec can't encode characters in position 1714-1715: ordinal not in range(256)

I found that modifying line 1228 of client.py to values[i] = one_value.encode('UTF-8') works (In fact, another error occurs, but let's solve this problem first.), but I'm afraid it will cause other errors. Is it OK to change the line?

No! You are modifying code in the python standard library

If your question contains "isn't working" or "can you add", this is most likely the wrong template.
If you are in doubt if this is the right template, use another template!

Is it so difficult to pick the right template despite all these notes!?

I'm sorry. I thought that Broken site referred to a site with an error in itself, not a site that doesn't work in yt-dlp, and I only saw the case of "can you add" by reading the line roughly. By the way, the checklist of the Broken site template is different, so should I edit this or write a new one?

The most important part is verbose log, which you have provided. So not much point in editing it now. The only missing question relevant to this issue at this point is:

[ ] I've read about sharing account credentials and I'm willing to share it if required

If you are willing, send it to me via discord (pukkandan#4207) / reddit (/u/pukkandan) / email<EMAIL_ADDRESS>

I sent you an email.

Error occurs when encoding request header, --restrict-filenames is irrelevant, but --encoding should fix this. (But obliviously it's not doing that atm.)
Adding -X utf8 to interpreter options or using PYTHONUTF8=1 environment variable might help for the time being.

I can't reproduce, even after setting my codepage to 949. Output of --print-traffic may be helpful (it contains sensitive info - only send it privately)

Adding -X utf8 to interpreter options or using PYTHONUTF8=1 environment variable might help for the time being.

check if this works around the issue. Also try chcp 65001

I'm sorry for replying late. All three methods didn't work, so I sent you the log file by email.

I'm sorry for replying late. All the three method didn't work, so I sent you the log file by email.

My suspicion is that your cookies file contains non-latin characters.
Given the following cookies file: (notice the file has non-latin characters)
# Netscape HTTP Cookie File
# This file is generated by yt-dlp.  Do not edit.

.google.com	TRUE	/	TRUE<PHONE_NUMBER>	tracking-id	日本語テスト

I was able to reproduce:
$ yt-dlp --cookies japanese.txt https://google.com --verbose
[debug] Command-line config: ['--cookies', 'japanese.txt', 'https://google.com', '--verbose']
[debug] Encodings: locale UTF-8, fs utf-8, out utf-8, err utf-8, pref UTF-8
[debug] yt-dlp version 2022.04.08 [7884ade65]
[debug] Python version 3.10.4 (CPython 64bit) - Linux-5.15.0-40-generic-x86_64-with-glibc2.35
[debug] Checking exe version: ffmpeg -bsfs
[debug] Checking exe version: ffprobe -bsfs
[debug] exe versions: ffmpeg 4.4.2 (setts), ffprobe 4.4.2, rtmpdump 2.4
[debug] Optional libraries: brotli, certifi, Cryptodome, secretstorage, mutagen, sqlite, websockets
[debug] Proxy map: {}
[debug] [generic] Extracting URL: https://google.com
[generic] google: Requesting header
ERROR: 'latin-1' codec can't encode characters in position 12-17: ordinal not in range(256)
Traceback (most recent call last):
  File "/usr/lib/python3/dist-packages/yt_dlp/YoutubeDL.py", line 1408, in wrapper
    return func(self, *args, **kwargs)
  File "/usr/lib/python3/dist-packages/yt_dlp/YoutubeDL.py", line 1478, in __extract_info
    ie_result = ie.extract(url)
  File "/usr/lib/python3/dist-packages/yt_dlp/extractor/common.py", line 641, in extract
    ie_result = self._real_extract(url)
  File "/usr/lib/python3/dist-packages/yt_dlp/extractor/generic.py", line 2728, in _real_extract
    head_response = self._request_webpage(
  File "/usr/lib/python3/dist-packages/yt_dlp/extractor/common.py", line 767, in _request_webpage
    return self._downloader.urlopen(url_or_request)
  File "/usr/lib/python3/dist-packages/yt_dlp/YoutubeDL.py", line 3601, in urlopen
    return self._opener.open(req, timeout=self._socket_timeout)
  File "/usr/lib/python3.10/urllib/request.py", line 519, in open
    response = self._open(req, data)
  File "/usr/lib/python3.10/urllib/request.py", line 536, in _open
    result = self._call_chain(self.handle_open, protocol, protocol +
  File "/usr/lib/python3.10/urllib/request.py", line 496, in _call_chain
    result = func(*args)
  File "/usr/lib/python3/dist-packages/yt_dlp/utils.py", line 1543, in https_open
    return self.do_open(functools.partial(
  File "/usr/lib/python3.10/urllib/request.py", line 1348, in do_open
    h.request(req.get_method(), req.selector, req.data, headers,
  File "/usr/lib/python3.10/http/client.py", line 1282, in request
    self._send_request(method, url, body, headers, encode_chunked)
  File "/usr/lib/python3.10/http/client.py", line 1323, in _send_request
    self.putheader(hdr, value)
  File "/usr/lib/python3.10/http/client.py", line 1255, in putheader
    values[i] = one_value.encode('latin-1')
UnicodeEncodeError: 'latin-1' codec can't encode characters in position 12-17: ordinal not in range(256)

I don't know how it should be encoded for transfer (or it was supposed to be exist from the first place)
Can you check it? (no need to send or paste here)

My suspicion is that your cookies file contains non-latin characters.
Oh, you're right! Hangul was included in my cookie file.
The line is about the current time, so it doesn't seem necessary.
So I tried without the line, and it said that it's an unsupported URL.
Then what do you guys think is the problem?
Should I restore the cookie file or request for the site support?

Should I restore the cookie file or request site support?

Should open site support request

#4306

https://github.com/yt-dlp/yt-dlp/issues/4306

I opened the site support request. #4306

cookie (file too) should always be ASCII-safe. whatever wrote the cookie and didn't escaped its value, has a bug.
workarounds such as LC_ALL=en_US.UTF-8 before yt-dlp,
or UTF-8 encoding the text (your_string_variable.encode('utf-8')),
or passing a charset parameter in the content-type header of the payload of your requests object (headers={'Content-type': 'text/plain; charset=utf-8'})
should not be needed.

Hey. This has been an issue with yt-dlp from day 1 in my experience but not so with youtube-dl. What do you tell your user to do to avoid this problem?

but not so with youtube-dl.

Show logs of both programs with same options adding -Uv --print-traffic


OG Pump

When you have Pump to OG Pump ON and oupen your  inventory it shows the icon of the new Pump. I dont know if this is a bug. I just wanted to inform you. Have a nice day/weekend proswapper developers

When you have Pump to OG Pump ON and oupen your inventory it shows the icon of the new Pump. I dont know if this is a bug. I just wanted to inform you. Have a nice day/weekend proswapper developers

I had to remove the -L (large) icon for the pump so you wouldn't get kicked, it's a small "bug" but you can live with it :)


Offset reset to default after /reload

When reporting, please verify the issue with only this addon enabled and no others unless explicitly reporting an incompatibility between this addon and another. Likewise, please fill in the below information to the best of your ability — this information helps us a great deal.
Issue description:
Hello, I encountered a problem where whenever I /reload, the vertical and horizontal offset I set no longer works. For example, if I set an offset to 0.2, every time I reload, on the raid frame that indicator is actually at position 0(yet in the configuration it still shows 0.2). I have to do something like setting the offset to 0.21 and setting it back to 0.2 to solve the issue. Is there any solution to this? Thank you!
How to reproduce:
E.g., Steps to reproduce the behavior...

/reload
*offset I set doesn't work anymore unless I retune the offset back and forth *

Very weird. I can't replicate this. Is this on Classic or Retail?

This is retail

Very weird. I can't replicate this. Is this on Classic or Retail?

This is retail

Hrm, these are the lines in question:
local iconVerticalOffset = floor(self.db.profile[i].indicatorVerticalOffset * frame:GetHeight()) --round down
local iconHorizontalOffset = floor(self.db.profile[i].indicatorHorizontalOffset * frame:GetWidth()) --round down

The only way I can see this line of code failing is if for some reason frame:GetHeight() and frame:GetWidth() were erroneously returning 0 instead of a real value, or self.db.profile[i].indicatorVerticalOffset and self.db.profile[i].indicatorHorizontalOffset were erroneously returning 0 (the database default) instead of their set value (between -1 and 1). This ladder should only happen in the case that the database isn't correctly loading by the time we create the frames, but that should then logically apply to ALL database values not just that one set of values.
I've only seen database errors like this happen in the case where someone's computer is REALLY slow or REALLY fast, thus breaking some order of operations based functions. Do you think your computer falls into either of those categories?
Is this on all of your toons or just one?
Was this always the case, or did it just start happening in a specific release?

Is this still happening?


What does TF stands for in Accuracy equation?

Hi, and thank you for this useful resource.
I was checking the definition of Accuracy in the Classification Metrics section. There you wrote:
Accuracy = \frac{TP+TF}{TP+TF+FP+FN}
I believe TF is not an actual quantity, and maybe you meant to refer to TN (true negative) in this equation.
Cheers,
-- SergioA.

Thank you for pointing this out, this should be a typo. Already fixed now.


Run the demo code on Bluemix following the instructions on slide

As a product owner
I want every team member to get familiar with the deployment procedures to  bluemix
so that it will be easy for us to run our own code on the bluemix.
## Assumptions
• Follow the instructions of the slide and be careful of the content of the manifest file which should be in consistent of our own workshop on bluemix.
Acceptance Criteria
• Get the same outcomes with the slides
##Story #

Looks like @alekh16 and myself have completed this step, If @MaskyYoung and @KehanWu , you guys are done, maybe we can move this to 'Done'.


Added explicit cast to numpy to avoid issues with pre-release

This PR fixed issue of incompatibility with the new version of OpenVINO. InferRequest.infer() method does not accept torch.Tensor. numpy.ndarray is required.
Generative models were fixed in the past.
Also moved to OpenVINO pre-release to fix the discrepancy in the results for quantized models with large constants.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

Thanks for the quick fix @AlexKoff88 ! I hope this can be fixed from the OpenVINO side before release because we just added official support for using numpy-compatible tensors in 2022.


Fix minor issues

This pull request has following changes:

Add Mscgen-generated PNG files to Git Ignore target
Correct some spelling issues

Look good. Thank you for your contribution!


Font and widgets are too small

Please add ways increase font size and widgets size.
and if there is any ways of doing this please tell me

We should be able to add a global font size setting today for v0.1-alpha 8 (this will also change the widget sizes). However a individual font setting per widget will be in v0.2. I will close this issue once the font setting is available!

It is also worth noting that the current font (a direct carry over from Dear ImGui) is not really meant to be scaled, so may become a little blurry. We plan on adding our own fonts soon that will be more applicable for scaling.

Added a temporary solution with v0.1-alpha8 using set_global_font_scale. Temporary until we add a more complete system for adding custom fonts and modifying existing ones. Hope this works for now. Closing the issue.


Docker Instances Unable to Use tar Command

Bug Report
System Information

Operating System - Ubuntu 22.04 LTS
AMP version and build date - v<IP_ADDRESS>, built 26/07/2022 20:08
Which AMP release stream you're using - Mainline

I confirm:

[x] that I have searched for an existing bug report for this issue.
[x] that I am using the latest available version of AMP.
[x] that my operating system is up-to-date.

Symptoms

What are you trying to do?
TeamSpeak 3 requires the tar.bz2 to be extracted on Linux. It works fine in a native instance, but in Docker it fails with Error 2.

What are you expecting to happen?
Docker should behave the same as native.

What is actually happening? ('Nothing' is not an acceptable answer!)
It fails to complete the download/extract stage due to Error 2.

Reproduction
Use the TeamSpeak 3 config inside Docker and click Update.

It might be missing the bzip2 package (although I'd have thought that'd be included as standard) - but regardless I've added it to the base image.

Fixed in image update.


Add cluster fetch method

cc @xichen2020

Changes Unknown when pulling 3ab037916102d47412f9b11602fa4fbd6d31bf25 on add-cluster-fetch-method into ** on master**.

LGTM


Klarna Pay Over Time payment method does not redirecting back to the app after payment

Following your official documentation Klarna changed their protocol or something.
when paying with Klarna Pay Over Time. the transaction is set up well and the backend acknowledge it as a valid payment but the SDK is stuck on loader and not redirected back to the app. the didProvide function is not running in this case.
after i close the app and open it again it looks like a successful payment but only the redirection is breaking the flow.
are you planning on deal with this issue? is there a workaround to activate the didProvide callback when paying with Klarna?
thanks in advance, Ben Sagir.

looks like Paypal cancelation has the same issue
i got this warning in the console:
Sending didFailCallback with no listeners registered.

Hey @BenSagir

Sending didFailCallback with no listeners registered.

This means that native code emitting event, no one is listening. Is it possible that AdyenCheckout object or it's parent was recreated?
Make sure you are using useCallback:
import { useCallback } from 'react';

Could you create a blank app and reproduce this behaviour?

@descorp  Is working we fix it Thank you

@descorp  Is working we fix it Thank you

Thanks for letting us know @YankovWeb !
What was the cause of the issue?

We change the screen components  on loading to show animation of loading and AdyenCheckout was re-created.
Thank you.


Issue on page /mastering-plone/dexterity_reference.html

I'm trying to do the mixed field datagrid example:
https://training.plone.org/mastering-plone/dexterity_reference.html#mixedfield-datagrid-field
When I follow the example and load the widget in Classic UI, it looks like this:

And when I try to load this in Volto I just get this error page:
Sorry, something went wrong with your request
Objects are not valid as a React child (found: object with keys {items}). If you meant to render a collection of children, use an array instead.

    at span
    at TextWidget (http://localhost:3001/static/js/client.js:155088:5)
    at div
    at Segment (http://localhost:3001/static/js/client.js:243650:24)
    at div
    at GridRow (http://localhost:3001/static/js/client.js:238762:24)
    at div
    at Grid (http://localhost:3001/static/js/client.js:238538:22)
    at div
    at div
    at Container (http://localhost:3001/static/js/client.js:241118:24)
    at DefaultView (http://localhost:3001/static/js/client.js:152228:5)
    at injectIntl(DefaultView)
    at div
    at View (http://localhost:3001/static/js/client.js:154076:5)
    at ConnectFunction (http://localhost:3001/static/js/client.js:213702:68)
    at injectIntl(Connect(View))
    at Route (http://localhost:3001/static/js/client.js:219264:29)
    at Switch (http://localhost:3001/static/js/client.js:219470:29)
    at main
    at div
    at Segment (http://localhost:3001/static/js/client.js:243650:24)
    at MultilingualRedirector (http://localhost:3001/static/js/client.js:148102:5)
    at PluggablesProvider (http://localhost:3001/static/js/client.js:125787:5)
    at App (http://localhost:3001/static/js/client.js:143713:5)
    at ConnectFunction (http://localhost:3001/static/js/client.js:213702:68)
    at injectIntl(Connect(App))
    at ConnectFunction (http://localhost:3001/static/js/client.js:213702:68)
    at Route (http://localhost:3001/static/js/client.js:219264:29)
    at Switch (http://localhost:3001/static/js/client.js:219470:29)
    at Route (http://localhost:3001/static/js/client.js:219264:29)
    at AsyncConnect (http://localhost:3001/static/js/client.js:159705:5)
    at AsyncConnectWithContext (http://localhost:3001/static/js/client.js:159824:7)
    at C (http://localhost:3001/static/js/client.js:219525:37)
    at ConnectFunction (http://localhost:3001/static/js/client.js:213702:68)
    at ScrollToTop (http://localhost:3001/static/js/client.js:164867:1)
    at ConnectFunction (http://localhost:3001/static/js/client.js:213702:68)
    at Router (http://localhost:3001/static/js/client.js:218893:30)
    at ConnectedRouter (http://localhost:3001/static/js/client.js:17738:7)
    at ConnectedRouterWithContext (http://localhost:3001/static/js/client.js:17844:25)
    at ConnectFunction (http://localhost:3001/static/js/client.js:213702:68)
    at IntlProvider (http://localhost:3001/static/js/client.js:210828:9)
    at ConnectFunction (http://localhost:3001/static/js/client.js:213702:68)
    at Provider (http://localhost:3001/static/js/client.js:213416:20)
    at CookiesProvider (http://localhost:3001/static/js/client.js:175725:28)

I'm not sure what I might have done wrong.

I understand that this is confusing.
The mixed field / datagrid field is not rendered by the default view for non-block-enabled content types.
The default view, which lists all field values should render the value of the mixed field without error. A PR on https://github.com/plone/volto/blob/4e0f9097c900fb81f98a5fd54dbd01e532df371f/packages/volto/src/components/theme/View/DefaultView.jsx#L86 to get an appropriate widget also for a JSONField field could prevent this confusion.
In the mean time, until the default view for content types that are not blocks enabled is enhanced, you can enable the behavior "blocks" on your custom content type. You will see just the title. In edit mode you can see all fields including the mixed field / datagrid field on the right inside the sidebar.
For the display it is up to the developer to create a component displaying the datagrid values. This is highly depending on the needs of the project and the values of the datagrid. An example content type view with a component rendering the datagrid field values can be found in the tutorial.


refactor: use Arc Arc for in-memory cache handle

What's changed and what's your intention?

Please explain IN DETAIL what the changes are in this PR and why they are needed. :D

As title.
Checklist

[x] I have written the necessary rustdoc comments
[x] I have added the necessary unit tests and integration tests
[x] I have passed make all (or make fast instead if the old tests are not modified) in my local environment.


Add support for storage.conf,policy.json; fix user config support

Add support for storage.conf and policy.json
Fix user support - have to get user home dir and add that to paths
Separate config file testing to another test file - too difficult
to figure out which options will interfere with regular container
testing.

[citest]

ok - ready for review


Plugin Config Values

Hi Teppo,
I have another plugin to write to influxdb2 and it is working fine but it hasn't been updated in a long time and I wanted to use yours instead.
I used the same configuration as the other plugin (URL, token, Organisation, Bucket...) but it is not working.
I didn't add gzipThreshold, maxBatchBytes, exponentialBase or any HTTP headers since I didn't know what to put. Is it something needed?
Here is my configuration.
Please let me know if you see something obvious.
Sorry to bother you with that and thank you for your great work.
Best regards.
Romain
Url
http://localhost:8086
Authentication token
PZgGQwoJlXiZpeEHtxDL62ZJe-sDFpwUYvbdoG_uHsR3KbdNUv0U0MoH7oe7s_WabE6Sy9uhuAaQ7sPDVjs6MQ====
Organisation
8e63af9d3843ee9d
Bucket
6824a8eca8241954
batchSize
1000
flushInterval
30000
gzipThreshold
maxBatchBytes
consistency
any
maxRetries
3
maxRetryTime
540000
maxBufferLines
32000
retryJitter
200
minRetryDelay
5000
maxRetryDelay
180000
exponentialBase
randomRetry
X

Try adding just the minimal configuration: url, token, org and bucket.
How do you know it is not working - see any errors on the Dashboard or server log?

You can start from scratch by removing $HOME/.signalk/plugin-config-data/signalk-to-influxdb2.json and restarting the server.

Thank you for your response.
I tried to erase the parameters, remove the json file, update the plugin to today's version, create a new token, new organisation, new bucket but it didn't solve the issue.
I know it is not working because there is no tag keys found in the data explorer of infludb (I try to look the same way I do with the other plugin).
There is one error message created in the log when the plugin starts but then nothing else, hopefully it'll be useful:
v [HttpError]: organization name "8e63af9d3843ee9d" not found at IncomingMessage. (/home/pi/.signalk/node_modules/@influxdata/influxdb-client/dist/index.js:16:3875) at IncomingMessage.emit (node:events:525:35) at endReadableNT (node:internal/streams/readable:1358:12) at processTicksAndRejections (node:internal/process/task_queues:83:21) { statusCode: 404, statusMessage: 'Not Found', body: '{\n' + '\t"code": "not found",\n' + '\t"message": "organization name \"8e63af9d3843ee9d\" not found"\n' + '}', contentType: 'application/json; charset=utf-8', json: { code: 'not found', message: 'organization name "8e63af9d3843ee9d" not found' }, code: 'not found', _retryAfter: 0 }

Well, the log message says your org is not correct.

I see, I put the Organization ID instead of Name. Sorry about that.
It works now. Thank you very much.
Let me know if there is anything I can do to help.
Regards.
Romain

If Signal K is valuable to you consider supporting our work at https://opencollective.com/signalk


Can we make the path to the DLL configurable?

We are using zstd-jni-1.3.8-6.  A handful of our clients get this error, when using ZStd to compress the data exchanged over the Internet:
java.lang.UnsatisfiedLinkError: no zstd-jni in java.library.path Unsupported OS/arch, cannot find /win/x86/libzstd-jni.dll or load zstd-jni from system libraries. Please try building from source the jar or providing libzstd-jni in you system.
(Text was typed, based on a screenshot, and might not be exactly correct)
We don't have any direct access to the computers of those clients, but those that had the error were using "Windows Terminal Server 2008 R2".
Recently we had an issue with one of our servers HDD running full. We discovered that over 4200 copies of libzstd-jni.dll were in the Windows Temp directory, consuming multiple GB of space. That is the point where we realised, that the DLL is copied into the Temp directory.
While those 1000s of copies are annyoing, they are not a deal-breaker; I just coded that the app always try to delete all of those on start. But the real issue is that we cannot fully switch to ZStd, which I would very much like because it gives us much better results than GZip, until all clients can use it.
I have a suspiscion, that those that have a problem, have it because of some permission/anti-virus issue related to the Windows Temp dir.
Now I'm getting to the point ... Could this library be made configurable, so that we can tell it where to store the DLL? Maybe we can then get it to work for those few clients where the DLL is not found.
Currently, all our client and servers are running on Windows, and Java 1.8.X. Most of them x64, but some clients still have 32-bit Java, I believe.

OK, there is a way to specify a path; you have to call this method with a path, before accessing Zstd in any form:
static {
    Native.load("myTempDir");
}

If it is documented anywhere, I did not see it. If it is not documented, it probably should be.
Note: I have yet to try it...
The "thousands of copies" problem seems to be typical under Windows: tensorflow issue

Hi, yes, it's not yet documented (apart from https://www.javadoc.io/doc/com.github.luben/zstd-jni/1.4.0-1), and I agree it is a miss to call it out. I would accept PR for patching the README with a warning, but I don't feel I know all the details as I don't have Windows on hand.
The whole Native.load() thing is a huge mess as it tries to do the right thing for all platforms (unix, windows, mac, android) but I guess there are more edge-cases than I can imagine. I am glad that Native.load(tmpDir) unblocks you.
This method was conriuted with https://github.com/luben/zstd-jni/pull/70 and addresses https://github.com/luben/zstd-jni/issues/69 but I agree it does not explain the motivation.

Looked at the tensorflow issue - looks it's hard problem to solve for Windows, as deleteOnExit does not work as expected.


How should metadata be written in a partitioned dataset?

So far the spec has only covered single-file Parquet data. However Parquet also supports saving as a "dataset", where there are several Parquet files in a folder structure. In this case, how should geospatial metadata be stored? There's a Parquet best practice that writes _common_metadata and _metadata sidecar files to the root of the folder structure, but that's not part of the actual Parquet specification.
If I understand correctly, the geo metadata would automatically be included in the _common_metadata file, and additionally statistics are stored in the _metadata file, which is relevant for #13
Should this be part of the geoparquet spec? Should it be a "best practice" that we document?

Adding here for discussion from https://github.com/opengeospatial/geoparquet/discussions/101#discussioncomment-2784284; we should clarify what the bounding box represents.
Is it possible for each file's metadata to contain only its bounding box, but for the _metadata file to contain metadata for the entire dataset's bounding box? Otherwise, we should probably have the same bounding box in every file, representing the entire dataset?

Where do we stand on this in relation latest discussion pushing for stac 1.0.0-beta.1 sooner rather than later. Should we attempt to put something in there? Has anyone experimented with this and have a good recommendation here?

Do we want to do something here for 1.1? We've now got 'spatial optimizations' ready to go for 1.1, so it seems like a good time to get this to completion to. It sounds like libraries do make use of the _metadata sidecar files, in the comment referenced above Even said

[GDAL] uses the arrow-dataset library helpers. When it finds _metadata, it uses the arrow::dataset::ParquetDatasetFactory class, which avoids reading each fragment file for most operations (except for GetExtent() since there's no way of having the global extent without iterating over each fragment metadtaa). When it doesn't find _metadata, it uses the arrow::dataset::FileSystemDatasetFactory class (not sure if that one needs to open each file)'

Do we want to explicitly say that the _metadata sidecar sets the extent of the whole dataset? Or just leave things as they are?

_metadata sidecar sets the extent of the whole dataset

I don't think there's an API (at least in pyarrow) to manually set the geoparquet metadata on the _metadata file, so the bbox in the file metadata of the _metadata file would be wrong I think. @jorisvandenbossche might know better

Discussed on call 6/3/24 - this should be in best practices, and is not dependent on the specification, so moved off the 1.1 release. We hope to do best practices soon, perhaps make a 'milestone' and push on the things we want there soon after 1.1, but not block release on it.


App doesn't remove lock from apps

Although I set the lock off on an app that was previously locked, AppLock refused to release the app.
I tried stopping the app, clearing the cache and rebooting. I finally had to uninstall AppLock.

Confirm this issue. Sometimes this work but mostly not

One workaround is to add another isLockedNew field and use it for locking and unlocking.


Use Name Instead of File Contents for Untitled Tab

Use Name Instead of File Contents for Untitled Tab
Currently, untitled yaml config files have the tab name matching the first line of our static untitled yaml template:

This appears to be a result of the workbench.editor.untitled.labelFormat setting:
// Controls the format of the label for an untitled editor.
  //  - content: The name of the untitled file is derived from the contents of its first line unless it has an associated file path. It will fallback to the name in case the line is empty or contains no word characters.
  //  - name: The name of the untitled file is not derived from the contents of the file.
  "workbench.editor.untitled.labelFormat": "content",

We can set this setting for our extension to use the name instead of content (default):

Alternatively, it seems we could leave the first line of our untitled.aiconfig.yaml as a newline, but that seems a bit flaky. We can't use a comment as the first line, though:

Merging so Sarmad can create new extension vsix file for others to test now


branch-2.0 fix date_add and diff functions (#35080)(#35104)

fix overflow of date_add function
Proposed changes
Issue Number: close #xxx
pick https://github.com/apache/doris/pull/35080 and https://github.com/apache/doris/pull/35104
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall


Compiler plugin resources addition not working as expected

Description:
We are using compile-time code generators in WebSub module to add an additional resource to the generated executable programmatically.  But even though we are using this code-generator inside WebSub compiler-plugin, the resource is included into a different resource path rather than resources/ballerina/websub/<major-version>/<additional-resource> inside the executable..
This is regarding the  ballerina-lang change #33987

Closing this issue since the original use-case can be done using code modifiers.
Also, resources will be deprecated due to the suboptimal design.


Mixpanel console showing multiple A/B Tweaks even though they are not read from mixpanel in new SDK. (v5.2.3)

We are running multiple AB tests with different tweaks we consider all of them independent.
Before upgrading to version 5.2.3 when ever we need to set a new AB test we essentially used to comment out rest of the AB tests so we are reading just one Tweak from mixpanel SDK and then mixpanel console used to show only that tweak if we were reading multiple tweaks all of them were shown on the console.
After updated Mixpanel version to 5.2.3, we are getting trouble and console is showing some unwanted developer tweakseven though we are not reading those tweaks from sdk in this particular build.
This means that we are not able to create a new AB test independently where we want that experiment to only touch a specific toggle and to create that toggle we even downgraded our mixpanel sdk for particular dev build just to create toggle.
This is a new issue i guess its related to applying all tweaks that user falls in to to even thought those tweaks are not yet defined ie read from mixpanel sdk.
let me know if you need more details.

@patedit Please check out this issue. Any updates on this. Let me know if anything is confusing?

Hi @attifmazhar! Could you write into<EMAIL_ADDRESS>providing a concrete example? Thank you!

Thanks for writing into support! I took a look at your ticket and I open a PR that will fix this issue https://github.com/mixpanel/mixpanel-android/pull/541
Will include it in the next release

Thanks for providing support on this bug


Interoperability issue between sjson and gjson on AppEngine

Hi @tidwall, huge fan of this lib! Never had any issues until I tried using it along sjson on AppEngine, where I noticed sjson.Set* and sjson.Delete* don't appear to be working as intended. Though the bug is encountered when using sjson, I believe the root cause is here inside gjson.
I believe this is because the fillIndex func is noop'd for GAE, but sjson needs to use the index to determine where to append/delete the field from.
The result is all Set* calls simply prepend the field to the start of the JSON, while all Delete* calls always return an errNoChange.

Sorry I haven't responded sooner. I don't have immediate access to AppEngine resource and debugging & development is a little slow for me for that platform. I'll need to dig into this problem a little deeper, but I think the problem stems from the use of the unsafe package for determining byte offsets for fillIndex. There's probably a better way to handle this. When I find a solution I'll ping this issue. Thanks!

No worries @tidwall. FYI, it turns out unsafe is now importable in the Go 1.11 runtime , so at least there is a workaround (which I was able to employ for my case).

That's great to hear. It's been an annoyance to deal with supporting special GAE builds.
Considering GAE intends on supporting unsafe moving forward, I think I'll not make any changes until a new big release. So that we can maintain the same level of compatibility for Go 1.9 GAE users.
Perhaps soon I can remove the _ngae.go and _gae.go files altogether.


Accept Dynamic Entries and RuntimeChunks

Hi,
I'm trying to integrate this plugin with next.js (https://github.com/zeit/next.js/issues/7563)
Next.js has Dynamic Entries, Webpack Documentation
Source:
https://github.com/zeit/next.js/blob/a71cb644b5b7f45521d7a49f5cfca171b3d81a1e/packages/next/build/webpack-config.ts#L336-L341
So had to make a few changes.

Thanks for this work :)
Can you post a small sample webpack config for this? I'll use for adding tests?

I'll add tests itself.

I've added the tests. I might have gone overboard with the various configs, so let me know and I'll delete whatever is unnecessary. A few files are the same. I don't know if I should de-duplicate or let them be? These files would have to be split if webpack modifies it's runtime to use es2015+ features (highly unlikely).

This looks great thanks 👍


raise valueError

raise ValueError(
ValueError: The requested one is expected to belong to ['SE', 'MCD', 'MME', 'ADDA', 'CDAC', 'DAEL', 'DANN', 'AdaBN', 'M3SDA', 'SourceOnly', 'DDAIG', 'DAELDG', 'Vanilla', 'CrossGrad', 'DomainMix', 'EntMin', 'FixMatch', 'MixMatch', 'Mean
Teacher', 'SupBaseline'], but got [CoOp] (do you mean [CrossGrad]?)

looks like a mis-registration error
if you're building a new trainer, please do like this https://github.com/KaiyangZhou/CoOp/blob/main/trainers/coop.py#L211 to register the trainer

Thank you so much for your great job and kind reply.

Hi, I have the same problem. Could you help me how to register the trainer?
Thank you very much!

Hi My solution is to put the CoOp under Dassl.pytorch direction.

Hi, I have the same problem. Could you help me how to register the trainer? Thank you very much!

@ZiyueWu59 , Sir. Did you solve the problem, I have the same error.
It woule be appreciated if you can provide your tips.

looks like a mis-registration error
if you're building a new trainer, please do like this https://github.com/KaiyangZhou/CoOp/blob/main/trainers/coop.py#L211 to register the trainer

Hi, @KaiyangZhou , author, I register my new trainer as CoOp does, but the error still exists. Can you provide some suggestions?


cargo doc is failing

running cargo +nightly doc on a project that includes futures-preview fails with the below error:
...
error: `[Box::pinned]` cannot be resolved, ignoring it...
   --> /home/alfio/.cargo/registry/src/github.com-1ecc6299db9ec823/futures-util-preview-0.3.0-alpha.13/src/try_stream/mod.rs:602:84
    |
602 |     /// first have to pin the stream. This can be done by boxing the stream using [`Box::pinned`]
    |                                                                                ^^^^^^^^^^^^^ cannot be resolved, ignoring
    |
    = help: to escape `[` and `]` characters, just add '\' before them like `\[` or `\]`

error: aborting due to 147 previous errors

Caused by:
  process didn't exit successfully: `rustdoc --edition=2018 --crate-name ruggine_concurrency ruggine-concurrency/src/lib.rs  ...

Meta

rustdoc 1.35.0-nightly (94fd04589 2019-03-21)

The old method name will fix in #1498, but the errors is a problem of rustdoc (see https://github.com/rust-lang/rust/issues/43466#issuecomment-410650216 and https://github.com/rust-lang/rust/issues/56922).
Also, if that project uses #![deny(warnings)] or RUSTDOCFLAGS=-Dwarnings, you may be able to avoid the error by changing it (e.g., use #![cfg_attr(test, deny(warnings))], remove it, etc). (intra_doc_link_resolution_failure is a warning by default).

Thanks - the project uses #![deny(warnings)] and removing it resolved the issue ... now I can generate docs for the project excluding dependencies


Update to Swagger-UI release 2.1.4

This Swagger-UI release fixes issue #516
See: https://github.com/swagger-api/swagger-ui/pull/1668

See 5.3.1


fragment ready/initialized callback

hi folks,
I have been trying to figure out a way to do some initialization after a fragment has been created.
Here is pseudocode:
//parent
export default DS.Model.extend({
  items: fragmentArray('item')
});

// child

export default MF.Fragment.extend({
  init() {
    this._super(...arguments);
    this.get('foo') //null
    this.get('owner') //null
  },

  ready() {
    //never called
  }
})

parentInstance.get('items').createFragment({ foo: 'bar' });

The main issue is that in init, I don't have access to either the directly passed argument foo or the implicit owner.
I also thought that may not be the right callback, and attempted to use ember-data's ready callback but it never gets called here.
I believe these two behaviors differ from ember-data.
Is there a proper callback to use so that I can access explicit attributes and the owner after initialization?

and now I'm seeing that owner is not a public attribute. Is this correct?

I suspect you're referring to Ember's owner they use for instance management and dependency injection? If so, you should use Ember.getOwner(this) to look it up.
With regards to the properties not being available on init. Are you sure it works this way in ember-data? I thought that there internals did not make this possible.
With regards to ready ... To be honest I didn't even know ready existed. So if it this doesn't work, it's probably a bug that we should fix.

@workmanw sorry, I don't know why I assumed there would be an owner property on a fragment.
What I'm referring to is being able to access the fragment's "owner" or "parent". The only way I see to do this now is via _internalModel._owner. Is there a better way to access that? Sorry for the confusion.
In regards to init: I think you are right in that I just tried to do what I suggested in Ember Data itself, and I was unable to access the arguments.
However, (in regard to ready), it does have access to the attributes, so I think it is in fact the callback I'm looking for.
fwiw, I didn't know about ready either :/

@JackCA you can access the fragment owner with https://github.com/lytics/ember-data-model-fragments/blob/158a12f5d7adb24be1a22310402b58b10e584349/addon/attributes.js#L330

@jakesjews awesome, thank you. Should I make a PR to add that to the top-level README?
That solves the access to owner question I have, but still open to discussing the initialization callback

A PR would be cool. I'll take a look into the initialization problem when I get some more free time

Curious about this same topic myself.
https://emberjs.com/api/data/classes/DS.Model.html  shows the list of events
The main ones for most purposes are didLoad or ready
Want me to noodle around and try and hook this up?

I believe @danielspaniel's PR closed this issue. If I'm wrong, let me know and I will reopen.


API endpoints for hardware models

Description
Added API endpoints for  hardware model
Fixes #190
Type of change

[x] New feature (non-breaking change which adds functionality)

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] Any dependent changes have been merged and published in downstream modules

Reviewer: Vinit Shahdeo

@PragatiVerma18 please review the changes.


Python support/installation with python

First of all Sqruff is a very cool project so thank you!
I would like Sqruff to be made available as a python package so that I am able to install Sqruff using a python package manager such as PIP, Poetry  or UV.
Python support would make using Sqruff easier for many python based projects as:

Some Python based teams will only have access to python packages and no ability to install packages to the OS.
Having the Sqruff version be defined in a pyproject.toml file makes it much easier to set up the environment of a project. Currently manual steps would be required.

Similar Rust based project such as Ruff and Polars are made available as a python package and I think this really helps their reach.

Hey @concur1, thanks for all the fantastic feedback! We'll get to it 🚀

No problem!
I am happy to try and pitch in and help a bit with some of the things I suggested. So if you are happy with some of these suggestions and mark them ass accepted  I can will make a PR.

Feel free to open PRs for the suggestions and we'll review.

This is being deployed now with version 0.18.0 so we will have to see if it works.

@concur1 pipeline has run so if you want to try it, let us know.

@concur1 pipeline has run so if you want to try it, let us know.

Thanks, Yep, I have tested it on Ubuntu and it works fine.

Need to remember to document that this installation method exists.


2.1.0 -> 2.2.1 breaks on centos 6.6 with chef 12.3.0

i just tried to upgrade and was forced to roll back. see below...
  * service[portmap] action start[2015-06-30T11:55:18-04:00] INFO: Processing service[portmap] action start (nfs::_common line 43)

    ================================================================================
    Error executing action `start` on resource 'service[portmap]'
    ================================================================================

    NoMethodError
    -------------
    undefined method `action=' for Chef::Resource::Service

    Resource Declaration:
    ---------------------
    # In /var/chef/cache/cookbooks/nfs/recipes/_common.rb

     43:   service component do
     44:     service_name node['nfs']['service'][component]
     45:     pattern node['nfs']['service'][component]
     46:     provider node['nfs']['service_provider'][component]
     47:     action [:start, :enable]
     48:     supports :status => true
     49:   end
     50: end

    Compiled Resource:
    ------------------
    # Declared in /var/chef/cache/cookbooks/nfs/recipes/_common.rb:43:in `block in from_file'

    service("portmap") do
      provider Chef::Resource::Service
      action [:start, :enable]
      supports {:status=>true}
      retries 0
      retry_delay 2
      default_guard_interpreter :default
      service_name "rpcbind"
      pattern "rpcbind"
      declared_type :service
      cookbook_name "nfs"
      recipe_name "_common"
    end

Yeah @koertkuipers I am aware of a few platform-specific SNAFUs trying to get the Service Provider mapped correctly.  Sorry, but I should have time to test, and put out a fix later today.

@atomic-penguin okay thanks! i didnt run into problems on other boxes but that was maybe because they were centos 5 and/or debian

To workaround in the meantime, one can remove the provider parameter in services resources.  This is basically a hack for Upstart on Ubuntu.

@koertkuipers I put out a 2.2.2 patch just now to address this.  Please let me know if this does not work for you, and thanks for reporting!

worked. thanks!
On Wed, Jul 1, 2015 at 12:37 PM, Eric G. Wolfe<EMAIL_ADDRESS>wrote:

@koertkuipers https://github.com/koertkuipers I put out a 2.2.2 patch
just now to address this. Please let me know if this does not work for you,
and thanks for reporting!
—
Reply to this email directly or view it on GitHub
https://github.com/atomic-penguin/cookbook-nfs/issues/58#issuecomment-117742280
.


"Close" button overlaps the scrollbar.

Scrollbar clashes with the "x"/close button.
https://github.com/watergis/maplibre-gl-legend/blob/de661ee8e05cfa85242c10fb3e0aced35a2e1ca9/packages/maplibre-gl-legend/src/scss/maplibre-gl-legend.scss#L6

Firefox 116. DEMO: https://maps.nina.no/datasets/planlagt-utbyggingsareal-i-norge/.
I wonder what would be appropriate here. Putting the cross on the left goes against the usual placement of this kind of button. Maybe a "v" button on the left to minimize/collapse the widget would be preferable?

@frafra Thank you for reporting the issue. I think putting a button to expand / collapse the content is good idea on the top-left. Feel free to create a PR for this issue.


Support retry instead of exit process

Support retry instead of exit process
Error:
  failed to write response body: "task[13]: \"xxxx.tar.32/xxxx.tar.32.13\"": unexpected EOF

pget has supported resumable download. I think it's enough

But when download large file, most the use case would be downloading in background and take hours. Please re-consider about this use case.

FYI, I really love pget. pget is portable (aria2, axel must install by root user) and really fast! (aria2 ~ 600KB/s, pget ~ 3MB/s!)


Add CI

It would be nice to have an automated CI, since there is an actual CD already done, with no CI. The Vercel deploy offers a tool to do CI, I would have to check.

Donde


fix "IP reported on the INFO page is not correct" #13

fix "IP reported on the INFO page is not correct" #13
ip4addr_ntoa() is a "not reentrant" function, that return a static pointer, intended for using into LOG like ESP_LOGI ex. ESP_LOGI(TAG, "got ip:%s\n", ip4addr_ntoa(&event->ip_info.ip)); not convenient  for storing info on a string, beacause in the next use of the function probably the static pointer is the same buffer.
In fact, in the code, the next use of ip4addr_ntoa() is just at the dns variable, which "overwrites" the last string with the same IP as the DNS.
P.s. I also noticed that it is the DNS of the AP interface, not necessarily the GW IP. It may be another bug, change to TCPIP_ADAPTER_IF_AP if you want the DNS server to be assigned to the new network (e.g., the one used in the settings).
So I used a local buffer and secure strlcpy.
See: https://www.nongnu.org/lwip/2_0_x/ip4__addr_8c.html

Convert numeric IP address into decimal dotted ASCII representation. returns ptr to static buffer; not reentrant!

code merged with new release #15
Thank you for your contribution......


Add option to get label value from an HTTP header

This allows to use the proxy in settings where query parameters
cannot be used; this could be behind specific reverse proxies, or
when HTTP POST is required (see #53).
This work is heavily based on the work done by
Andreas Plesner<EMAIL_ADDRESS>in 1. I (Jonas Schäfer) ported
this to the current head of master.
This is a rebase of #54. Please review carefully. I only ran the test suite so far.

It will be userful in grafana datasource, could any one can update for this? :)

@hanjm It indeed is very useful with Grafana :).
I rebased onto current master.

Any update on this?

Hi @horazont . Are you still working on this?

@m-yosefpor Thanks for the ping. I rebased onto current master, but I don't currently have time to test the rebase thoroughly.

Any movement on this patch?


Removing support for Google OAuth2

Removing support for Google OAuth2 in favour of Google OAuth2 plugin: https://github.com/coreruleset/google-oauth2-plugin.

Thank you @azurit. There are a few lint warning about whitespace. Could you check that out, please?

@dune73 See https://github.com/coreruleset/coreruleset/issues/2374.

I don't see any warnings, are they gone?

Still there in the files tab
If you rebase then they will be gone.

Should i care about that warnings? They are not seems to be coming from my changes.

No, don't worry about them.

@azurit Are we removing 930121 on purpose? If yes, what happens with the t:lowercase on that rule. Is it needed in the new 930120?

Yes, also 930121. It was created to simplify rule 930120 after adding GOAuth2 support into it (so it's not needed after GOAuth2 was removed). I restored previous regex of 930120.
I removed t:lowercase as it's not needed - @pm is doing case-insensitive search.

Merging.


generate nfpm at runtime appending the version name

Problem
Updating from 0.0.3 to 0.0.4 caused issues. mysql (pam?) doesn't like reloading pam modules dynamically; and won't pick up the updated lib in /usr/lib64/security
Solution
Have the rpm output the library with version number appended, then when updating this we can update our references in /etc/pam.d/mysql to include the version for example:
#%PAM-1.0
auth required pam_oidc.so.0.0.4 issuer=...
auth required pam_warn.so

Still remaining
If this is ok i'll push up the tag, validate the action behaves properly, and merge

FYI @mjcmtb @jbrandhorst

Some validation this works:
~/blt/pam_oidc:set_version_on_rpm_output ✗ ➭ make rpm
go build -buildmode=c-shared -o pam_oidc.so
env VERSIONED_OIDC_LIB="pam_oidc.so.dev" envsubst '${VERSIONED_OIDC_LIB}' < src_nfpm.yaml > nfpm.yaml
env VERSION=dev nfpm package --packager rpm
using rpm packager...
created package: pam_oidc-dev.x86_64.rpm
~/blt/pam_oidc:set_version_on_rpm_output ✗ ➭ cat nfpm.yaml
name: "pam_oidc"
arch: "amd64"
platform: "linux"
version: "${VERSION}"
description: |
  pam_oidc authenticates users with an OpenID Connect (OIDC) token.
vendor: "Salesforce"
homepage: "https://salesforce.com"
license: "BSD-3-Clause"
contents:
  - src: pam_oidc.so
    dst: /usr/lib64/security/pam_oidc.so.dev
~/blt/pam_oidc:set_version_on_rpm_output ✗ ➭ rpm -qlip pam_oidc-dev.x86_64.rpm
Name        : pam_oidc
Epoch       : 0
Version     : dev
Release     : 1
Architecture: x86_64
Install Date: (not installed)
Group       :
Size        : 9072640
License     : BSD-3-Clause
Signature   : (none)
Source RPM  : pam_oidc-dev-1.src.rpm
Build Date  : Wed  9 Feb 14:42:23 2022
Build Host  : ialpert-ltm.internal.salesforce.com
Packager    :
Vendor      : Salesforce
URL         : https://salesforce.com
Summary     : pam_oidc authenticates users with an OpenID Connect (OIDC) token.
Description :
pam_oidc authenticates users with an OpenID Connect (OIDC) token.

/usr/lib64/security/pam_oidc.so.dev

Note
I did attempt to find a "within nfpm" solution to this, but i believe replacement on these values isn't supported https://nfpm.goreleaser.com/configuration/#templating

This worked
➭ rpm -qlip pam_oidc-0.0.5.x86_64.rpm
Name        : pam_oidc
Epoch       : 0
Version     : 0.0.5
Release     : 1
Architecture: x86_64
Install Date: (not installed)
Group       :
Size        : 9917720
License     : BSD-3-Clause
Signature   : (none)
Source RPM  : pam_oidc-0.0.5-1.src.rpm
Build Date  : Thu 10 Feb 08:38:15 2022
Build Host  : fv-az216-974
Packager    :
Vendor      : Salesforce
URL         : https://salesforce.com
Summary     : pam_oidc authenticates users with an OpenID Connect (OIDC) token.
Description :
pam_oidc authenticates users with an OpenID Connect (OIDC) token.

/usr/lib64/security/pam_oidc.so.0.0.5

I don't know of any other pam library that does this, and unless you carefully update your pam config at the same exact time you upgrade your rpm, this is likely to break auth horribly?

I don't know of any other pam library that does this, and unless you carefully update your pam config at the same exact time you upgrade your rpm, this is likely to break auth horribly?

Yeah!  It's not great, the alternative is to think you've upgraded, but not actually be using the new library (h/t Smiley https://github.com/golang/go/issues/11100).


Filters should filter on slug

See #1746

@cfpb/cfgov-backends is this still an issue?

This can probably be closed.  The code in question has been changed numerous times and we don't reference tags.names() anymore.

👍


fix: Rollback to latest stable release

What
We are reverting the staging and master branches to the codebase of 1.4.0 version. Currently this is the stable and tested one
Changes reverted

Revert "fix: move incremental filter (#388)"
Revert "fix: rollback repartition (#386)"
Revert "chore: level (#382)"
Revert "fix: performance adjustments, migrate (#378)"
Revert "fix: performance improvements (#374)"

Changes maintained

#385
#376

What's next
We'll add in the next PR's (and releases) the new features

It is still in draft to block merge until we fix the commit tree problem


Split embed finders into separate modules

This pull request cleans up the wagtailembeds/embeds.py module by moving the "finders" into a separate folder.

Rebased

Rebased

Merged in d68f4829c43898bdb72c5fbd5c3c9f5b283dcb20 + parents - thanks!


add comments to the test library

Codecov Report

Merging #521 (1e633a3) into master (3874eb9) will increase coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #521      +/-   ##
==========================================
+ Coverage   72.67%   72.69%   +0.02%
==========================================
  Files          83       83
  Lines       10974    10974
==========================================
+ Hits         7975     7978       +3
+ Misses       2999     2996       -3

Impacted Files
Coverage Δ

fedimint/src/net/connect.rs
95.20% <0.00%> (+0.25%)
:arrow_up:

fedimint/src/net/peers.rs
92.95% <0.00%> (+0.52%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Wav File - Malformed fmt  chunk for PCM data

https://github.com/naudio/NAudio/blob/5449b95ffe921c7bbdf55b17c75b2e7d63f1b64e/NAudio.Core/Wave/WaveFormats/WaveFormat.cs#L311-L313
this writes the extraSize into the file, even though it shouldn't exist if the data is PCM.

As a temporary workaround, one can inherit from WaveFormat and override the Serialize-Method. Note, that in addition to deleting line 312 you need to also change this line:
https://github.com/naudio/NAudio/blob/5449b95ffe921c7bbdf55b17c75b2e7d63f1b64e/NAudio.Core/Wave/WaveFormats/WaveFormat.cs#L305
to something like:
writer.Write((int)16);


portal can't get the right favicon from connected site

<EMAIL_ADDRESS>the favicon and the dapp site must be at the same origin
https://developer.mozilla.org/en-US/docs/Web/API/Request/mode


MNT: Re-rendered with conda-build 3.18.9, conda-smithy 3.4.8, and con…

…da-forge-pinning 2019.09.27

Checklist

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

Hello @njzjz! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file build-locally.py:

Line 21:1: E302 expected 2 blank lines, found 1
Line 22:80: E501 line too long (87 > 79 characters)
Line 43:80: E501 line too long (84 > 79 characters)


Writing Unicode text with console code page 65001 (UTF-8) may fail

dlang-bugzilla reported this on 2014-10-25T02:13:44Z
Transfered from https://issues.dlang.org/show_bug.cgi?id=13651
CC List

brocolis
d.bugs
mk
mzfhhhh

Description
Created attachment 1447
Data for demo program

//////////////// demo.d ///////////////
import core.sys.windows.windows;

import std.file;
import std.stdio;

void main()
{
    SetConsoleOutputCP(65001);

    stdout.write(readText("data.txt"));
}
///////////////////////////////////////

data.txt attached.

This crashes in different ways on Win32 and Win64. In both cases, fwrite completes partially (no error with DMC runtime, "broken pipe" with MSVC runtime).

!!!There are attachements in the bugzilla issue that have not been copied over!!!

dfj1esp02 commented on 2014-10-27T15:07:46Z
I think, it applies to console only. By implementation, WriteFile checks if the file is console and calls WriteConsoleA, which can think in characters instead of bytes.

dfj1esp02 commented on 2014-10-27T12:02:54Z
(In reply to Vladimir Panteleev from comment #1)
> BTW, I've tried reducing data.txt, and I think it's no coincidence that its
> size is 1 past 16384 (0x4000).

You mean it works for smaller files? There's size limit for string written to console: https://connect.microsoft.com/VisualStudio/feedback/details/635230/

d.bugs commented on 2018-09-20T07:22:19Z
Minimal reproducible test cast:

import core.sys.windows.windows;
import std.stdio;

void main()
{
	SetConsoleOutputCP(65001);
	stderr.writeln("あ");
}

Which causes this exception:

std.exception.ErrnoException@C:\D\dmd2\windows\bin\..\..\src\phobos\std\stdio.d(2889): Enforcement failed (No error)
----------------
0x00402499
0x0040242E
0x00402328
0x00402263
0x00404275
0x0040410F
0x0040268F
0x760B343D in BaseThreadInitThunk
0x77039832 in RtlInitializeExceptionChain
0x77039805 in RtlInitializeExceptionChain

Normal writeln without stderr or using stdout works.

dfj1esp02 commented on 2014-10-27T11:54:24Z
Is "broken pipe" error reported by MSVC runtime itself? The file interface to console stream in utf-8 encoding is a little tricky: the program shouldn't check the number of written bytes (it's probably messed by transcoding).

brocolis commented on 2014-10-27T18:24:20Z
FYI there's a similar problem with the curl command line tool with chcp 65001.

chcp 65001
curl https://issues.dlang.org/attachment.cgi?id=1447
curl: (23) Failed writing body (3019 != 8000)

dlang-bugzilla commented on 2014-10-27T12:08:26Z
I guess sometimes fwrite will return a number different from the "count" parameter, even though it wrote everything without an error?

dlang-bugzilla commented on 2014-10-25T02:15:28Z
BTW, I've tried reducing data.txt, and I think it's no coincidence that its size is 1 past 16384 (0x4000).

dfj1esp02 commented on 2014-10-27T17:54:55Z
Indeed, docs for WriteConsole say it operates in terms of characters without definition of the character and indeed it reports the number of written characters, not bytes.

dlang-bugzilla commented on 2014-10-27T12:07:53Z
Yes.

The program I posted throws an exception because fwrite returns a number (bytes written) smaller than the size of the data. According to its documentation, this indicates a write error.

I've tried modifying std.stdio to retry incomplete fwrites with the remaining data segment, but although now no exception is thrown, the output is corrupted (some data is repeated).

bugzilla commented on 2021-03-24T20:16:21Z
*** Issue 14129 has been marked as a duplicate of this issue. ***


How do I calculate index inside *ngFor now?

Before switching to an infinite scroll, my ngfor used to look like:
*ngFor="let row of GridData; let index = index;"
I need index because some methods I want to call from the grid rows need to act on rows adjacent to the selected one - so i pass the index as a parameter in the click event. This is no longer possible with this component because index is relative to just the viewport items - not the whole list.

For reference, the index of the first element can be used with ngFor index to display correct item indices.
In ts:
startIndex:number;
vsChange(ev) {
this.startIndex = ev.startIndex;
}
In HTML:
 {{i+1+startIndex}}


cannot get this to work

Hello,
I have been unable to successfully download and run vcfkit and have exhausted all options. I have tried the various install methods and even tried changing the primer3.py script to include the path to the primer3_config but I keep getting the same result: Cannot find thermo path '/primer3_config/
Neither vcfkit nor primer3 is being installed with a primer3_config that is unless I just clone the entire repositories. which still didn't solve the : Cannot find thermo path '/primer3_config/ issue.
Any help?

Hello,
I have been unable to successfully download and run vcfkit and have exhausted all options. I have tried the various install methods and even tried changing the primer3.py script to include the path to the primer3_config but I keep getting the same result: Cannot find thermo path '/primer3_config/
Neither vcfkit nor primer3 is being installed with a primer3_config that is unless I just clone the entire repositories. which still didn't solve the : Cannot find thermo path '/primer3_config/ issue.
Any help?

My vcfkit was installed using conda, and I create the following folder to solve the same issue:
/PATH/TO/CONDA/ENV/lib/python3.7/site-packages/vcfkit/static/primer3_config/
However, after solve this, I still can't get right primer result. The script can run without error, but I got noting in the output.

Same problem. I installed primer3 through brew, ran vcfkit as the following: vk primer sanger --ref=genome.fa test.vcf.gz
I get no error and no output too.

Hello -
Have you tried to run VCF-kit using docker?
docker run -it andersenlab/vcf-kit vk primer

Unfortunately I am not always able to keep up with the changing dependencies that can break installation, but the docker image is static and should work for you.
Please try to use the Docker image and let me know if you run into any problems.

The following command worked for me:
mamba create -n vcf-kit   danielecook::vcf-kit=0.2.6   bwa==0.7.17   samtools==1.10   bcftools==1.10   blast==2.2.31   muscle==3.8.31   primer3==2.5.0


Support tags

Would be great to support adding tags to created alerts.

Note - from the swagger docs - https://github.com/logzio/public-api/blob/master/alerts/swagger.json - isn't in the example on the API page
"tags" : {
    "type" : "array",
    "items" : {
        "type" : "string"
     },
    "maxItems" : 10,
    "minItems" : 0
},

Done at https://github.com/logzio/logzio_terraform_provider/pull/43