remove overloads codemod

Codecov Report
Patch coverage: 96.95% and project coverage change: +0.32 :tada:

Comparison is base (c2af1ce) 90.42% compared to head (df65698) 90.74%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##            alpha    #5046      +/-   ##
==========================================
+ Coverage   90.42%   90.74%   +0.32%
==========================================
  Files         105      110       +5
  Lines        3791     3988     +197
  Branches      952     1000      +48
==========================================
+ Hits         3428     3619     +191
- Misses        330      336       +6
  Partials       33       33

Impacted Files
Coverage Δ

...es/codemods/src/v5/remove-overloads/utils/index.js
91.42% <91.42%> (ø)

packages/codemods/src/utils/index.js
97.56% <93.33%> (-0.95%)
:arrow_down:

...ads/transformers/filter-aware-usage-transformer.js
97.95% <97.95%> (ø)

...s/transformers/query-fn-aware-usage-transformer.js
98.68% <98.68%> (ø)

...demods/src/v5/remove-overloads/remove-overloads.js
100.00% <100.00%> (ø)

...c/v5/remove-overloads/utils/unknown-usage-error.js
100.00% <100.00%> (ø)

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

can you add documentation to the upgrade guide please:
https://tanstack.com/query/v5/docs/react/guides/migrating-to-v5

not sure if this is possible but it would be much easier to look at the test cases if we had input and output co-located for each situation. Something like:
useIsMutating(['foo', 'bar'])

will become
useIsMutating({
  mutationKey: ['foo', 'bar']
})

right now, I have to know that line 18 in the input file corresponds to lines 36-38 in the output file ...

can you add documentation to the upgrade guide please:
https://tanstack.com/query/v5/docs/react/guides/migrating-to-v5

Of course, that will be added in a separate commit.

not sure if this is possible but it would be much easier to look at the test cases if we had input and output co-located for each situation. Something like:

Unfortunately not, at least I haven't found a way yet. 😞 I will try to come up with something nicer but probably not in this PR.


Restructure org.bonej.utilities and org.bonej.wrapperPlugins

Go through the utility methods in both packages, and move them to appropriate places. Depending on how many are left in org.bonej.utilites, delete the package. So many utilities in the corresponding legacy package are now deprecated that likely there is not going to be many new utility classes.

The deciding factor on where utilities belong is whether they're general, or related to user interaction, e.g. popping open a certain kind of dialog.
Like said before if there aren't that many utilities that could be generally useful to others, then the utilities maven artefact could be dropped. Maintaining the project would be easier with less modules. Also some utilities could eventually find their way to scijava-common etc.

This issue needs more consideration & discussion before it is decided how to approach it or if it's an issue at all.
Originally BoneJ was divided into several different modules, because even though we always planned to migrate as much code as possible to imagej-ops and other core artefacts, we thought there was also going to BoneJ specific utilities and ops that wouldn't fit elsewhere. Having them in different modules would allow others to just depend on them without having to also download the wrapper plug-ins i.e. Commands. This structure was also thought to make migration easier, because components only depended on each other through Maven.
The real deciding question is: after most of BoneJ has been modernized, is there going to be enough code to warrant separate modules, and more complex maintenance? We were always aiming to eventually do away with the modules under Legacy, but will we need ops, utilities and wrapperPlugins in Modern? If there is going to be code too specific to be added to a pre-existing artefact in the ecosystem, will it still be valuable enough for other developers to offer it as a library module of its own?


Update ESP-IDF to v5.1.3

Hi.
It's a minor change but I thought it'd better if it gets updated to 5.1.3 before 5.2 is merged.
Last couple of weeks I got into quite a few issues because of the changes between 5.1.2 and 5.1.3. nothing breaking but some APIs have changed and I had to manually check them. The main idf documentation opens 5.1.3 now.
I did verify it with my projects. It seems fine with both esp32 and esp32s3

Thanks!


{{ csrf_token }} inside of sekizai tag invalidates cache on every page load

There is {{ csrf_token }} template variable inside of the {% addtoblock "js" %} block in cms/toolbar/toolbar_javascript.html.
For logged in admin users, this variable changes on every page load. Since this variable is inside of the sekizai tag, it generates new .js file in static/CACHE/js/on every page load.
This causes 2 major problems for me:

disk space quickly runs out on the server, as the CACHE folder fills up with gigabytes of cached js files.
page loads are slower since sekizai compresses and caches js files on every page load

Here's the snippet of code from cms/toolbar/toolbar_javascript.html.

{% load i18n l10n sekizai_tags admin_static cms_tags cms_static %}

{% addtoblock "css" %}<link rel="stylesheet" href="{% static_with_version "cms/css/cms.base.css" %}" />{% endaddtoblock %}

{% addtoblock "js" %}
<script src="{% static_with_version "cms/js/dist/bundle.toolbar.min.js" %}" type="text/javascript"></script>
{% language request.toolbar.toolbar_language %}
<script>
CMS._plugins = [];
// this is a global shared configuration
CMS.config = {
    'mode': {% if request.toolbar.edit_mode %}'draft'{% else %}'live'{% endif %},
    'auth': {% if user.is_authenticated %}true{% else %}false{% endif %},
    'debug': {% if debug %}true{% else %}false{% endif %},
    'csrf': '{{ csrf_token }}',
    'request': {
        'language': '{{ request.GET.language|escape|escapejs }}',
        'model': '{{ request.toolbar.get_object_model }}',
        'page_id': '{% if request.current_page.publisher_is_draft %}{{ request.current_page.pk }}{% else %}{{ request.current_page.publisher_public_id }}{% endif %}',
        'pk': '{{ request.toolbar.get_object_pk }}',
        'url': '{% language request.toolbar.language %}{% cms_admin_url "cms_page_resolve" %}{% endlanguage %}',
        'tree': '{% language request.toolbar.language %}{% cms_admin_url "cms_page_changelist" %}{% endlanguage %}'
    },
...

I suggest moving the CMS.config = {...} outside of the sekizai tag.

I have this problem as well. I assume you are using django-compressor along with sekizai?
It's sort of a hack, but I fixed it by overriding the toolbar_javascript.html file, and moving the js outside of the addtoblock.
I think a better fix would be to change CMS to use the CSRF token cookie rather than spitting it out in a template somewhere.

Hello,
The cms does not support putting its internal js in compressor.
We already minify the internal js files.
If you want to use compressor with sekizai, please use a different block to add your custom styles, so instead of js block, you can use custom-js block and compress that one.

I really don't think that's the correct solution. For compressor to work with sekizai, you actually HAVE to name the block either js or css. Or you have to hack the compressor processor for sekizai. For a normal sekizai/compressor setup, django-cms will break it.
I'm really not sure, but is there a reason djangoCMS can't use the csrf token cookie that django provides? For my projects, I always grab the token out of the cookie when I need it for ajax requests. Django actually talks about how to do there here: https://docs.djangoproject.com/en/2.0/ref/csrf/#ajax

It is not only about the csrf token but rather all the other dynamic js variables in that file which can't / shouldn't be compressed into a static file.
This is a limitation of sekizai / compressor integration

@czpython django-cms is using sekizai_tags, not the client code. See the first line of cms/toolbar/toolbar_javascript.html. it clearly loads sekizai  {% load i18n l10n sekizai_tags admin_static cms_tags cms_static %}
And the {% addtoblock "js" %} block used in toolbar_javascript.html is a sekizai tag.
There's nothing we can do on the client side to fix this. I agree with @jproffitt and vote for the ticket to be reopened.

@czpython you are right, there is more than just the csrftoken. But you’re basically saying that if you use Django-compressor with sekizai, then there’s nothing you can do. That’s not a very good solution. Right now, the only way to fix this is for me to override the toolbar_javascript.html template in every project, and rename the sekizai block to something else. That is especially painful if I have to upgrade CMS, and that template changes.
I’m not exactly sure what the solution is, but I think we need to think about some way to fix it. We could introduce a second sekizai block not named “JS”, or we could put the JavaScript in this file outside of a sekizai block altogether.

Another idea: perhaps make the name for the sekizai block a setting that users could change in order to put it in a different block?

We are thinking to bundle a compressor processor with sekizai which allows for custom namespaces instead of using css / js only.


Add documentation about requiring high port numbers for gunicorn

Bug description
If you want to run Vespene on a port different from 8000 (still behing Nginx) and change it in the 0_common.sh it doesn't work.
Steps to reproduce
Change GUNICORN_OPTS="--bind <IP_ADDRESS>:8000" to GUNICORN_OPTS="--bind <IP_ADDRESS>:1000"
Expected behavior
The app should work on the defined port.
Actual behavior
The app dies with not much (if any) useful info in the logs.
Any additional info

Hi @muki,
As it is intended to work, 0_common.sh is just the input to all the other scripts.
You would have to rerun 6_services.sh after changing the port.
When you say the app dies, I'm a bit curious how that happened because nothing would know about that change?
Did you mean you ran some of the other scripts?
Logs go into /var/log/vespene/ - the supervisor log probably contains any failures about starting the service.
As such, I'm inclined to thing this isn't a bug as written, but maybe something we need to clarify in the docs.
Am I reading that correctly?
Thanks!

Oh, yeah, so that's it.
Vespene runs as the 'vespene' user, so if you are just running it like that you need to be above port 3000.
https://stackoverflow.com/questions/10182798/why-are-ports-below-1024-privileged
If that is indeed it - and it seems to be - it seems like gunicorn is not throwing a good error about not being able to bind to a privileged port.
From the error messages here - https://stackoverflow.com/questions/40121221/gunicorn-not-starting-on-port-80-but-starting-on-port-8000 - that seems to be exactly it.
It seems the best answer here is to comment that it needs a port > 1024 in 0_common.sh
(You could of course technically run it as root, but that's something you should really want to avoid anyway)
I'll leave this open so I can document this better.
Thank you very much for the heads up!

Somehow I didn't know about the 1024 limit. Today I learned ...


added Manicaland State University of Applied Sciences @staff email addresses

@mashcom I have to close this request without merging, because you requested to add a non-existent domain to this repository: Host staff.ac.zw not found: 3(NXDOMAIN) Please ensure that you are familiar with this repository rules located at the bottom of this page and submit a new request with the correct information. Thank you for your understanding!


Canonical bbolt

What does this PR (Pull Request) do?

[x] This PR is related to - but does not fix - #4380 and #4835

This changes the import path of the bbolt library used by Grove to provide its cache from github.com/coreos/bbolt to go.etcd.io/bbolt, as that is the canonical import path as per go.etcd.io/bbolt#224. That makes switching to a Go-module based dependency system impossible. So this is necessary to move to that.
Which Traffic Control components are affected by this PR?

Grove

What is the best way to verify this PR?
Run all the tests, make sure Grove builds.
The following criteria are ALL met by this PR

[x] Tests are unnecessary
[x] Documentation is unnecessary
[x] An update to CHANGELOG.md is not necessary
[x] This PR includes any and all required license headers
[x] This PR does not include database migration
[x] This PR DOES NOT FIX A SERIOUS SECURITY VULNERABILITY

retest this please

Retest this please.

Weasel fails on this PR, it should have been rebased onto master so the checks could run.

Yeah, I see. I'm fixing it


Multi choice dropdopw choices in a section does not show all options and is cutoff

I created a multichoice dropdown inside a section, but the values are not all showing since it is cropped to fit inside the section.

Using streamsync 0.4 and firefox as the browser but same issue in chrome.

Additionally, I would like it better if the options for the multichoice used the KEYS to show as the labels and the values should be the data to return. This way you can sort the keys since it seems even if I give it a sorted dict, where the VALUES are sorted (since that is what you show) it still seems to revert to sorting on the keys itself.

I did a work-around by mocking the "Section" by using a HTML object, and dragging the select-component into it.

Helllo guys, I'm working on a potential fix to solve the issue. It just seems a overflow: hidden issue.

I would like it better if the options for the multichoice used the KEYS to show as the labels and the values should be the data to return. This way you can sort the keys since it seems even if I give it a sorted dict, where the VALUES are sorted (since that is what you show) it still seems to revert to sorting on the keys itself.

@thondeboer , Yes, the order of keys are not preserved.  I'm affraid we can't use the JSON value as HTML value since it's a breaking change...


NPM run watch doesn't work to JS projects without Node or DOM

I'm submitting a ...
[x ] bug report
[ ] feature request
[ ] question about the decisions made in the repository
[ ] question about how to use this project

Summary
When you select the option to create a pure javascript project without Node neither DOM and you run the initial command described in README (npm run watch) the build fails.

Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. StackOverflow, personal fork, etc.)

> run-s clean build:main && run-p "build:main -- -w" "test:unit -- --watch"

> trash build test

> tsc -p tsconfig.json

src/types/example.d.ts:24:33 - error TS2503: Cannot find namespace 'NodeJS'.

24   export interface Hash extends NodeJS.ReadWriteStream {
                                   ~~~~~~

src/types/example.d.ts:27:22 - error TS2580: Cannot find name 'Buffer'. Do you need to install type definitions for node? Try `npm i @types/node` and then add `node` to the types field in your tsconfig.

27       data: string | Buffer | DataView,
                        ~~~~~~

src/types/example.d.ts:30:15 - error TS2580: Cannot find name 'Buffer'. Do you need to install type definitions for node? Try `npm i @types/node` and then add `node` to the types field in your tsconfig.

30     digest(): Buffer;
                 ~~~~~~

Found 3 errors.

got these errors too.
 Javascript library
? 💬 Enter the package description: lll
? 🚄 Will this project use npm or yarn? npm
? 📚 Which global type definitions do you want to include? None — the library wo
n't use any globals or modules from Node.js or the DOM
? 🚀 More fun stuff: (Press <space> to select, <a> to toggle all, <i> to invert
selection)Enable tslint-immutable, Include .editorconfig, Include VS Code debugg
ing config, Include CircleCI config

But I'm seeing 2 tests passing when I run npm run watch. What settings are you selecting?

I see this same error when I run npm run doc.
My settings:
? 📦 Enter the new package name: my-package
? 🔨 What are you making? Javascript library
? 💬 Enter the package description: jjjj
? 🚄 Will this project use npm or yarn? npm
? 📚 Which global type definitions do you want to include? None — the library won't use any globals or modules from Node.js or the DOM
? 🚀 More fun stuff: Enable stricter type-checking, Include .editorconfig, Include VS Code debugging config, Include Travis CI config

npm test and npm run build are clean.
But npm run doc gives me
$ npm run doc

><EMAIL_ADDRESS>doc /Users/me/projects/my-project
> run-s doc:html && open-cli build/docs/index.html

><EMAIL_ADDRESS>doc:html /Users/me/projects/my-project
> typedoc src/ --exclude **/*.spec.ts --target ES6 --mode file --out build/docs

Using TypeScript 3.5.3 from /Users/me/projects/my-project/node_modules/@bitjson/typedoc/node_modules/typescript/lib
Error: /Users/me/projects/my-project/src/types/example.d.ts(23)
 Cannot find namespace 'NodeJS'.
Error: /Users/me/projects/my-project/src/types/example.d.ts(26)
 Cannot find name 'Buffer'. Do you need to install type definitions for node? Try `npm i @types/node` and then add `node` to the types field in your tsconfig.
Error: /Users/me/projects/my-project/src/types/example.d.ts(29)
 Cannot find name 'Buffer'. Do you need to install type definitions for node? Try `npm i @types/node` and then add `node` to the types field in your tsconfig.

Looks like this error is due to including example code in src\types\example.d.ts that relies on NodeJS so it fails when you run any command that calls build.
Workaround: comment out or delete the code in example.d.ts (or as mentioned above you can add node to your project but that's kinda silly because you just told the generator you didn't want it 😆).
Repro steps:

npx typescript-starter and choose JS library and None for global types
cd my-project
Fix ava issues (#234) with npm i<EMAIL_ADDRESS>and edit package.json to remove sources from ava section
npm run build

Really awesome app! I'm learning a lot by seeing what this can do. Thanks @bitjson!

Hey all – the latest release of typescript-starter should solve this issue, but please let me know if you're still having trouble. 👍


Simulated device testing

Several microcontrollers seem to have at least experimental support in QEMU. Could be useful to be able to run MicroFlo BDD test against that

Xtensa/ESP8266 http://www.esp8266.com/viewtopic.php?f=9&t=26&start=10
TI Tiva/Stellaris http://gnuarmeclipse.livius.net/wiki/How_to_use_QEMU
http://beckus.github.io/qemu_stm32/
https://github.com/turbosree/QEMU-AVR32
https://github.com/buserror/simavr

Should come after #13 and maybe after #9

One approach is to compile for host, and inject simulated I/O devices there. http://www.jonnor.com/2017/03/host-based-simulation-for-embedded-systems/


We are charged up!

Changed readme to better communicate our excitement!

no way

We are charged up! (work in progress)


Substraction with 3D Text 3RF

Describe the bug
TypeError: Cannot read properties of undefined (reading 'count')
I am getting an error when trying to subtract <3DText> element from Drei. It seems that the type of 3DText is invalid (is not textGeometry).
Minimal reproduction of issue
Put <3DText> inside  and see the console.
Additional context
I have used  from three js (I know it's deprecated but it's the only way I found) but for some reason the performance is very slow.

@marcofiletto please could you provide a worked example of this issue via Stackblitz or similar?

Hey! I have used simple CSG (without ts) to reproduce the error
the problem comes when try to put  from Drei as a subtraction
sandbox here: https://codesandbox.io/s/fervent-diffie-qr140c?file=/src/App.js:268-285

@marcofiletto your example doesn't use this library. The react-three-csg library you are using seems to use three-bvh-csg under the hood.


Add Singular Value Decomposition to cupy

Merge after #2412.
This PR aims to add the Singular Value Decomposition (SVD) to cupy.
Note: this PR is based on #1402.

Please rebase the master

I got
>>> cupy.linalg.svd(cupy.array([[1]], dtype=cupy.float32))
(array([[-4765.43359375]], dtype=float32), array([ 1.], dtype=float32), array([[ 1.]], dtype=float32))
>>> cupy.linalg.svd(cupy.array([[1]], dtype=cupy.float64))
(array([[ 0.]]), array([ 1.]), array([[ 1.]]))

while
>>> cupy.linalg.svd(cupy.array([[1,0],[0,1]], dtype=cupy.float32))
(array([[ 1.,  0.],
       [ 0.,  1.]], dtype=float32), array([ 1.,  1.], dtype=float32), array([[ 1., -0.],
       [ 0.,  1.]], dtype=float32))

is OK.

@toslunar Thanks for reporting the bug. It seems that this unexpected behavior arises from (s|d)gesvd in cusolver, but I have no idea whether it is better to deal with this problem by ours or not.

@unnonouno I fixed all the points you have mentioned. Could you check them?

I fixed the problem of copy=False.

OK, LGTM!


Operation timed out but not retrying

Describe the bug
When writing a parquet file with delta-rs using an ObjectStore filehandler, it will sometimes fail when the operation times out, which is fine. The only thing is it should retry which it's not doing.
OSError: Generic MicrosoftAzure error: Error after 0 retries in<PHONE_NUMBER>91s, max_retries:10, retry_timeout:180s, source:error sending request for url (https://<redacted>.blob.core.windows.net/<redacted>/<redacted>/<redacted>=<redacted>/<redacted>=202110/0-afa3cc5d-d0b7-4258-8baf-e84bca411da2-0.parquet?comp=block&blockid=ICAgICAgICAgICAgICAgICAgIDM%3D): operation timed out

To Reproduce

Expected behavior
Respect the retry amount?

Timeouts are currently only retried automatically on idempotent requests, i.e. reads

@tustvold could be useful to propagate that info in the error msg, that retry_count is ignored because it's a PUT request

Closed by #5609


Switch chokidar for watch to increase performance

Chokidar is much quicker than Watch. Watch still relies on fs.watchFile, which is less efficient. From the Node docs:

Note: fs.watch is more efficient than fs.watchFile and fs.unwatchFile. fs.watch should be used instead of fs.watchFile and fs.unwatchFile when possible.

Cool, thanks! I'll take a look at this ASAP.


Create.php

Build the create.php file so that it allows users to create new songs and places them in the database. Users should be prompted to try again if they attempt to rate a song that they have already rated (but different users can rate the same song).

Completed by Jackson


ClassCastException in Azure Synapse Analytics, Getting Started, 5-Analyze using a dedicated SQL pool

Azure throws an error when attempting to populate dbo.NYCTaxiTripSmall from NYCTripSmall.parquet (line 43).
`IF NOT EXISTS (SELECT * FROM sys.objects O JOIN sys.schemas S ON O.schema_id = S.schema_id WHERE O.NAME = 'NYCTaxiTripSmall' AND O.TYPE = 'U' AND S.NAME = 'dbo')
CREATE TABLE dbo.NYCTaxiTripSmall
(
[DateID] int,
[MedallionID] int,
[HackneyLicenseID] int,
[PickupTimeID] int,
[DropoffTimeID] int,
[PickupGeographyID] int,
[DropoffGeographyID] int,
[PickupLatitude] float,
[PickupLongitude] float,
[PickupLatLong] nvarchar(4000),
[DropoffLatitude] float,
[DropoffLongitude] float,
[DropoffLatLong] nvarchar(4000),
[PassengerCount] int,
[TripDurationSeconds] int,
[TripDistanceMiles] float,
[PaymentType] nvarchar(4000),
[FareAmount] numeric(19,4),
[SurchargeAmount] numeric(19,4),
[TaxAmount] numeric(19,4),
[TipAmount] numeric(19,4),
[TollsAmount] numeric(19,4),
[TotalAmount] numeric(19,4)
)
WITH
(
DISTRIBUTION = ROUND_ROBIN,
CLUSTERED COLUMNSTORE INDEX
-- HEAP
)
GO
COPY INTO dbo.NYCTaxiTripSmall
(DateID 1, MedallionID 2, HackneyLicenseID 3, PickupTimeID 4, DropoffTimeID 5,
PickupGeographyID 6, DropoffGeographyID 7, PickupLatitude 8, PickupLongitude 9,
PickupLatLong 10, DropoffLatitude 11, DropoffLongitude 12, DropoffLatLong 13,
PassengerCount 14, TripDurationSeconds 15, TripDistanceMiles 16, PaymentType 17,
FareAmount 18, SurchargeAmount 19, TaxAmount 20, TipAmount 21, TollsAmount 22,
TotalAmount 23)
FROM 'https://contosolake.dfs.core.windows.net/users/NYCTripSmall.parquet'
WITH
(
FILE_TYPE = 'PARQUET'
,MAXERRORS = 0
,IDENTITY_INSERT = 'OFF'
)`
Messages
Started executing query at Line 43
HdfsBridge::recordReaderFillBuffer - Unexpected error encountered filling record reader buffer: ClassCastException: class java.lang.Long cannot be cast to class java.lang.Integer (java.lang.Long and java.lang.Integer are in module java.base of loader 'bootstrap')
Total execution time: 00:00:04.926
Attempts to resolve this by dropping dbo.NYCTaxiTripSmall and recreating with all int data types changed to other datatypes like bigint, smallint, numeric, date, datetime have resulted in different errors.
HdfsBridge::recordReaderFillBuffer - Unexpected error encountered filling record reader buffer: ClassCastException: class parquet.io.api.Binary$ByteArraySliceBackedBinary cannot be cast to class java.lang.Long (parquet.io.api.Binary$ByteArraySliceBackedBinary is in unnamed module of loader 'app'; java.lang.Long is in module java.base of loader 'bootstrap')
HdfsBridge::recordReaderFillBuffer - Unexpected error encountered filling record reader buffer: HadoopUnexpectedException: Request failed due to an internal error that occurred during map-reduce job execution. Exception message = [Parquet column with primitive type [optional int64 VendorID] cannot be convertedto [DECIMAL] because parquet type is not annottated with decimal metadata].
HdfsBridge::recordReaderFillBuffer - Unexpected error encountered filling record reader buffer: ClassCastException: class java.lang.Long cannot be cast to class parquet.io.api.Binary (java.lang.Long is in module java.base of loader 'bootstrap'; parquet.io.api.Binary is in unnamed module of loader 'app')
What can I do to load the parquet data?
Thanks in advance.

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

ID: da9820f3-e186-4f52-7492-cb936823ea80
Version Independent ID: 2746ca67-f71e-85c6-f5b3-0f828bcbbc68
Content: Tutorial: Get started analyze data with dedicated SQL pools - Azure Synapse Analytics
Content Source: articles/synapse-analytics/get-started-analyze-sql-pool.md
Service: synapse-analytics
Sub-service: sql
GitHub Login: @saveenr
Microsoft Alias: saveenr

@PAWinTX
Thanks for your feedback! We will investigate and update as appropriate.

@PAWinTX
Thank you for your feedback!
I'd recommend working closer with our support team via an [Azure support request] (https://docs.microsoft.com/en-us/azure/azure-portal/supportability/how-to-create-azure-support-request). Or you can leverage our Q&A forum by posting your issue there so our community, and MVPs can further assist you in troubleshooting this issue or finding potential workarounds.
[Teams Q&A forum] (https://docs.microsoft.com/en-us/answers/topics/46488/office-teams-windows-itpro.html) for technical questions about the configuration and administration of Microsoft Teams on Windows.
[Microsoft Teams Community forum] (https://answers.microsoft.com/en-us/msteams/forum?sort=LastReplyDate&dir=Desc&tab=All&status=all&mod=&modAge=&advFil=&postedAfter=&postedBefore=&threadType=All&isFilterExpanded=false&page=1)
Thank you for your time and patience throughout this issue.


Next Federation 7 Support

The next major version of the next integration with module federation.
https://github.com/module-federation/universe/pull/726
This is a major rewrite, and removes the majority of dirty hacks that were needed in previous versions.
Lets discuss any upgrade issues here.
Common Issues
__webpack_modules__[moduleId]
This happens when webpack attempts to require a moduleId thats not yet loaded into the module cache at runtime.
Theres two causes ive encountered where this happens in v7

your using delegate modules and import or require additional files - delegates should only dynamic import import() other modules. This is to preserve the async boundary of the delegate module since i do not hoist its dependency tree into the webpack runtime, if youre requiring other files in the delegate, it needs to be async while webpack fetches the additional chunks

You are loading the current hosts own remoteEntry.js file. This scenario often happens with nested remotes like A->B->A
You should never load the hosts own remote, the host webpack.js runtime has its own remote interface {get,init} already embedded into it - hosts auto expose the remoteEntry interface as a side effect of booting. If you load the hosts own remote, a conflict arises around webpacks chunk loading global which gets overwritten and chunks end up pushed into the wrong runtime, causing the error.

Hydration Errors
Note on SEO: the HTML is still SSR'd so youre not losing SEO, its during hydration where react throws out the html for a moment then brings it back
next/dynamic converts async imports into sync imports or tracking events. Since federated imports cannot be tracked from a single build, next/dynamic receives a Promise which makes it return null, then set state with the component a few milliseconds later after hydration.
To workaround this, use React.lazy for federation. React.lazy causes react to suspend and wait for the resolve of the promise, dynamic just renders what it can instantly.

Hey @ScriptedAlchemy, any plan to support App router ?
thanks

It seems that version 7 has been shipped! Thanks a lot
As for App router support - it seems to be working partially, but fails with ChunkLoadError

Hi @nick4fake  @RexGalicie  @ScriptedAlchemy  any update on the above issue, I'm getting the hydration error after adding module federation in host app(mfe)


feat: build, deploy, support generator args, opts

Create a new project, build & deploy it to static Surge-hosted site. Support generator args:
appname, version + options. Fix --help option.

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


NGINX avatar serving

Why is the avatar filetype being forced to .jpg lol

Why is the avatar filetype being forced to .jpg lol

default_type "text/jpg";

Why is the avatar filetype being forced to .jpg lol

default_type "text/jpg";
default_type

That's not what I meant xd
I was talking about
        try_files $uri $uri.jpg /$default_avatar =404;

when it was running with guweb avatars were jpg

the thing is guweb accepts png jpeg jpg and gif

then you make it any different and see if you can

try_files $uri.png $uri.jpg $uri.jpeg $uri.gif /$default_avatar =404;
removed $uri because I'm pretty sure there can't be no file extension

current condition all file types that are accepted works any questions???

bruh mkfer stole half the shit i did

Cope


Adds new target that runs all the checks (to be used before creating a PR)

That's a good idea, thanks!


Compatibility Issues with Ethernet library

Basically I want to convert the Settimino (arduino Snap7 library) across to WiFi using the nodeMCU as an arduino. I have edited the Settimino library to #include ESP8266WiFi and changed the class from ethernet to WiFi, however I get compiler errors from Arduino IDE that I dont quite know how to fix.
Has anyone else had experience with porting an ethernet based library to wifi or how to fix these errors.
Thank you.
In file included from \Documents\Arduino\libraries\Settimino\Settimino.h:42:0,
from \Documents\Arduino\libraries\Settimino\Settimino.cpp:26:
\Local\Arduino15\packages\esp8266\hardware\esp8266\2.3.0\libraries\ESP8266WiFi\src/WiFiClient.h: In instantiation of 'size_t WiFiClient::write(T&, size_t) [with T = unsigned char [22]; size_t = unsigned int]':
\Documents\Arduino\libraries\Settimino\Settimino.cpp:426:41: required from here
\Local\Arduino15\packages\esp8266\hardware\esp8266\2.3.0\libraries\ESP8266WiFi\src/WiFiClient.h:123:36: error: request for member 'available' in 'source', which is of non-class type 'unsigned char [22]'
size_t left = source.available();
^
\Local\Arduino15\packages\esp8266\hardware\esp8266\2.3.0\libraries\ESP8266WiFi\src/WiFiClient.h:127:5: error: request for member 'read' in 'source', which is of non-class type 'unsigned char [22]'
source.read(buffer.get(), will_send);
^
\Local\Arduino15\packages\esp8266\hardware\esp8266\2.3.0\libraries\ESP8266WiFi\src/WiFiClient.h: In instantiation of 'size_t WiFiClient::write(T&, size_t) [with T = unsigned char [25]; size_t = unsigned int]':
\Documents\Arduino\libraries\Settimino\Settimino.cpp:446:39: required from here
\Local\Arduino15\packages\esp8266\hardware\esp8266\2.3.0\libraries\ESP8266WiFi\src/WiFiClient.h:123:36: error: request for member 'available' in 'source', which is of non-class type 'unsigned char [25]'
size_t left = source.available();
^
C:\Users\Geoff-Work\AppData\Local\Arduino15\packages\esp8266\hardware\esp8266\2.3.0\libraries\ESP8266WiFi\src/WiFiClient.h:127:5: error: request for member 'read' in 'source', which is of non-class type 'unsigned char [25]'
source.read(buffer.get(), will_send);
^
\Local\Arduino15\packages\esp8266\hardware\esp8266\2.3.0\libraries\ESP8266WiFi\src/WiFiClient.h: In instantiation of 'size_t WiFiClient::write(T&, size_t) [with T = unsigned char [35]; size_t = unsigned int]':
\Arduino\libraries\Settimino\Settimino.cpp:543:34: required from here
\Local\Arduino15\packages\esp8266\hardware\esp8266\2.3.0\libraries\ESP8266WiFi\src/WiFiClient.h:123:36: error: request for member 'available' in 'source', which is of non-class type 'unsigned char [35]'
size_t left = source.available();
^
\Local\Arduino15\packages\esp8266\hardware\esp8266\2.3.0\libraries\ESP8266WiFi\src/WiFiClient.h:127:5: error: request for member 'read' in 'source', which is of non-class type 'unsigned char [35]'
source.read(buffer.get(), will_send);
^
exit status 1
Error compiling. - See more at: http://www.esp8266.com/viewtopic.php?f=6&t=11109&e=0#sthash.kz6iclMu.dpuf

Just in case somebody runs into this issue. The following post solved this error.
http://www.esp8266.com/viewtopic.php?f=29&t=7525


missing S1B AUX_POEORB for 2021-02-06

This orbit file is not available on CloudFerro LTA.

Anomaly report is created on ESA CAMS:
https://cams.esa.int/browse/PDGSANOM-9443

This missing file has been retrieved and ingested.


Mobile App

Description
I'm trying to create a user friendly flow for my parents and friends which allows non tech-savvy users to simply browse & discover movies/series then download and stream at highest possible quality from one single app.
The vast majority of whom stream directly from a TV use a Android based system (like NVIDIA Shield, Android TV, Chromecast, Firestick, Plasma Bigscreen/Anbox, ..) which itself uses Playstore Apps. This implies that they have to first navigate to a web browser with a fiddly TV controller, type in a long server address and then do the deed in Jellyseerr which then redirects to another app. Such process isn't as seamless as just opening one app and having everything in one place (to minimize confusion with older people and avoid quality problems due video/audio codecs).
Desired Behavior
One idea how this could work is to integrate Jellyseerr directly into Jellyfin's start page. This way no new app would have to be created and it would be possible to utilize all codecs from Jellyfin's browser & mobile apps.
Additional Context
Any ideas/solutions to this problem are highly welcomed!
Code of Conduct

[X] I agree to follow Overseerr's Code of Conduct

Unfortunately the behavior described:

One idea how this could work is to integrate Jellyseerr directly into Jellyfin's start page. This way no new app would have to be created and it would be possible to utilize all codecs from Jellyfin's browser & mobile apps.

would not be possible. Jellyseerr can be linked and added to jellyfin using custom menu links. In addition, Jellyseerr is a PWA so you can install onto your mobile like any other native app (this also unlocks some cool features like web push notifications [native-like notifications]) and PWA is supported on desktops, macs, android, and iphones (iOS 16+).

would not be possible. Jellyseerr can be linked and added to jellyfin using custom menu links.

Nice find, thanks for sharing.
Having Jellyseerr as the front page would be the holy grail.. I tried asking this question in various forums and there's a lot of interest but sadly no real solutions.
On the other hand, there's one project which does this: Reiverr but sadly there's no TV/android app yet and some basic features like subtitles are still being worked on (but the lack of developers makes the progress slow)

In addition, Jellyseerr is a PWA so you can install onto your mobile

The problem are mainly android TVs/Nvidia Shield/.. since they don't support PWA :/
I don't have enough knowledge to know if tweaking Jellyfin app to make a different startpage is even feasible.
Another idea that comes to mind is using Kodi for client/app (with a custom theme to show Jellyserr instead) and Jelly as backend
Wish there was a official way of integrating Jelly and Jellyseerr into one beast 🙁

would not be possible. Jellyseerr can be linked and added to jellyfin using custom menu links.

Just a update:
Android TV clients can't actually see Web Links from the suggested workaround.
With Plex there's a baked in 'Discovery' page but I'd much rather use Jellyfin and Jellyseerr if there's a way to combine them into one app

Just a update:
Android TV clients can't actually see Web Links from the suggested workaround.
With Plex there's a baked in 'Discovery' page but I'd much rather use Jellyfin and Jellyseerr if there's a way to combine them into one app

That's because androidTV does not use the Web link. You still cannot combine jellyseerr and jellyfin unless you create a custom app.
Jellyseerr is in no way affiliated with jellyfin to be able to combine them.

I know, I'm fishing for more ideas or similar projects!
In the meantime, here's a feature request to display custom menu links in Jellyfin's native clients which would make it possible to have everything contained to one app. For future readers, consider dropping a upvote to make it more visible :)
https://features.jellyfin.org/posts/2435/custom-menu-links-in-jellyfins-native-apps-using-e-g-webview

Here's a update for future readers.
Jellyfin Server has a --webdir option which allows you to serve a different webroot other than the default Jellyfin Web interface. One way would be to tweak Jellyfin Web to display a iFrame of Jellyseerr which would make it visible in the website, Android (mobile) app and Desktop client (but you have to enable web version in settings). Currently only exception is Android TV which is completely native (and I'm not sure what's the status of iOS clients).
I'm still fishing for a better solution 🙂

@GrabbenD how is fishing going along? Any Updates on this topic?😃

@thorbolo Instead of building a native app from scratch (as discussed here), it's much simpler to just add support for D-pad navigation and display Jellyseerr as a WebView from pretty much anywhere:
https://github.com/Fallenbagel/jellyseerr/issues/601#issuecomment-2092917457

Ah this has been something I have wanted to do for ages, I gave up in the end. Would be very cool to have both of them in one place. Its no big deal having 2 seperate apps for me but some non-techy users who use my jellyfin server would certainly love it.


Set vim.o.background

It would be great if themes in this repo would also set vim.o.background, which is used by some plugins.
For example, https://github.com/nvim-tree/nvim-web-devicons, use this variable to select different colors for icons

I'm open to suggestions, but there's no good way to automate vim.o.background for the themes which are sourced from the base-16 repo. A heuristic could work, but it'd be brittle.

I like the idea of setting vim.o.background.
How about the following method of determining whether a color scheme has a light or dark background?

Get the background color (base00) of a color scheme.
Calculate the relative luminance of base00. Call it l1.
Calculate the relative luminance of the middle gray (i.e. rgb(50% 50% 50%) in the CSS syntax). Call it l2.
If l1 < l2, the color scheme is dark because it is darker than the middle gray, otherwise, it is light.

@mamekoro I'm open to PRs.


Fix Issue 6244 - Add powmod / modpow function to std.math

This implementation for powmod correctly handles overflow for unsigned types.
For at most 32-bit inputs, to avoid overflow during modulo sum or multiplication, we can cast the values to 64-bit types and do the operations.
For unsigned 64-bit types I implemented a special mulmod and addmod functionality to avoid overflows.
Tested with boost multiprecision powm function.

@ZombineDev
Thanks a lot for the comments!
Alex

Glad to help, @jercaianu. For more general advise on contributing to phobos, be sure to check https://wiki.dlang.org/Contributing_to_Phobos and of course ask around ;)


Per-cell line numbers cause issues with other extensions

Environment data

VS Code version: 1.57.1
Jupyter Extension version (available under the Extensions sidebar): 2021.6.999662501
Python Extension version (available under the Extensions sidebar): 2021.6.944021595
OS (Windows | Mac | Linux distro) and version: Windows 10 build 19042
Python and/or Anaconda version: Python 3.9.1, conda 4.10.3
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): conda
Jupyter server running: Local

Expected behaviour
When using the Error Lens extension (found here), error messages from the Problems pane are shown inline on the line where the error occurs.
Actual behaviour
The error messages are displayed on every open cell at the corresponding line:

A similar error occurs with the markdownlint extension flagging every cell that doesn't start with a title line as incorrect, e.g.,

when the user probably wants some markdown cells to denote subsections.
Opening the Problems pane and clicking on a listed problem highlights the correct cell and line, but it seems that extensions are unable to distinguish between cells.
Steps to reproduce:
[NOTE: Self-contained, minimal reproducing code samples are extremely helpful and will expedite addressing your issue]

Install the Error Lens extension
Create multiple cells, and give each a couple blank lines
Introduce a syntax error in one of the cells

If I'm understanding the Error Lens extension correctly, it merely amplifies diagnostics being reported for notebooks. We recently fixed a problem in the Python extension's language server support for notebooks which caused diagnostics to be reported for markdown cells. If you opt into the Python Daily Insiders channel (Cmd+Shift+P > Python: Switch to Daily Insiders Channel, wait for the insiders build to finish downloading and installing, and reload when prompted), does the problem persist?

FWIW the markdown lint extension is behaving correctly for me:

Actually the Error Lens extension is also behaving correctly for me:

I'm using the following extension versions in VS Code Insiders.

@circumspect any chance you can switch to Insiders and try again? I can't seem to repro the behavior you're describing 😕

Hi,
Thanks for taking a look! I was still running into the issue on stable VS Code even with the Insiders build of the Python extension, so I downloaded the latest portable version of VS Code Insiders. However, the issue persists there as well (I've attached my test notebook here).

These are the extensions on VS Code Insiders:

While the duplicated message doesn't always appear, if I type anything into the final cell, it reappears in the first python cell.

Thanks for the info. Looks like we're merging language server results for all cells 🤔 we have some outstanding issues with language server support for notebooks. This may be related to https://github.com/microsoft/vscode-jupyter/issues/5932

@DonJayamanne did you file the upstream bug?

Done https://github.com/microsoft/pylance-release/issues/2118


Fixing wrong URL and changed param to string

Fixed Wrong url on this method
change param to string to permit an string containing an customToken to be passed to this method.
This commit fixes this
method to work with updated to the last google Documentation see google docs reference

Thanks for taking the time to create this PR, I do appreciate it!
If you're willing to rename the commit message to "Update link
The verifyToken endpoint is not per se wrong, it uses a different base URL (see https://github.com/kreait/firebase-php/blob/master/src/Firebase/Factory.php#L241) and still works, but you're right, why not use the current endpoint.
As for the change of the Token parameter, I'd prefer not to have this: this makes it possible to call the method with new stdClass()ortrueand would cause side effects. I know, it's unlikely that someone would do that, especially since this is an internal class (see the@internal` annotation), only meant to be used from the Auth component, but still 😓
I don't mind the change of URL, but I'd prefer not to change the method argument.
I'd like to ask you to include only the changed URL in the PR and change the commit message to something like Update custom token exchange endpoint (without the extended commit message) - it's just that I like the "feeling" of a message like it a bit better.
If you're just interested in the changed URL and don't want to make the changes, please let me know with a short reply, I will then make it myself (and make sure that you still are named as the contributor in the commit history).
Thanks!

Merged with https://github.com/kreait/firebase-php/commit/4156d751241895ad2148db04466f6b4bd18af6ae, thanks!

thanks @jeromegamez maibe my english is not good enough.
So i thing is good to have an method that resolves an idToken from custom token directly, because sometimes this is usefull for an backend API that uses Firebase.
On my case this is necessary to do some integration tests.


Added Hyperlinks to Brand Assets and Contribution Guideline

Added Hyperlinks to Brand Assets and Contribution Guideline into the footer.

LGMT

LGMT


Agree on code style

The project doesn't really have an enforced style, i think it should!
Things that I find weird right now:

Float Tensor classes don't indent sometimes

Indent is currently 8 (!) spaces in most places

I propose that we adopt microsoft .net standard (https://docs.microsoft.com/en-us/dotnet/csharp/programming-guide/inside-a-program/coding-conventions) and enforce PRs follow this style.
This will make code more familiar for new contributors, and easier to read for existing contributors.

Using dotnet conventions makes sense to me


Supporting ERG mode, slopes and structured workouts

https://github.com/dvmarinoff/Flux has several notable features:

ERG mode support
Slope control
Structured workouts (Zwift zwo).
Follow FIT GPS routes

#470 will offer inclination control = slope


ppm Themes searching uses different methodology than implemented

Thanks in advance for your bug report!

[X] Have you reproduced issue in safe mode?
[X] Have you used the debugging guide to try to resolve the issue?
[X] Have you checked our FAQs to make sure your question isn't answered there?
[X] Have you checked to make sure your issue does not already exist?
[X] Have you checked you are on the latest release of Pulsar?

What happened?
The Packages & Themes buttons of Settings/Install doesn't have effect.
Pulsar version
1.109.0
Which OS does this happen on?
🪟 Windows
OS details
10 22H2
Which CPU architecture are you running this on?
64-bit(x86_64)
What steps are needed to reproduce this?

Go to Settings -> Install -> type any text e.g. Hydrogen
Check switch to Packages

Check switch to Themes

Observe the results are the same, so the switch doesn't have effect

Additional Information:
No response

Interesting
How in the world did we not know until now that the themes searching was busted in the editor? (Not blaming you, just for clarity)
We've had several people question things about custom themes and syntax highlighting before now 🤔
I wonder if people were finding the themes on the web and installing from there, installing them as github packages, or manually installing them 🤷♂
But additionally, how do we prevent this regression in the future? Do we have a few static theme entries we explicitly test for? (this could balloon costs)

@Spiker985
We probably never noticed because searching for packages still returns themes.
We only exclusively return themes on the themes endpoint, but the packages endpoint returns both. So you can still find themes by searching for theme, syntax or ui.
Otherwise it's a bit difficult to test. Since this is one of those cross-cutting concerns, where something can't really be decomposed into a test.
Since to test something like this, we need to make sure that settings-view does it properly, and that ppm does it properly, and that package-backend does it properly.
I really think the best way is just having proper documentation. Since if Atom's Server Side docs had been accurate, we would have never had this issue. They simple didn't document many endpoints, and many query parameters.
Potentially we could have PPM test hitting actual endpoints (Which it does already do a few) and one of those test for finding a specific package, and ensuring we don't find another particular package, but that isn't always the easiest. And like you said would increase costs.
Also just to mention, the website doesn't actually even support any kind of filtering, it only searches the packages endpoint, which returns everything, so it's just as if you searched only packages in the settings UI.
I feel like the best step to fixing this would be to reduce the amount of 'hops' involved in this logic, such as moving PPM into core, or having settings-view make requests like this one itself, rather than shelling out to PPM. But not sure if that's totally worth it, as it would still have to shell out for many other tasks, at least until we fully move PPM into core.

Yeah, while trying to debug this I was consistently confused by seeing zero network traffic (outside of user icons) inside the dev tools - I actually hadn't realized it was shelling out to ppm to begin without
Something I do worry about (not necessarily regarding this particular issue itself, but for Pulsar as a whole) is the security risks involved with the outdated-ness of the editor coupled with web requests
It's arguably something we're working towards overall, but still - makes me think sometimes. Specifically, it makes me wonder if they tried to keep as many "web" calls relegated to ppm, so as not to treat the editor (which is basically a mini web browser) as an actual web browser


Create canvas is not defined

When i run the program,It shows Create canvas is not defined? Any advice

theres a bunch of people with this issue. seems like an original file was removed and it broke the code.

Isnt there something we can do?

I believe devs updated the code yesterday morning, check for a new repo


Adds Espresso test for shared_preferences

Demonstrates an Espresso test running on Firebase Test Lab using the new espresso plugin.

Is there a reason this never landed? More test coverage seems like a good idea 🙂

Closing as obsolete.


Stop errorfactory from creating errors with nil causes

Q
A

Bug fix?
yes

New feature?
no

API breaks?
no

Deprecations?
no

Related tickets

License
Apache 2.0

What's in this PR?

Currently when the errorfactory is invoked with a nil error the errorfactory will produce an error that has nil error field. This PR will create a new underlying error when presented with a nil error.
Why?
Some consumers of the error assume that the error has been populated and when they access the field it triggers a panic. In particular logging code like this in nificluster_controller.go fails:
r.Log.Info("Nodes unreachable, may still be starting up", zap.String("reason", err.Error()))

Some error producers need to report errors even the underlying failure doesn't produce an error itself. For example in client.go:
clusterEntity, err := n.DescribeCluster()
if err != nil || clusterEntity == nil || clusterEntity.Cluster == nil {
	err = errorfactory.New(errorfactory.NodesUnreachable{}, err, fmt.Sprintf("could not connect to nifi nodes: %s", n.opts.NifiURI))
	return err
}

Additional Context
This failure made it more difficult to diagnose some underlying connectivity and configuration issues that I encountered when I moved on from an insecure cluster to an SSL-secured one.
Checklist

[x] Implementation tested
[x] Append changelog with changes

OK, I've updated the CHANGELOG. It seemed presumptuous to update the CHANGELOG before a maintainer accepted it.


[codemode] new missing codemode: fast-imports

fast-imports
Converts all @material-ui/core top level imports to direct path imports
-import Card, { CardActions, CardContent } from '@material-ui/core';
+import CardContent from '@material-ui/core/CardContent';
+import CardActions from '@material-ui/core/CardActions';
+import Card from '@material-ui/core/Card';

find src -name '*.js' -print | xargs jscodeshift -t<EMAIL_ADDRESS>
on https://material-ui.com/guides/minimizing-bundle-size/  it says that direct path imports are the fastest way. Current existing codemodes don't satisfy the need to rapidly use this fast import option.
screenshot from minimizing-bundle-size doc:

on https://material-ui.com/guides/minimizing-bundle-size/ it says that direct path imports are the fastest way.

@ilanbm The Babel plugin on https://material-ui.com/guides/minimizing-bundle-size/#option-2 is meant to provide equal bundling speed. If somebody care enough to apply a codemod, would it be better to use the Babel plugin?

TL;DR Optimizations should be handled at compile time not manually.
It makes more sense to put this into a babel plugin since this can give us the freedom to move things around internally and address that in the babel plugin. Any optimization with a codemod has to be reapplied if the codemod improves/changes. Same applies to new code that you introduce which also needs this codemod applied.
We had this stance on the /es build as well. Put path transforms into a single place instead of spreading it throughout the entire codebase.

@eps1lon the goal here is to have fast imports that don't need extra compilation / tree shaking.
So what you suggest contradict the goal of having a simple way to change imports to the fast way, doing it on compilation contradicts the goal

@oliviertassinari if option 2 has equal bundling speed, then why is it the direct import titled "fast"?

So what you suggest contradicts the goal of having a simple way to change imports to the fast way, doing it on compilation contradicts the goal

I'm pretty sure that by doing it during transpilation webpack won't pick up the first bundle. We're implementing naive tree-shaking at the transpiler level. The cost of rewriting an import is negligible.
But if you have some benchmarks comparing a babel plugin which rewrites the import with pre-rewritten imports then I would take a look again. Otherwise I'll reiterate that optimizations should almost never be done manually. That's what a compiler is for.

@ilanbm I would propose that we focus on the documentation. If the page makes you think that option 1 is the fastest, then we don't do a good job with the current wording. It's fast, as much as option 2 with the Babel plugin. I think that we can expand this sentence.

This option provides the best DX and UX.

Should we continue the documentation changes in this pull request?

@oliviertassinari I agree, if they are fast the same way than the docs weren't so clear
(and thanks for the chrome autofill commit :) )

@ilanbm I have tried to improve the wording with a new commit. Let us know if it's better of it you have a different suggesting :).

@oliviertassinari it's way better now, now it is clear. thanks!


Draft: ECO-697 - client library

This PR separates the client into client-lib and client, moving the logic and responsibility for creating requests to the casper-node into client-lib.

Will wait for #376 to merge so I can factor in the changes from that.

@Fraser999 I've changed my approach a bit in the hopes of both cleaning up the directory structure and abstracting the generic RPC-related options away from the functions themselves, while preserving the verbose behaviour as far as possible, though I did opt to remove the process::exit calls from within the library part itself.

@Fraser999 I've tried to address your points above in atomic commits, the only point I was less sure about was your first:

* I think we should move more of the logic out of the binary and into the library.  Specifically I'm thinking of the logic around creating a deploy.  Really the binary shouldn't be doing any heavy lifting - we should just parse args and pass these through to lib functions which actually construct the deploy

For this I took it to mean that anything we were doing around Deploys in general, but I could only see the supported CLTypes/CLValues and their related parsing as somewhat matching what you're saying there. Let me know if I've missed the mark here.

Adding @henrytill as a reviewer - please have a look at the client to ensure I've preserved #414 properly.

I've merged master and tested each of the subcommands, which lead to finding some more errors in parsing code and error handling. This has been resolved now and I'm hopeful this can merge soon. @Fraser999 thanks for all the effort on reviews.

After some test runs, there are still a couple of issues as per commented inline.
Also, it seems like --list-deploys now returns the whole block, which defeats the purpose of the subcommand. I think we should remove list_deploys from the lib since it's just calling get-block. Let's call get_block from the bin and display just the deploys' hashes from the response. Probably still worth displaying them as a JSON array to be consistent with our other outputs.

I ended up pulling list-deploys out of the lib, but keeping the translation struct ListDeploysResult for json-serialization/prettification.

I have a remaining issue here with simple_args_* arguments, but it also happens on master: (single quotes are stripped from the value portion of the name:cl_type='value', unless it's all wrapped in double-quotes (or appropriately escaped single-quotes). Not sure if that issue should block this, but wanted to raise it because I saw this during testing.
dan@zaphod ~/D/c/client (master)> cargo run -- put-deploy --secret-key ../resources/local/secret_keys/node-1.pem --chain-name casper-charlie-testnet-1 --payment-amount 1000 --session-arg name_01:bool='false' --session-path ../target/wasm32-unknown-unknown/release/standard_payment.wasm
   Compiling casper-types v0.6.0 (/home/dan/Development/casper-node/types)
   Compiling casper-execution-engine v0.7.0 (/home/dan/Development/casper-node/execution_engine)
   Compiling casper-node v1.5.0 (/home/dan/Development/casper-node/node)
   Compiling casper-client v1.5.0 (/home/dan/Development/casper-node/client)
    Finished dev [unoptimized + debuginfo] target(s) in 47.31s
     Running `/home/dan/Development/casper-node/target/debug/casper-client put-deploy --secret-key ../resources/local/secret_keys/node-1.pem --chain-name casper-charlie-testnet-1 --payment-amount 1000 --session-arg 'name_01:bool=false' --session-path ../target/wasm32-unknown-unknown/release/standard_payment.wasm`
thread 'main' panicked at 'value in simple arg should be surrounded by single quotes unless it's a null optional value', client/src/deploy/creation_common.rs:541:13
stack backtrace:...

@momipsl Unrelated to my comment, but --timestamp arguments are now parsing the rfc339 format (here)[https://docs.rs/humantime/2.0.1/humantime/fn.parse_rfc3339_weak.html].

bors r+


--retain and --refresh types

Is it possible to set --retain/--refresh (basically the types that takes a .type annotation) to something else than .days and .seconds? I tried setting --retain 2.hours and restarted the process, but the charts still shows data from yesterday. There were no exceptions when I changed the type though

2.hours should be valid (it should support all the duration types supported by scala's Duration). There has been some report of retain not working properly in some cases, maybe this is the case.

I set 3.days, but the charts still shows data over 3 days ago.
2015-01-29 9:32 GMT+08:00 Pierre Andrews<EMAIL_ADDRESS>
2.hours should be valid (it should support all the duration types
supported by scala's Duration). There has been some report of retain not
working properly in some cases, maybe this is the case.
—
Reply to this email directly or view it on GitHub
https://github.com/quantifind/KafkaOffsetMonitor/issues/38#issuecomment-71953021
.

--
马哲超
为了理想我宁愿忍受寂寞，饮尽那份孤独。

I can't really reproduce this. Do you see any exceptions in the logs? maybe it's due to #28

I honestly don't know, as I upgraded to <IP_ADDRESS> and now any sort of list seems to load forever. Sorry I can't be of more assistance

Thanks for the feedback, I'll let you know if I find a fix.
On Feb 16, 2015 7:24 AM, "Kasper Jacobsen"<EMAIL_ADDRESS>wrote:

I honestly don't know, as I upgraded to <IP_ADDRESS> and now any sort of list
seems to load forever. Sorry I can't be of more assistance
—
Reply to this email directly or view it on GitHub
https://github.com/quantifind/KafkaOffsetMonitor/issues/38#issuecomment-74524192
.

The app deletes old data at the same frequency as the retention interval.  So the amount of data retained will be between 1-2x the configured retention.


Question: Uninstall procedure

Just a couple of questions about the intended usage of this application:

Does dsefix.exe need to be executed again if the PC is restarted, or is the effect removed upon restart?
If the PC has been restarted, does 'dsefix.exe -e' have any effect?
If I had previously used dsefix.exe (but now I want to completely remove it), what is the procedure?

Dsefix is an exploit that uses the virtualbox driver so that it can load non-signed drivers digitally, it only works during the current session of the system, if restarting loses all effect, since the virtualbox driver is stopped.
if you want to remove it completely is just delete the executable, since the driver that is installed is a normal driver of the VBox, it has no harmful effect, but in windows 10 after a while, the patchguard detects this anomaly and causes BSDO.


apotheosis structure overriding cracker's wither storm mod

whenever this mod is played with cracker's wither storm mod on the latest version, apotheosis has a high chance to spawn a tower on the generated structure cracker's wither storm mod generated at x=0, z=0, effectively making said mod unprogressable.
is there a way to make apotheosis structures generate around x=0 z=0 but not actually on it [50 blocks away from 0, 0 is good enough]

a config to change the spawn rate of the tower would be nice if implemented aswell

The spawn frequency of the towers are controllable via datapack: https://github.com/Shadows-of-Fire/Apotheosis/tree/1.20/src/main/resources/data/apotheosis/worldgen/structure_set


NonMatchingSplitsSizesError when passing a dataset configuration parameter

As noticed in https://github.com/huggingface/datasets/pull/4125 when a dataset config class has a parameter that reduces the number of examples (e.g. named max_examples), then loading the dataset and passing max_examples raises NonMatchingSplitsSizesError.
This is because it will check for expected the number of examples of the config with the same name without taking into account the max_examples parameter. This can be fixed by checking the expected number of examples using the config id instead of name. Indeed the config id corresponds to the config name + an optional suffix that depends on the config parameters

Why not adding max_examples as part of the config name?

Yup it can also work, and maybe it's simpler this way. Opening a PR to fix bigbench instead of https://github.com/huggingface/datasets/pull/4463

Hi @lhoestq,
Thank you for taking a look at this issue, and proposing a solution.
Unfortunately, after trying the fix in #4465 I still see the same issue.
I think there is some subtlety where the config name gets overwritten somewhere when BUILDER_CONFIGS(link) is defined.
If I print out the self.config.name in the current version (with the fix in #4465), I see just the task name, but if I comment out BUILDER_CONFIGS, the num_shots and max_examples gets appended as was meant by #4465.
I haven't managed to track down where this happens, but I thought you might know?
(Another comment on your fix: the name variable is used to fetch the task from the bigbench API, so modifying it causes an error if it's actually called. This can easily be fixed by having config_name variable in addition to the task_name)


Bump carbon identity framework

Bump carbon identity framework 5.21.62 -> 5.21.63

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/2792385889

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/2792385889
Status: success


sails.io.js errors with l __getcookie net::ERR_CONNECTION_CLOSED trying to connect to my Sails API

I'm running Sails v0.11.0 for an API to my AWS relataional database.
I have installed sails.io.js via bower. sails.io.js installed at v0.11.5
I'm serving my Ionic application with sails.io.js installed from localhost:8100. It has the following script tags in index.html;
`<script src="lib/socket.io-client/socket.io.js"></script>
<script src="lib/sails.io.js/sails.io.js"></script>
<script>io.sails.url = 'https://localhost:1337/';</script>`

I serve my sails API from localhost:1337.
As soon as I start my Ionic server with the Sails server running I get an error in the console.
GET https://localhost:1337/__getcookie net::ERR_CONNECTION_CLOSED sails.io.js:205
If I remove the '` from my index.html it repeatedly tries to connect to localhost:8100 with the following error;
GET                                                                                socket.io.js:2919  http://localhost:8100/socket.io/?__sails_io_sdk_version=0.11.0&__sails_io_s…sails_io_sdk_language=javascript&EIO=3&transport=polling&t=1428417501863-3 404 (Not Found) 17.Request.createsocket.io.js:2842 Requestsocket.io.js:2773 17.XHR.request    socket.io.js:2803  17.XHR.doPollsocket.io.js:3192 18.Polling.pollsocket.io.js:3136 socket.io.js:2919 18.Polling.doOpensocket.io.js:2313  14.Transport.opensocket.io.js:1743  13.Socket.opensocket.io.js:1625  Socketsocket.io.js:1560  Socketsocket.io.js:299  3.Manager.open.Manager.connectsocket.io.js:564 (anonymous function) sails.io.js:136           Socket is trying to reconnect to Sails... _-|>_-  (attempt #4)

It has the following script tags in index.html;

Are these inside of <head> ?

Yes in the head tag.
I think the problem may be solved though. See my SO post. Matan suggested adding io.sails.useCORSRouteToGetCookie = false in the script tag in the <head>
It now connects with no errors. I was able to make a request with io.socket.get('/', function(){console.log(results)});
But @sgress454 mentions here "If you don't need socket support, just set io.sails.useCORSRouteToGetCookie to false."
Does that mean this somehow isn't a true socket connection?

Ok thanks for following up. I'm going to close since you've figured out the issue, but I'd still like to see what insight @sgress454 has to offer on this.


Sync "Not Found"

Visual Studio Code Version : 1.13.1
Code Settings Sync Version : 2.8.1
Operating System : macOS Sierra 10.12.4
Occurs On: Download and Upload
Proxy Enabled: No
GIst Id: Any Gist ID (New, Old)
When trying to Download the Gist settings, I just get the error message Not Found:

The Gist token is not saved to the settings. But even if I add it manually I still get that same error message.

what is the exception in the developer console. Paste image

If you are referring to the Debug console I get no output. Otherwise, I would like to know how to open the Developer console.

Help > Toggle Developer tool > Console

Perfect, thanks!
This is what I get

Try this :
https://github.com/shanalikhan/code-settings-sync/issues/177#issuecomment-269873967

It not work for me also, got the same error with @DonSanto
After I try #177 then It still get the same error.

No, that did not help att all.

Hello, same error here.  I did an uninstall all (as it is a bucket extension for several others) but it is still giving "Error Not Found".
I then tried to uninstall and reinstall Code Settings Sync extension but again I'm still getting  "Error Not Found". I'm uninstalling this extension until this bug is fixed as it's completely non-functional. @shanalikhan this is a super cool extension and I'm glad you've labored away at making it what it is. May you be victorious in solving this bug as well.

Same error "Not Found"
I also get the same output in my developer console, but it ends with
[Extension Host] Sync : Connected with user : 'gunnim'
on windows 10, vscode 1.13.1

I am also getting this error. I've look on other posts and it says "well you've likely got the wrong gist". I haven't as I copied it straight from the source after resetting it and yes I did wait a few hours till trying it again. I tried restting the extension and uninstalling it and reinstalling the extension. No effect I am still getting Error Not Found.
I am on Linux Ubuntu 17.04.
Thank you!

[Code Setting Sync]
Version : 2.8.2
[Visual Studio Code]
Version : 1.13.1
Commit : 379d2efb5539b09112c793d3d9a413017d736f89
[OS]
Edition Windows 10 Pro
Version 1703
OS build 15063.413
and
macOS Sierra
version 10.12.5
WFM
I check both Windows and Mac. I entered my PAToken to sync.gist by mistake.
Try below :
Ctrl + , (command + ,) -> setting.json -> "sync.gist": "" -> Shift + Alt + D
You need GIST ID for download, NOT TOKEN. Check your setting.json. You don't need change syncLocalSetting.json.
Thank you.

@mizutoki79 this right a way, It's working, thanks

Yeah user error on my part as well...

I'm getting the same error as @gunnim
https://jumpshare.com/v/TwPgvGrUw5McsBZgGd60
Here are my settings:
{     "sync.gist": "xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx", // Generated this a thousands times, tried activating everything on the access token, no luck.     "sync.lastUpload": "",     "sync.autoDownload": false,     "sync.autoUpload": false,     "sync.lastDownload": "",     "sync.forceDownload": false,     "sync.anonymousGist": false,     "sync.host": "",     "sync.pathPrefix": "",     "sync.quietSync": false,     "sync.askGistName": false }

@mizutoki79 solved my issue, I think this is not super clear on the plugin instructions. thanks for the help.

@mizutoki79 It works for me.
Please fix the bug ~ @shanalikhan

@mizutoki79 solved it also for me. I was adding everytime the token. Now used the GistID in the URL and I could download my settings.
Plugin instructions are not clear in this.

@mizutoki79 solved for me too.
Existing screenshot in instructions kind of imply its the Token that should be entered.  Should be edited with screenshot showing how to grab Gist ID from URL.  Great extension otherwise.

@DonSanto have you checked this solution of @mizutoki79 ?
https://github.com/shanalikhan/code-settings-sync/issues/343#issuecomment-312908504
I will add line to view the readme to check the uploaded gist by URL

I'm closing this issue.
Documentation updated.
This message is also changed to let user know the URL from which he can verify the gist

Yes, finally I've got this working. Thanks @mizutoki79.
The instructions still are not clear. Please remove the image where you highlight the token ID. it is really confusing. Thanks.

I'm still running into this error, and the gist id is definitely correct. I'm definitely using the gist id:
"sync.gist": "********",

then gist.github.com/******** redirects to my gist. But still getting the not found error. It is a secret gist, not sure if that makes a difference...


Include other languages and make it configureable

It would be nice if one could spell check different languages...

Found something interesting:
http://stackoverflow.com/questions/30704264/how-to-access-chrome-spell-check-suggestions-in-javascript
http://stackoverflow.com/questions/12040169/how-can-i-access-chromes-spell-check-dictionary
https://src.chromium.org/viewvc/chrome/trunk/deps/third_party/hunspell_dictionaries/

In theory, all you'd need to do would be to download a different dictionary.

@jackycute I'm wondering if Chromium's dictionaries would be better than the ones used currently. I've noticed there are a number of words, abbreviations, etc. that are missing.

Yeah, I mean I might to do so.
The only thing I'm not satisfied here is that GitHub and HackMD are marked as error now.
But this still a great repo, thank you @WesCossick !

I am looking to use your library in my own project, which is designed to be mulitlanguage.
I only have 2 languages now(english and french), but it can be extended.
I want to put this project as a dependency (bower or npm).
I think that you can make this project configureable to use others dictionnaries like these ones: https://github.com/titoBouzout/Dictionaries or any others, so you don't need to maintain all the dictionnaries of the world, and we can use what we want (as dependencies).
It's a very good idea !

I'd vote for some standardized way to add custom words to the otherwise-standard dictionary it downloads. Like
"add_words": ["GitHub", "blahblahblah"]

Note to self... look into https://github.com/titoBouzout/Dictionaries

@chriscoyier definitely agree with you here. Adding an option to pass an array or "ignored words" or custom words would be the deal-breaker.

I'd also love to see support for other languages besides english added to codemirror-spell-checker. Cheers!

All the Popular Browsers can handel spell check so why not use this option?
http://caniuse.com/#feat=spellcheck-attribute
it looks like extremely easy to implement:
https://stackoverflow.com/questions/21306199/setting-spellcheck-language-in-google-chrome
or im wrong?

Never gonna happen. PR https://github.com/sparksuite/codemirror-spell-checker/pull/39 allowing this change has been sitting around unmerged for 6+ years.


Maven dependency issue

I suspect that this issue can be closed, see https://twitter.com/jatin1anand/status/729652955289358336


Add support for 128 bit external integers

Currently it only goes up to 64 bits, but Rust natively supports up to 128 bits, so Poem should probably support it too.

Disregard, turns out serde doesn't support this.


docs: fix typo in code snippet

https://github.com/lukso-network/docs/pull/427

Codecov Report

Merging #169 (974e5a1) into develop (b1a7814) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #169   +/-   ##
========================================
  Coverage    20.00%   20.00%
========================================
  Files           16       16
  Lines         1015     1015
  Branches       344      344
========================================
  Hits           203      203
  Misses         790      790
  Partials        22       22

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Package.getStylesheetsPath is deprecated.

Store package style sheets in the styles/ directory instead of stylesheets/ in the es-navigation package
Package.getStylesheetsPath (/Applications/Atom.app/Contents/Resources/app.asar/src/package.js:460:9)
Package.getStylesheetPaths (/Applications/Atom.app/Contents/Resources/app.asar/src/package.js:471:32)
Package.loadStylesheets (/Applications/Atom.app/Contents/Resources/app.asar/src/package.js:453:38)
<unknown> (/Applications/Atom.app/Contents/Resources/app.asar/src/package.js:185:19)
Package.measure (/Applications/Atom.app/Contents/Resources/app.asar/src/package.js:165:15)
Package.load (/Applications/Atom.app/Contents/Resources/app.asar/src/package.js:179:12)

Thanks, this project is no longer actively maintained unfortunately. If you're interested in helping out and becoming a maintainer you are more than welcome.


[Bug] Broken Links

Description
Broken Links
I found 7 broken links by using a broken link checker

1
https://www.cssportal.com/ui-generators.php
url src 404
2
https://uiverse.io/all
url src 404
3
https://owasp.org/www-project-appsec/
url src 404
4
https://www.sans.org/cyber-security-skills-roadmap/defensive-security
url src 404
5
https://cuckoosandbox.org/
url src timeout
6
https://www.fireeye.com/services/freeware/flare.html
url src 530
7
https://www.mile2.com/web-application-penetration-tester-cwapt.html
url src 404
Screenshots
No response
Any additional information?
No response
What browser are you seeing the problem on?
No response
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I am willing to work on this issue (optional)

@JuanPabloDiaz do you like work on this?

Absolutely, I can help.
Should I look for the correct link? And if there is any just delete it or what should I do @rupali-codes ?

@JuanPabloDiaz do you like work on this?

@JuanPabloDiaz has multiple issues open already according to the bot, so I think it's best for them to focus on one for now.

@JuanPabloDiaz do you like work on this?

@JuanPabloDiaz has multiple issues open already according to the bot, so I think it's best for them to focus on one for now.

Hey @CBID2, thanks for your comment. I'm happy to work on this issue, but I don't mind if someone else wants to take it. I've been creating issues, but that does not mean that I want to hold them for myself. anyone can claim them.
Please feel free to grab any of them.
That's the spirit of open source. Collaboration!

Hey @JuanPabloDiaz  I would love to work on this issue.

Hey @JuanPabloDiaz I would love to work on this issue.

@JuanPabloDiaz
Thanks for the collaborative spirit. That is the spirit that helps in the longer run.
@Muhammad-Owais-Warsi
I have assigned you the issue. Please proceed with it, and don't hesitate to tag any of the maintainers if you require any assistance.

@JuanPabloDiaz do you like work on this?

@JuanPabloDiaz has multiple issues open already according to the bot, so I think it's best for them to focus on one for now.

Hey @CBID2, thanks for your comment. I'm happy to work on this issue, but I don't mind if someone else wants to take it. I've been creating issues, but that does not mean that I want to hold them for myself. anyone can claim them.
Please feel free to grab any of them.
That's the spirit of open source. Collaboration!

Ahh I see. That's the spirit! 😊

Hey @JuanPabloDiaz I would love to work on this issue.

@JuanPabloDiaz Thanks for the collaborative spirit. That is the spirit that helps in the longer run.
@Muhammad-Owais-Warsi I have assigned you the issue. Please proceed with it, and don't hesitate to tag any of the maintainers if you require any assistance.

Sure 😄


Support swagger

How about supporting swagger like grape-swagger? It's useful.
https://github.com/tim-vandecasteele/grape-swagger

Thank you.
This feature has been added to todo-list, And I'll give priority to it.

I start a standalone maru_swagger.
Let's discuss this under the new repo!


Update issue templates

A issue template standard is needed.  Ideally we should have a best practices for issues that mirrors the RFC. For example, we should adopt the modal language commonly used in technical documentation https://www.rfc-editor.org/rfc/rfc2119.
This is a suggested starting point, but very open to changes/additions, as it is very bare bones.

adding RFC/BIP templates in this PR: https://github.com/BeanstalkFarms/Beanstalk/pull/163


Better support for ARM / Hybrid architecture clusters

Following the installation instructions, the genie-plugin pods are in a crashloop when scheduled on an arm-based node. The binary doesn't appear to be built for ARM.
Are there plans to deliver an arm-architecture build of the cni-genie image?

@marccampbell , thanks for bringing this up. We definitely like to have cni-genie image built for ARM as well. But right now, we dont have it. Any support here will be beneficial

@sushanthakumar any update on cni-genie image built for ARM ?


ignore 'unknown' state of stalled resources

This would be my idea of fixing issue #715.

Superseded by #722


TestFoundation: limit test_arrayUsedAsCFArrayInvokesArrayMethods

This test uses CoreFoundation methods.  However, CoreFoundation is only
available on Darwin and Linux.  Limit the test to those environments
specifically.

@swift-ci please test


Exclude packages

Add a list of packages to be excluded both on outdated and upgradable lists
./bin/climb outdated --exclude=doctrine/cache,willdurand/negotiation

Covers #45

Thanks!

I only considered this option for packages checking, but wouldn't it be usefull on update command?

No, I don't think so. The update command is more of a bulk update.

:+1:


Added support for some immutable collections

Description
Added support for ImmutableArray, ImmutableList, ImmutableDicrionary
Closed issues

Checklist

[x] Reviewed the contribution guidelines
[ ] Linked the issue(s) the PR closes
[x] Implemented automated tests and checked coverage
[x] Provided inline documentation comments for new public API
[ ] Ran the full solution build and validation locally

Hi,
Could you please make review and release new version with this change @aivascu  ?
Thank you

Sorry, wrong branch. closed


Round out the untyped api s

Objective
Bevy uses custom Ptr types so the rust borrow checker can help ensure lifetimes are correct, even when types aren't known. However, these types don't benefit from the automatic lifetime coercion regular rust references enjoy
Solution
Add a couple methods to Ptr, PtrMut, and MutUntyped to allow for easy usage of these types in more complex scenarios.
Changelog

Added as_mut and as_ref methods to MutUntyped.
Added shrink and as_ref methods to PtrMut.

Marking as breaking change as into_inner now trigger change detection. While I agree with the change, it needs to be documented in the changelog and ideally migration guide

bors r+


Spevacus: Watch fastcustomboxes.com

Spevacus requests the watch of the watch_keyword fastcustomboxes\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

fastcustomboxes\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Machavity in Charcoal HQ


CSP Support

I'm a strong advocate of CSP. I think it'd a great idea to implement it.

I took a quick look at this last night and discovered the code uses quite a bit of unsafe eval(). This needs to be done differently as it allows for xss attacks.

@dav-is are you worries about ‘eval require dev.server’? cause that is nothing to worry about its just a patch for prod server to not inlice dev.server.js in build.

No I think it’s something one of the dependencies does on build. If you search “eval” inside the bundle.js, you will see what I mean. There’s several lines of eval at the very end.

An explaination of why eval shouldn’t be used: https://stackoverflow.com/questions/37155270/content-security-policy-csp-safe-usage-of-unsafe-eval

Actually, I think what I said only happens during development mode... a bit inconvenient for testing the headers

I can't seem to get a production build to work with: https://www.npmjs.com/package/express-csp-header
It throws an error no matter what I try...
ERROR in server.js from UglifyJs
    Unexpected token: name (cspString) [server.js:56441,5]

Advise against getting too far down the InfoSec here's how you should do your security hole with boilerplates. And I always say the same for auth as both are very opinionated things and adding security features may give individuals a false sense they're, ahem, covered when it comes to their node deps.
Adding Snyk should be a good enough way to provide a baseline of security for the app itself. And knowledge of and implementation of rel=noopener in case of Link component with target attribute would be something worth sliding into the generator if it makes sense.
Sidebar: Snyk has caught a literal boatload of DoS and other kinds of easy attack vectors in my RN boilerplate.
Do like where @dav-is head is at. Might be worth a call out in the readme or Wiki to help educate consumers.

Never thought of adding rel=noopener to the Link component. Nice thinking

Ok. I have added support for CSP, but leaving the control in hands of user/developer and it might vary according to env where the app is running:
Editing src/config/config.js as below will do the job
{
  // ... other config params
  seo: {
    // ...
    meta: [
      // ....
      {
        "httpEquiv": "Content-Security-Policy",
        "content": "default-src 'self'; img-src https://*; child-src 'none';"
      }
    ]
  }
  // ... other config params
}

Do you provide secure defaults? Have you tested the config? In development mode, unsave-eval is used so it may cause errors.


Customizing validation on specific fields or form

I tried to disable validation of a form or it's specific fields. but it seems that django-bootstrap5 forces validation for all bound forms: https://github.com/zostera/django-bootstrap5/blob/fa61c349d4c02db617e7b18e53c23898fb0d7cc3/src/django_bootstrap5/renderers.py#L426

I tried to remove 'is-valid' from my form object, but the validation css class is added before rendering.
I can use server_side_validation in django-bootstrap5 settings  but it's global and I don't want to stop validating other forms.

I suggest adding a new parameter to template tags responsible for rendering forms and fields. E.g. {% bootstrap_form form server_side_validation=False %}.

I found myself with the same problem. However it seemed easy to implement, I opened a PR with this feature.


"docker ps" reports deleted image name/tag

When an image has been "renamed" (e.g. give it a new tag then remove the old tag), "docker ps" reports the old image tag even though such tag does not exist any more.
I have reproduced this on 1.6.2 release and 1.7-dev but not 1.5.0 or earlier.
e.g.
# ./docker version
Client version: 1.7.0-dev
Client API version: 1.19
Go version (client): go1.4.2
Git commit (client): 1f22676
OS/Arch (client): linux/amd64
Server version: 1.7.0-dev
Server API version: 1.19
Go version (server): go1.4.2
Git commit (server): 1f22676
OS/Arch (server): linux/amd64
# ./docker run -d nginx
9728a4af7ed370106dddd36e843315ca057efa95782dc41c90b0c0969530d0f3
# ./docker ps
CONTAINER ID        IMAGE               COMMAND                CREATED             STATUS              PORTS               NAMES
9728a4af7ed3        nginx               "nginx -g 'daemon of   7 seconds ago       Up 6 seconds                            hopeful_blackwell
# ./docker images
REPOSITORY          TAG                 IMAGE ID            CREATED             VIRTUAL SIZE
nginx               latest              a785ba7493fd        5 days ago          132.8 MB
# ./docker tag a785ba7493fd foo
# ./docker rmi nginx
Untagged: nginx:latest
# ./docker images
REPOSITORY          TAG                 IMAGE ID            CREATED             VIRTUAL SIZE
foo                 latest              a785ba7493fd        5 days ago          132.8 MB
# ./docker ps
CONTAINER ID        IMAGE               COMMAND                CREATED             STATUS              PORTS               NAMES
9728a4af7ed3        nginx               "nginx -g 'daemon of   32 seconds ago      Up 31 seconds                           hopeful_blackwell

In 1.5.0 or earlier, "docker ps" will report image as "foo:latest" as expected.

Hi!
Please read this important information about creating issues.
If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
This is an automated, informational response.
Thank you.
For more information about reporting issues, see https://github.com/docker/docker/blob/master/CONTRIBUTING.md#reporting-other-issues

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
uname -a:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

# ./docker version
Client version: 1.7.0-dev
Client API version: 1.19
Go version (client): go1.4.2
Git commit (client): 1f22676
OS/Arch (client): linux/amd64
Server version: 1.7.0-dev
Server API version: 1.19
Go version (server): go1.4.2
Git commit (server): 1f22676
OS/Arch (server): linux/amd64
# ./docker info
Containers: 1
Images: 12
Storage Driver: btrfs
Execution Driver: native-0.2
Logging Driver: json-file
Kernel Version: 3.18.13-1-ec2-lts
Operating System: Arch Linux
CPUs: 2
Total Memory: 7.306 GiB
Name: ip-172-31-20-7
ID: ZQ34:FFT4:TI2Q:MJIM:UHDI:46H3:WOC4:XIDB:VX2A:OAVI:WI4C:YIFS
# uname -a
Linux ip-172-31-20-7 3.18.13-1-ec2-lts #1 SMP Wed May 20 20:00:18 PDT 2015 x86_64 GNU/Linux

This behavior was changed from about 1.5.+ at https://github.com/docker/docker/blob/v1.5.0/daemon/list.go#L126
From 1.6.+ and above now we get Image from container.Config.Image
I'm not really sure if this is a bug/regression cause I don't know the history tho
ping @tiborvass @duglin

ok found
https://github.com/docker/docker/commit/e45deceb462351f841aa077819fdef3166381c50
and the associated issue https://github.com/docker/docker/issues/10599
So not a bug nor nothing I think

@softboysxp to sum up, docker ps isn't going to change the Image name the container was started with

IMO this is a bug since the name being shown might no longer even exist, which means we're showing bad/useless data.
I understand why we continue to show the name used when the container was created, but if that name is no longer correct (either because its been removed, or worse, points to some other image) then we're doing the user a disservice.
While this will probably be a breaking change, I think we should change it so that we ALWAYS show the image ID since that will never change and, I think, we won't allow it to be deleted while the container is still using it.
ping @tiborvass @LK4D4

agree to have ImageID instead

This is a dup... of some other issue... in the pile of issues.

might be this one: #12595

I'm closing this as a dup of https://github.com/docker/docker/issues/12595, please continue the discussion over there.
Thanks for reporting @softboysxp !


Feature/dockerfile best practice

Description
I have made a couple of changes to the repositories Dockerfile, so that it follows the standard-best-practice recommendations.

Pin package versions
No need to run apt-clean in official debian image

Dockerfile lint

Version pinning forces the build to retrieve a particular version regardless of what’s in the cache. This technique can also reduce failures due to unanticipated changes in required packages.

Dockerfile best practice

Official Debian and Ubuntu images automatically run apt-get clean, so explicit invocation is not required.

Type of change

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

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[X] Image can build through make
[X] ldap2pg still works as expected

Checklist:

[X] I have performed a self-review of my own code
[X] My changes generate no new warnings

Codecov Report

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

@@          Coverage Diff          @@
##           master   #280   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          12     12
  Lines        1767   1767
=====================================
  Hits         1767   1767

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 bad48c4...cc049ed. Read the comment docs.

Hi @lokenorlinjohannessen, thanks for your contribution. Can you tell me what problem did you encounter ?
I guess that release process should be updated. That would be nice if make release updates Dockerfile, just like Changelog.

This PR allows to enable build cache on Docker hub automated build. I'll take care of enabling this once it's merged.

Hi @lokenorlinjohannessen, thanks for your contribution. Can you tell me what problem did you encounter ?
I guess that release process should be updated. That would be nice if make release updates Dockerfile, just like Changelog.

I did not encounter any problems, but I am looking into using ldap2pg in some of our systems, and we follow best-practice in regards to version pinning.

Besides that, I also noticed that the docs also just refer to non-pinned packages. It might be beneficial to pin all dependencies for all package managers.

Pinning deps is a maintainance burden. I don't want issue because one yum install downgrade a dependency. Feel free to write internal documentation with pinned dependency matching your own company requirements.

And you only tag your Docker image as latest on Dockerhub
The automated build should also tag images based on the release version.

Thanks for pointing this. I added automatic tag build in Docker Hub. I'm waiting for next build. If your company requires you to pin docker image, you could still use SHA.

@lokenorlinjohannessen I think this PR can be closed :

Hi @lokenorlinjohannessen , I think we can close this PR :

Docker image is now pinned
Docker image now pin ldap2pg on versionned image (not latest)
I cherry-picked your commit for apt-clean golf.

I don't want to pin other debian packages. I'm ok if you suggest so version boundary such as python3<3.8, not more precise.
What do you think of this ?

If we need to burst docker cache. I would go for python3>=3.7.2, updated as needed. This way, newer releases of ldap2pg gets newer packages versions.
How would you update previous image of ldap2pg (say 4.12) with latest python from Debian ?

Closing. Please open an issue or a new PR if you have updates on this topic.


Update raylib-lua and raylib-wren binding versions

Those bindings got updated recently.

@TSnake41 Nice! Thanks for the update!


Why bower not found this package

Bower doesnt found this package. I cant use another package's system

ok.

you added to bower but not all versions
Bower said
available versions in https://github.com/NightCatSama/vue-slider-component.git: 2.5.9, 2.3.5, 2.2.9, 2.2.6, 2.2.5, 2.2.4, 2.2.2, 2.2.1, 2.2.0, 2.1.9, 2.1.8, 2.1.7, 2.1.6, 2.1.5, 2.1.4
But i want 2.7.4
Is it real?

Sorry, I haven't used bower, I'm just a simple bower register, if you can, pull the project down and upload it yourself.

Maybe it wil be interested.
i finded last commit in intereting version here and specify in my bower.json
"vue-slider-component": "https://github.com/NightCatSama/vue-slider-component.git#581ebad28bfd42d384af9ecae2227d391fd9deda"
And i got 2.7.5 version


iOS Voice control cannot delete line

import 'package:flutter/cupertino.dart';
import 'package:flutter/material.dart';
void main() {
  runApp(
    MaterialApp(
      home: Scaffold(
        appBar: AppBar(
          title: Text('Expansion Tile Splash'),
          centerTitle: true,
        ),
        body: TextField(),
      ),
    ),
  );
}

in iOS, turn on voice control
tap text field to start editing
speak "type how are you"
speak "delete line"

expect: the entire line is deleted
actual: only the word "you" was deleted

speaking
delete word
delete paragraph
delete character
delete document

seem to work fine.

I want to work on this issue

Hi @rajrawat37 I already have a fix for this issue.
If you are interested in iOS voice access, there is another similar issue https://github.com/flutter/flutter/issues/77614.


Enhance hash exchange

When there is only 1 receiver, skip the hashing
Do not pre-allocate the array because the rows will be re-distributed

Codecov Report

Merging #11217 (b108313) into master (0598955) will increase coverage by 0.00%.
Report is 1 commits behind head on master.
The diff coverage is 0.00%.

@@            Coverage Diff            @@
##           master   #11217     +/-   ##
=========================================
  Coverage    0.11%    0.11%
=========================================
  Files        2227     2172     -55
  Lines      119594   117045   -2549
  Branches    18099    17788    -311
=========================================
  Hits          137      137
+ Misses     119437   116888   -2549
  Partials       20       20

Flag
Coverage Δ

integration1temurin11
?

integration1temurin17
?

integration1temurin20
?

integration2temurin11
?

integration2temurin20
?

unittests1temurin11
?

unittests1temurin17
?

unittests1temurin20
?

unittests2temurin11
?

unittests2temurin17
0.11% <0.00%> (ø)

unittests2temurin20
?

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

Files Changed
Coverage Δ

.../query/runtime/operator/exchange/HashExchange.java
0.00% <0.00%> (ø)

... and 57 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


499 Add note about ipv6 addresses in the docs

fix #499

@v0d1ch Note that this was not passing because of the flaky test fixed in #493. We usually would try to have a green CI before merging and also make PRs fast-forward. These are obviously unrelated changes so this is fine, but I though you should know going forward :)