Site Utilities Library

Creating Salt Site Utilities Library, equivalent to Salt Spec Utilities: https://www.figma.com/file/i8kSFpGbHIWXnv6LFvWKdU/Salt-Spec-Utilities?type=design&mode=design&t=NfBcCGpE9XLZ5AKi-0
The Site Utilities library are the components used to create pages with spec documentation which are consistent in appearance and layout for Salt site.

🟠30 June - build components for Site utilities
🟠29/30 June - Analysing the built site and the design in Figma files (to define the spec of components)

I'm a little unclear (and curious) about what the goal of this issue is.

Is the intent to create a Figma library of site components (e.g. page header, footer, example containers, image switcher, etc.)?
Or, is this a customized version of the spec utilities?

If it's the former, then can I ask that you liaise with me? I can explain what components we have implemented in code, since it would make sense to align their Figma counterparts. Also, @shey-v's original Figma designs for the Salt site are likely to contain component designs you could extract and reuse. Btw, I'm hoping to produce some team-internal component doc pages for the existing site components, so that would be another thing to align (or collaborate?) on.
If it's the latter, I'd be interested to understand what differences you anticipate compared to the generic Salt spec library? I.e. what extra or different things might a designer need when annotating a design for the Salt website versus for the Sal component libraries?

Thank you for raising it @james-nash
Should be the first one, I will schedule a call with @shey-v @bhoppers2008 to see how we can use what we've had and collaborate.

Cool. Could you include me in that call too, please?

@james-nash fyi I asked @yunjungyeh to quickly pull together a library based on the forms pattern page i've built, in order to help put together the International Address Pattern page. At this stage, that's the primary goal, with the secondary goal of having a library designer's can use to create these pages quickly. If there are more that Yun can extract from @shey-v 's work then that's brilliant but i know the one's i've built are representative having worked with Josh and a quick start for Yun :)

@bhoppers2008 Thanks for the clarification. Makes sense. Might be a good idea to update this issue's description then.
I think it may still be worth us all having a quick chat. For any new site pages/seciton I'm keen for us to maximize reusing things we already have and minimising "special snowflake" components. I think if we can start collating the available components somewhere, so that they're discoverable and somewhat documented for the next person who wants to add a new page or section, that'll go a long way to that end.
It can (and at this point, probably should) lean and scrappy. Maybe a Figma shared library where we can "dump" things we've already made, which the team can then use and add to/refine over time. As mentioned above, I'd like to do something analagous with the React components - i.e. use our component page template to write up some basic docs and examples, which we can then surface in a secret section of our site.

@james-nash no problem would include you as well! For now I would focus on getting International Address Input pattern done as pattern page is the first priority. Then we can sync and collaborate - will schedule some time possibly next week, thanks!

Thanks for looking into this @yunjungyeh - it would be brilliant to have a library specifically for patterns, I started some research and initial mockups in Figma https://www.figma.com/file/luPB1hhmKO8UKfhzgipK9D/Site---Birthday-cake%3A-Patterns-documentation?type=design&node-id=1%3A3482&mode=design&t=kXbel5RIyeb0cDBO-1 a while ago, if you need to reference it :)

✅5/6 July - sort out new components in Components template
✅30 June/3-4 July - build components for Site utilities
✅29/30 June - Analysing the built site and the design in Figma files (to define the spec of components)


Reading PyKOALA processed RSS fits files

Hi there,
Processed RSS files saved into fits files with task koala_rss_to_fits() can't be read using koala_rss() as this fails because they don't have bad fibres.
The preliminary solution I've included in my version is replacing this line in koala_rss_to_fits()
bad_fibres_list = (fibre_table['SPEC_ID'][fibre_table['SELECTED'] == 0] - 1).tolist()
for this:
    try:
        bad_fibres_list = (fibre_table['SPEC_ID'][fibre_table['SELECTED'] == 0] - 1).tolist()
    except Exception:
        bad_fibres_list = []

I had to change something else but I can't remember what was it and now I can't (again) run anything in my laptop till I uninstall and delete all Anaconda and put it back WITHOUT installing PyKOALA (running pip install . in my laptop messes everything).
Below my full version of the koala_rss()
def koala_rss(path_to_file, **kwargs):
    """
    A wrapper function that converts a file (not an RSS object) to a koala RSS object
    The paramaters used to build the RSS object e.g. bad spaxels, header etc all come from the original (non PyKoala) .fits file
    """
    verbose = kwargs.get('verbose', False)
    if verbose: print('\n> Converting RSS file',path_to_file,'to a koala RSS object...')

    header = fits.getheader(path_to_file, 0) + fits.getheader(path_to_file, 2)
    koala_header = py_koala_header(header)
    # WCS
    koala_wcs = WCS(header)
    # Constructing Pykoala Spaxels table from 2dfdr spaxels table (data[2])
    fibre_table = fits.getdata(path_to_file, 2)
    koala_fibre_table = py_koala_fibre_table(fibre_table)

    # List of bad spaxels from 2dfdr spaxels table
    try:
        bad_fibres_list = (fibre_table['SPEC_ID'][fibre_table['SELECTED'] == 0] - 1).tolist()
    except Exception:
        bad_fibres_list = []
    # -1 to start in 0 rather than in 1
    # Create the dictionary containing relevant information
    info = {}
    info['name'] = koala_header['OBJECT']
    info['exptime'] = koala_header['EXPOSED']
    info['fib_ra'] = np.rad2deg(koala_header['RACEN']) + koala_fibre_table.data['Delta_RA'] / 3600
    info['fib_dec'] = np.rad2deg(koala_header['DECCEN']) + koala_fibre_table.data['Delta_DEC'] / 3600
    info['airmass'] = airmass_from_header(koala_header)
    # Read RSS file into a PyKoala RSS object
    rss = read_rss(path_to_file, wcs=koala_wcs,
                   bad_fibres_list=bad_fibres_list,
                   intensity_axis=0,
                   variance_axis=1,
                   header=koala_header,
                   fibre_table=koala_fibre_table,
                   info=info,
                   verbose = verbose
                   )

    # Plot RSS image if requested
    if kwargs.get('plot'): rss_image(rss, **kwargs)

    return rss

Why koala_rss_to_fits exists? I/O operations should be done using RSS methods from the base class (indeed, there is a function for that RSS.to_fits).


FIX deprecation warnings in test_cli

Deprecations reported in #182 :

[x] autocompletion -> shell_complete
[ ] Creating a LegacyVersion has been deprecated and will be removed in the next major release
[x] User warning on editable config.

The last one with the LegacyVersion seems to be due to a bug in setuptools that have been fixed in : pypa/setuptools#2889

Codecov Report

Merging #267 (2e16b90) into main (7efe794) will not change coverage.
The diff coverage is 50.00%.

:exclamation: Current head 2e16b90 differs from pull request most recent head f011243. Consider uploading reports for the commit f011243 to get more accurate results

@@           Coverage Diff           @@
##             main     #267   +/-   ##
=======================================
  Coverage   62.96%   62.96%
=======================================
  Files          41       41
  Lines        2087     2087
  Branches      367      367
=======================================
  Hits         1314     1314
  Misses        677      677
  Partials       96       96


Add GoCon Canada 2020

Could you add GoCon Canada's Diversity Scholarship?
https://gocon.ca/#diversity-scholarship

✅ By submitting this issue, I have verified the following

[x] Checked to see if the issue has already been discussed before 🤔
[x] If proposing new content to be added, made sure enough details were provided 🔍

Hello. I made a PR to add KubeCon, and noticed your raised issue. I added GoCon conference with the link you sent to the Canada section and included it in my PR: https://github.com/fvcproductions/diversify-me/pull/14 If you do not mind.
I also added your name and Github link to the credits section since creating an issue is also contributing. Hope this PR gets reviewed and approved soon!

Added in!


[SYM-3792] Ensure params and prompt_fields are still optional

The original ticket for this PR was to ensure prompt_fields were optional.

First, I confirmed that prompt_fields were optional in provider 1.x.x. They were. The behavior is weird, but not broken.
Then, I tested whether they were still optional in 2.x.x. They were! But I discovered some other weird behaviors.
First weird behavior: not including the params block didn't actually register a diff as if existing params were removed from the configuration. This was because of the Computed: true flag we had on params. So I removed that, and now deleting the params block entirely will treat it as if you want all the default values.
Second weird behavior (bug): not including allowed_sources in params meant that it was stored as an empty list. This meant that the flow was unavailable in Slack or the API, and happened because Terraform was always defaulting us to an empty list instead of nil. In provider 1.x, we would not send it to the API at all if it was missing. Now, we will do that in 2.x as well.

Current dependencies on/for this PR:

main

PR #134   👈

This comment was auto-generated by Graphite.


Janpio pilot no 2fa 2

Checklist

[ ] I've run bundle exec rspec from the root directory to see all new and existing tests pass
[ ] I've followed the fastlane code style and run bundle exec rubocop -a to ensure the code style is valid
[ ] I've read the Contribution Guidelines
[ ] I've updated the documentation if necessary.

Motivation and Context

Description

          2 Warnings

:warning:
Big PR

:warning:
You modified SnapshotHelper.swift, make sure to update the version number at the bottom of the file to notify users about the new helper file.

  Generated by :no_entry_sign: Danger


KeyError: 'turn'

I keep getting this error it worked fine the first time I ran it but then the second time this error started popping up

Maybe it's a filter, check your chat

the error is outdated and most likely fixed


重启 management node时，heartbeat把自己从hash ring删掉了，还报错ConcurrentModificationException，但是数据库存在，而且是RUNNING。

2018-03-30 09:22:09,187 DEBUG [SimpleFlowChain] {} [FlowChain(FCID_0ee996c2): management-node-bootstrap] successfully executed flow[ManagementNodeManagerImpl.java:node-is-ready]
2018-03-30 09:22:09,198 DEBUG [SimpleFlowChain] {} [FlowChain(FCID_0ee996c2): management-node-bootstrap] start executing flow[ManagementNodeManagerImpl.java:say-I-join]
2018-03-30 09:22:09,206 DEBUG [ConsistentHash] {} the consistent hash ring currently has nodes[2bbb5eb375be472eb4ba07747d264b0a]
2018-03-30 09:22:09,187 DEBUG [SimpleFlowChain] {} [FlowChain(FCID_0ee996c2): management-node-bootstrap] successfully executed flow[ManagementNodeManagerImpl.java:node-is-ready]
2018-03-30 09:22:09,188 DEBUG [SimpleFlowChain] {} [FlowChain(FCID_0ee996c2): management-node-bootstrap] start executing flow[ManagementNodeManagerImpl.java:listen-node-life-cycle-events]
2018-03-30 09:22:09,197 DEBUG [SimpleFlowChain] {} [FlowChain(FCID_0ee996c2): management-node-bootstrap] successfully executed flow[ManagementNodeManagerImpl.java:listen-node-life-cycle-events]
2018-03-30 09:22:09,198 DEBUG [SimpleFlowChain] {} [FlowChain(FCID_0ee996c2): management-node-bootstrap] start executing flow[ManagementNodeManagerImpl.java:say-I-join]
2018-03-30 09:22:09,206 WARN  [ManagementNodeManager] {} management node[uuid:2bbb5eb375be472eb4ba07747d264b0a, hostname: null]'s heart beat has stopped for 3307 secs, add it in suspicious list
2018-03-30 09:22:09,206 WARN  [ManagementNodeManager] {} found that a management node[uuid:5d38fcf801e048abb21f6cb6e4d06d05] had no heartbeat in database but still in our hash ring,notify that it's dead
2018-03-30 09:22:09,206 DEBUG [ConsistentHash] {} the consistent hash ring currently has nodes[2bbb5eb375be472eb4ba07747d264b0a]
2018-03-30 09:22:09,211 DEBUG [ConsistentHash] {} after removing, consistent hash circle has management nodes[2bbb5eb375be472eb4ba07747d264b0a], 500 virtual nodes now
2018-03-30 09:22:09,219 DEBUG [ManagementNodeManager] {} management node is still initializing ...
2018-03-30 09:22:09,227 TRACE [GLock] {} [GLock]: thread[zs-thread-4] is acquiring lock[JobQueueFacade.orphanJobLock]
2018-03-30 09:22:09,231 TRACE [GLock] {} [GLock DB Lock]: thread: zs-thread-4 got DB lock[JobQueueFacade.orphanJobLock], during timeout[60 secs]
2018-03-30 09:22:09,231 DEBUG [JobQueueFacadeImpl2] {} management node[id:5d38fcf801e048abb21f6cb6e4d06d05] starts taking over jobs of left management node[5d38fcf801e048abb21f6cb6e4d06d05]
2018-03-30 09:22:09,328 TRACE [CloudBusImpl2] {} [event publish]: {"org.zstack.core.cloudbus.CanonicalEvent":{"path":"/managementnode/lifecycle","content":{"nodeUuid":"5d38fcf801e048abb21f6cb6e4d06d05","lifeCycle":"NodeJoin","inventory":{"uuid":"5d38fcf801e048abb21f6cb6e4d06d05","joinDate":"Mar 30, 2018 9:22:08 AM","heartBeat":"Mar 30, 2018 9:22:08 AM"}},"managementNodeId":"5d38fcf801e048abb21f6cb6e4d06d05","headers":{"schema":{"content":"org.zstack.header.managementnode.ManagementNodeCanonicalEvent$ManagementNodeLifeCycleData","content.inventory":"org.zstack.header.managementnode.ManagementNodeInventory"},"task-context":{}},"id":"45cf106121ab43408babdb01bde5b7f2","noError":false,"createdTime":1522372929201}}
2018-03-30 09:22:09,329 DEBUG [JobQueueFacadeImpl2] {} [Orphan Queue found]: management node is going to take over 0 orphan queues
2018-03-30 09:22:09,329 TRACE [GLock] {} [GLock]: thread[zs-thread-4] is releasing lock[JobQueueFacade.orphanJobLock]
2018-03-30 09:22:09,331 TRACE [GLock] {} [GLock Release DB Lock] thread[zs-thread-4] released DB lock[JobQueueFacade.orphanJobLock]
2018-03-30 09:22:09,331 TRACE [GLock] {} [GLock Release Memory Lock]: thread[zs-thread-4] released memory lock[JobQueueFacade.orphanJobLock]
2018-03-30 09:22:09,331 WARN  [ManagementNodeManager] {} an error happened when doing heartbeat, null, it's going to recover
2018-03-30 09:22:09,343 DEBUG [SimpleFlowChain] {} [FlowChain(FCID_0ee996c2): management-node-bootstrap] successfully executed flow[ManagementNodeManagerImpl.java:say-I-join]
2018-03-30 09:22:09,344 DEBUG [SimpleFlowChain] {} [FlowChain(FCID_0ee996c2): management-node-bootstrap] successfully completed
2018-03-30 09:22:09,344 TRACE [GLock] {} [GLock]: thread[zs-thread-0] is releasing lock[ManagementNodeManager.inventory_lock]
2018-03-30 09:22:09,334 WARN  [ManagementNodeManager] {} unhandled exception happened
java.util.ConcurrentModificationException: null
at java.util.HashMap$HashIterator.nextNode(HashMap.java:1437) ~[?:1.8.0_144]
at java.util.HashMap$KeyIterator.next(HashMap.java:1461) ~[?:1.8.0_144]
at org.zstack.portal.managementnode.ManagementNodeManagerImpl$21.checkAllNodesHealth(ManagementNodeManagerImpl.java:688) ~[portal-2.0.0.jar:?]
at org.zstack.portal.managementnode.ManagementNodeManagerImpl$21.call(ManagementNodeManagerImpl.java:709) [portal-2.0.0.jar:?]
at org.zstack.portal.managementnode.ManagementNodeManagerImpl$21.call(ManagementNodeManagerImpl.java:1) [portal-2.0.0.jar:?]
at org.zstack.core.thread.ThreadFacadeImpl$Worker.call(ThreadFacadeImpl.java:113) [core-2.0.0.jar:?]
at java.util.concurrent.FutureTask.run(FutureTask.java:266) [?:1.8.0_144]

2018-03-30 09:22:09,206 WARN  [ManagementNodeManager] {} management node[uuid:2bbb5eb375be472eb4ba07747d264b0a, hostname: null]'s heart beat has stopped for 3307 secs, add it in suspicious list
2018-03-30 09:22:09,206 WARN  [ManagementNodeManager] {} found that a management node[uuid:5d38fcf801e048abb21f6cb6e4d06d05] had no heartbeat in database but still in our hash ring,notify that it's dead
2bbb5eb375be472eb4ba07747d264b0a 是老的UUID
5d38fcf801e048abb21f6cb6e4d06d05  是新的UUID

@wj5633 你是HA环境同时启动多个管理节点？什么版本？最好描述一下环境。

单节点，1.7   场景是management 意外挂掉了，数据库还在，重启management。
我还和master对比了一下ManagementNodeManagerImpl.java文件

@wj5633 本次发布来不及处理，我们下个版本修复。不过 1.7 也太老了。


[RFC] [EntityContext] JSON for arrays

Hi,
Currently to specify an array we need to write it this way:
Item1, Item2, Item3

This is a problem mainly because it doesn't support multi-dimensions. I suggest to also support JSON so we'll be able to write array this way in build in fixtures:
{"cat1": ["item1", "item2"], "cat2": ["item3"]}

I don't think that JSON would be the answer, since JSON can be supported (for instance by MySQL) as a format as it is. How would you distinct real JSON and PHP arrays? :confused:
What about:
["cat1": ["item1", "item2"], "cat2": ["item3"]]

?

@pimolo yes and it's pretty easy to go on. Do it if you want :).


error

hello, i have this error. i have try with node 8.11/9/10/12/14.5/15.6
can you help me please
/root/poo/node_modules/bitcoinjs-lib/src/address.js:89
throw new Error(address + ' has no matching Script');
^
Error: undefined has no matching Script
at Object.toOutputScript (/root/poo/node_modules/bitcoinjs-lib/src/address.js:89:9)
at Object.exports.addressToScript (/root/poo/node_modules/stratum-pool/lib/util.js:266:32)
at generateOutputTransactions (/root/poo/node_modules/stratum-pool/lib/transactions.js:42:36)
at Object.exports.CreateGeneration (/root/poo/node_modules/stratum-pool/lib/transactions.js:239:30)
at new BlockTemplate (/root/poo/node_modules/stratum-pool/lib/blockTemplate.js:75:47)
at JobManager.processTemplate (/root/poo/node_modules/stratum-pool/lib/jobManager.js:185:32)
at /root/poo/node_modules/stratum-pool/lib/pool.js:635:62
at itemFinished (/root/poo/node_modules/stratum-pool/lib/daemon.js:162:36)
at /root/poo/node_modules/stratum-pool/lib/daemon.js:184:17
at parseJson (/root/poo/node_modules/stratum-pool/lib/daemon.js:91:17)

SprintPay coin (yespower)

SprintPay coin (yespower)

i had try to install it on 3 servers (ubuntu 16 - 20.04 - 2.10) with 10 versions of node but don't work. always this error

i had try to install it on 3 servers (ubuntu 16 - 20.04 - 2.10) with 10 versions of node but don't work. always this error

?


Adding changes to README.md

Adding a new line at end for better formatting and view
Please review and merge these changes.

Thank you for your contribution! We will review the pull request and get back to you soon.


fix: add missing UnknownNetworkError

The net package recently added support for a custom transport to be added, largely to support the changes in tinygo for wasip2.
This PR adds the missing error type UnknownNetworkError that was introduced in #25
After adding this and updating the submodule reference in tinygo for the net package - custom HTTP transports can be successfully called when targeting wasip2.
Referenced in #31

Also found the Unknown error in https://github.com/tinygo-org/net/blob/94998db67997d794f3fdc7b1ef5ad600c8f83f6b/net.go
This is in the open PR - https://github.com/tinygo-org/net/pull/24.
Regardless - getting one of these through would be ideal tinygo can update the net submodule and compile.

Looks OK to me, thanks for the addition @elewis787 now merging.


Getting started doc update

What this PR does / why we need it:
Update the getting started documentation to replace the kubeadm flag with bootstrap-kubeconfig that is used to specify kubeconfig to BYOH agent.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Additional information

Can we close this in favor of #667 ?


Firefox extension

Do you plan on uploading to Mozilla addons?

I can have a look at how much work it would be to port it if you want.

I tested it on Firefox 54.0a2, but it should work on the release version too since I don't think there are any APIs used that aren't supported it.

Added to Firefox marketplace, see README for details.


Delete TPR when NS is deleted

https://github.com/kubernetes/kubernetes/issues/46736

Fixed in #282


Grab slurm from development repos instead of contrib (SOFTWARE-2994)

Test results are good: http://vdt.cs.wisc.edu/tests/20171117-1442/results.html

Oops, accidentally deleted the origin branch


Refactor and tweaks to exp-video-physics

(Copying notes here because this is probably the more logical location) - issues:

When leaving FS, the "return to FS" button isn't shown, so the user is stuck.

If we get to the next exp-physics-video frame by way of exhausting the test and alternate video (i.e., if the user pauses during the test video, and then again during the alternate test video) then leaving FS does show the button but doesn't pause the video.

The test videos are only being played once--they should loop for a total duration of testLength s.

Looks great!


Add support prisma version 3.1.1

Describe the Bug
It looks like version 3.1.1 is not supported

I think I should update the warning message:
https://prisma.typegraphql.com/docs/advanced/prisma-version

Many thanks. My question is resolved !!!

@MichalLytek wouldn't it be better to require just the same major versions?
Are we really get into situations when patches and minors are going to break the code generator?

Are we really get into situations when patches and minors are going to break the code generator?

Yes, it's happening really often because DMMF nor generator are not considered public API so they change that without further notice - I mean they have typegraphql-prisma inside their tests pipeline so they let me know new version breaks the generator.
Anyway, it has to stay because I don't have time to close 10 issues about "not working with xyz" or "breaks on new Prisma". All you need is to use the env variable if you want experimental features.

You can try to do a periodic check using github action

@overpod New release v0.16.1 is now compatible with Prisma 3.1.1 🚀
It contains an improvement in Prisma version check message, which now points to docs website that describe the behavior in details: https://prisma.typegraphql.com/docs/basics/prisma-version
I agree that an ideal solution would be an automatic release mechanism which would be triggered by new Prisma release and that would check if there are any issues with new Prisma release. However, I don't have time to spend on digging into this solution, I think that the env variable workaround solves 99% of issues related to Prisma version check error.


were are nodes? (everything is updated)

same red

It is possible that you miss some required packages in your python environment.
Look at requirements.txt of this repo. This Node will not install them automatically. You must install them manually. Then will ComfyUI load the nodes correctly.
If you need help how to do it. Just let me know.

same issue as op, have latest version of comfyui portable and installed the requirements.txt, still same red missing nodes?

Following erros @ Boot:
Traceback (most recent call last):
File "D:\DEV\COMFY\ComfyUI\nodes.py", line 1998, in load_custom_node
module_spec.loader.exec_module(module)
File "", line 940, in exec_module
File "", line 241, in call_with_frames_removed
File "D:\DEV\COMFY\ComfyUI\custom_nodes\ComfyUI-PuLID-Flux_init.py", line 1, in
from .pulidflux import NODE_CLASS_MAPPINGS, NODE_DISPLAY_NAME_MAPPINGS
File "D:\DEV\COMFY\ComfyUI\custom_nodes\ComfyUI-PuLID-Flux\pulidflux.py", line 12, in
from facexlib.parsing import init_parsing_model
ModuleNotFoundError: No module named 'facexlib'

This fixed it for me: https://github.com/cubiq/PuLID_ComfyUI/issues/23

it literally says whts wrong...
ModuleNotFoundError: No module named 'facexlib'
pip install facexlib

I am also facing the same issue with the missing node and i have manually installed PuLID but am not seeing the nodes though.

@leonviz87
I have always the same question, did you install also the Python packages in requirements.txt ?
The problem is that I newer know what experience you all have with Python (I don't want to offend anyone) so I rather ask, because 90% of all missing node errors are caused by the missing packages (I even updated the install instructions to reflect the problem).

Many people encounter the same problem when using the portable version of ComfyUI. The issue with the node not showing is unrelated to this repository!
It occurs because the portable version has its own Python interpreter. If you run pip install -r requirements.txt, it will install the packages in your local Python environment, not in the embedded Python within the portable ComfyUI. You need to navigate to the directory where the embedded Python is located and install the packages from there.

Copy the requirements.txt file inside the python path, after copying this command python.exe -m pip install -r requirements.txt open the cmd and the installation is successful! Pro-tested it works, thanks for sharing!

Many people encounter the same problem when using the portable version of ComfyUI. The issue with the node not showing is unrelated to this repository!
It occurs because the portable version has its own Python interpreter. If you run pip install -r requirements.txt, it will install the packages in your local Python environment, not in the embedded Python within the portable ComfyUI. You need to navigate to the directory where the embedded Python is located and install the packages from there.

Copy the requirements.txt file inside the python path, after copying this command python.exe -m pip install -r requirements.txt open the cmd and the installation is successful! Pro-tested it works, thanks for sharing!

Copy the requirements.txt file inside the python path, after copying this command python.exe -m pip install -r requirements.txt open the cmd and the installation is successful! Pro-tested it works, thanks for sharing!

i did this but havent solved

Try: https://github.com/cubiq/PuLID_ComfyUI/issues/23

@threeforces @cchance27 @4lt3r3go @leonviz87 please see this solution: https://github.com/balazik/ComfyUI-PuLID-Flux/issues/20#issuecomment-2408536291
Of course, don’t just copy the links directly; make sure to adjust them according to your portable ComfyUI path.


Add custom components to each image slide

Hi Michael,
first of all thank you for this awesome library. Really slick, good looking solution!
I was wondering if there is a way to add some custom views to each slide. In my case I wanna show some tags, related to the image. Any idea how I could achieve this?
Best wishes, Sven.

Hi Sven!
I kept this component very simple on purpose, but this would be very easily achievable.
I'll see if I have time tomorrow, in any cases take a look at the code, there is not much of it for now. and submit a PR if you want :).

Hi @Michaelvilleneuve, that would be great. Thank you so much!

Hi, you can now add an overlay component to each data object in c16fce1ad21109eda9064e661e89e0e2a0e2591e, it will be rendered above the image.
Remember not to make it to large or you won't be able to zoom in anymore as it is an overlay.
Try this to see how it renders :
{ id: '1', image: 'xxx', overlay: <View style={{ position: 'absolute', top: 40, width: 100, height: 100, backgroudColor: 'blue' }} /> }

Hi Michael, thank you so much. Working perfectly fine!


Multiple errors handling for edit command not comprehensive

^2 errors are shown here  (missing index and wrong role name)
Either we show all the errors (hard), or errors detected should go from left to right.

Fixed closing issue


Fix: images relative path

Updates

Changed Images to images

@illunise Updated and now its linked properly. Merge it please


Can't start kvrocks in Docker container on macos.

Search before asking

[X] I had searched in the issues and found no similar issues.

Version
MacOS version: 12.6, MacBook Pro Apple M1 Max
Docker desktop version: 4.12.0 (85629)
kvrocks version: latest from Docker hub
https://hub.docker.com/layers/apache/kvrocks/latest/images/sha256-9ef819b07e071f03125846c08e01ee4651710f2ee70518987fbaf6944a6e5b62?context=explore
Minimal reproduce step
❯ docker run -it -p 6666:6666 apache/kvrocks
<jemalloc>: MADV_DONTNEED does not work (memset will be used instead)
<jemalloc>: (This is the expected behaviour if you are running under QEMU)

❯ docker ps -a
CONTAINER ID   IMAGE            COMMAND                  CREATED              STATUS                            PORTS                    NAMES
2c5f0f887f81   apache/kvrocks   "./bin/kvrocks -c ./…"   About a minute ago   Exited (137) About a minute ago

What did you expect to see?
kvrocks start log.
What did you see instead?
<jemalloc>: MADV_DONTNEED does not work (memset will be used instead)
<jemalloc>: (This is the expected behaviour if you are running under QEMU)

Anything Else?
No response
Are you willing to submit a PR?

[ ] I'm willing to submit a PR!

nightly build provides one more line of error:
❯ docker run -it -p 6666:6666 --name zkbnb-kvrocks apache/kvrocks:nightly
<jemalloc>: MADV_DONTNEED does not work (memset will be used instead)
<jemalloc>: (This is the expected behaviour if you are running under QEMU)
qemu: uncaught target signal 4 (Illegal instruction) - core dumped

@ruslangm Thanks for your feedback. It looks like the docker client wrongly recognise the CPU platform.  Your docker's qemu is running in linux/amd64 but from my side is aarch64. So you got the error since the docker client thought its arch is arm64 and pulled the image with the arm64 platform.
❯ ps aux|grep qemu
hulk             13890   1.9  7.4 418260112 1247520   ??  S     8:28PM   3:30.06 /Applications/Docker.app/Contents/MacOS/qemu-system-aarch64 -accel hvf -cpu host -machine virt,highmem=off -m 8092 -smp 5 -kernel /Applications/Docker.app/Contents/Resources/linuxkit/kernel -append page_poison=1 vsyscall=emulate panic=1 nospec_store_bypass_disable noibrs noibpb no_stf_barrier mitigations=off linuxkit.unified_cgroup_hierarchy=1    vpnkit.connect=tcp+bootstrap+client://<IP_ADDRESS>:62364/bcb9f8f0f5a636d920d70e033a3333c9f4fe0807b19bb748bbd1561af926d878 vpnkit.disable=osxfs-data console=ttyAMA0 -initrd /Applications/Docker.app/Contents/Resources/linuxkit/initrd.img -serial pipe:/var/folders/ls/jsfyf9g52h94_tmgys0895qw0000gn/T/qemu-console1146358452/fifo -drive if=none,file=/Users/hulk/Library/Containers/com.docker.docker/Data/vms/0/data/Docker.raw,format=raw,id=hd0 -device virtio-blk-pci,drive=hd0,serial=dummyserial -netdev socket,id=net1,fd=3 -device virtio-net-device,netdev=net1,mac=02:50:00:00:00:01 -vga none -nographic -monitor none

so you will

@git-hulk can it be related to difference between Docker Desktop for Intel Chip and Docker Desktop for Apple Chip?

❯ ps aux | grep qemu
user             20177  13.8  1.2 418377584 387504   ??  R     7:38PM  13:39.97 /Applications/Docker.app/Contents/MacOS/qemu-system-aarch64 -accel hvf -cpu host ...

❯ ps aux | grep qemu
user             20177  13.8  1.2 418377584 387504   ??  R     7:38PM  13:39.97 /Applications/Docker.app/Contents/MacOS/qemu-system-aarch64 -accel hvf -cpu host ...

ummm... I have no idea why it happened, your QEMU is running in aarch64 but it thought itself needs the amd64 image:
was found but does not match the specified platform: wanted linux/arm64, actual: linux/amd64

Moved questions to the Q&A forum.


Allow to set custom memory size.

At the moment, memory size is only 16mb so lib crash on big graphs with the following error:
Uncaught abort("Cannot enlarge memory arrays. Either (1) compile with  -s TOTAL_MEMORY=X  with X higher than the current value 16777216, (2) compile with  -s ALLOW_MEMORY_GROWTH=1  which adjusts the size at runtime but prevents some optimizations, (3) set Module.TOTAL_MEMORY to a higher value before the program runs, or if you want malloc to return NULL (0) instead of this abort, compile with  -s ABORTING_MALLOC=0 ") at Error

So this PR add setMemorySize function to force bigger memory size, for e.g. we set it to 32mb in our project.

Looks good, thanks!

Looks good, thanks!

@mdaines Thank you. Happy new year 🎄
Can you please publish the new version of npm package?

Yes, I plan to publish a new version in the next week or so.

Just a note that I've gone with a different approach: Viz.js will now set ALLOW_MEMORY_GROWTH, so you don't have to guess the amount of memory it needs. b1df4a9662188fb1aefb0489ae6f1c4667bec81f


Update hll-aggfunction.md

Learn Build status updates of commit c6adb63:
:warning: Validation status: warnings

File
Status
Preview URL
Details

data-explorer/kusto/query/hll-aggfunction.md
:warning:Warning

Details

data-explorer/kusto/query/hll-aggfunction.md

Line 25, Column 119: [Warning: link-out-of-scope] File 'includes/data-explorer-estimation-accuracy.md' referenced by link '../../includes/data-explorer-estimation-accuracy.md' will not be built because it is not included in build scope.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Learn Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit ca5b1a6:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

data-explorer/kusto/query/hll-aggfunction.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Merging based on approval comment: https://github.com/MicrosoftDocs/dataexplorer-docs/pull/1467#issuecomment-1290281079


Update darwin.go

@zpratt do you want to wrap this in test cases before we add this?

let's add tests first. we should also get some kind of CI in place for this

closing until we can get CI setup...


[Event Request] - OnBeforePrintMultiplePurchaseDocs Codeunit 99000813 "Carry Out Action"

Hello, please consider adding the following event to Codeunit 99000813 "Carry Out Action"
    [IntegrationEvent(false, false)]
    local procedure OnBeforePrintMultiplePurchaseDocs(var TempPurchaseHeader: Record "Purchase Header" temporary)
    begin
    end;

and call it accordingly inside the "PrintMultiplePurchaseOrders" procedure:
    procedure PrintMultiplePurchaseOrders(var TempPurchaseHeader: Record "Purchase Header" temporary)
    var
        PurchaseHeader: Record "Purchase Header";
        ReportSelections: Record "Report Selections";
        PurchCalcDiscByType: Codeunit "Purch - Calc Disc. By Type";
        SelectionFilterMgt: Codeunit SelectionFilterManagement;
        TempRecRef: RecordRef;
        RecRef: RecordRef;
        PurchaseOrderNoFilter: Text;

        **IsHandled: Boolean;**

    begin
        if not PrintOrder then
            exit;

        TempPurchaseHeader.Reset();
        if TempPurchaseHeader.IsEmpty() then
            exit;

        TempPurchaseHeader.FindSet();
        repeat
            PurchaseHeader.Get(TempPurchaseHeader."Document Type", TempPurchaseHeader."No.");
            PurchCalcDiscByType.ApplyDefaultInvoiceDiscount(0, PurchaseHeader);
        until TempPurchaseHeader.Next() = 0;

        **IsHandled := false;
        OnBeforePrintMultiplePurchaseDocs(TempPurchaseHeader);
        if IsHandled then
            exit;**

        TempRecRef.GetTable(TempPurchaseHeader);
        RecRef.GetTable(PurchaseHeader);
        PurchaseOrderNoFilter := SelectionFilterMgt.CreateFilterFromTempTable(TempRecRef, RecRef, PurchaseHeader.FieldNo("No."));

        PurchaseHeader.SetRange("Document Type", PurchaseHeader."Document Type"::Order);
        PurchaseHeader.SetFilter("No.", PurchaseOrderNoFilter);
        PurchaseHeader.SetFilter("Buy-from Vendor No.", '<>%1', '');
        ReportSelections.PrintWithDialogWithCheckForVend(
            ReportSelections.Usage::"P.Order", PurchaseHeader, false, PurchaseHeader.FieldNo("Buy-from Vendor No."));
    end;

Thank you !

What are you trying to achieve with this?
Adding an exit in the middle of procedure for event is not something we are very willing to do.

Hi,
i'm trying to customize the the printing process. Something like happens in procedure "PrintPurchaseOrder": same behaviour with "if handled then exit" in the middle of the proedure.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 10481.


Create an Interface for PackageStorage that talks to the new PackageStorage Service

Package Publish uses PackageStorage for storing packages. This will need to be moved out of cosmos into a separate service. PackageStorage will become an interface that talks to this new service.

We have moved away from this design. Closing the issue.


wdl_runner: monitor, tests, latest Cromwell release

Add wdl_runner/tools/monitor_wdl_pipeline.sh which can give some useful
insight into a running workflow.
Add test/tests/wdl_runner/test.README.wdl_pipeline_from_git.sh to make it
easy to test one of the examples from the README.
Update Dockerfile and setup.sh to pick up Cromwell 0.19.3.
Also modified poll.sh:

Default interval is now 60 seconds instead of 20
Default completed operation output is brief

LGTM


Remove Kitchen.serve(jsFile:) API

Created this issue so this breaking change is noticeable for everyone.
I decided to remove this API to keep things simple.
I assume nobody would do fancy things in their jsFiles via TVMLKitchen anyways.
Please open a feature request if really need to specify jsFile path and let TVJS to evaluate them.

Done in #44


fix texture applying on z- face

When we apply a detailed texture on the voxels, we can notice that on the z- face it is not correctly applied.
Here is a picture to illustrate this :

Thanks for the fix


[vuejs] Expand table of identified variants without need to hide filter form

Describe the bug
In the old variant filter interface, the resulting variant table can be scrolled and is not restricted to the visible area below the filter form. This is a UX regression, as now the filter form needs to be manually hidden to expand the list of variants.

Thanks, we are aware of that. We will move to a different table plugin after the switch and with the issue will be addressed. Thus we leave it as is for now, but I will also leave the ticket open for the next increment.

Solved by #961 #965


Go to implementation should support loading indicators

I suggest that ImplementationProvider should be support asynchronous loading indication in the same way as ReferenceProvider.
My understanding based on other conversation is that the current assumption with ImplementationProviders is that they will return almost immediately.
In popular languages which actually enable the static analysis to drive implementation finding (e.g. Java, Go), I think it's reasonable to expect the analysis to take more than a few seconds (see https://github.com/Microsoft/vscode-go/pull/1100), and the current API behavior means users are in the dark about what's happening while the analysis is processing.

cc @mjbvz

The progress indicator should be available in tomorrow's Insiders


What is the Target for 7.0.0-rc.1 and final release version number.

@TravisEz13 - when I run the 7.0.0-rc.1 MSI manually on a system that contains both 6.2.3 and 7.0 preview, it targets the 7.0 preview folder for upgrade, not the 6 folder.
This suggests you want it to stay in the preview distribution channel.
Can you please clarify which chocolatey packages sequences should have 7.0.0-rc.1 queued up as the next release?

PowerShell-preview (as a visible package since everything in it is of a pre-production nature)
PowerShell-Core (as a non-visible, prerelease package - would happen automatically due to "-rc" in version number).  Would require "-pre" switch to see or install from this feed.

My concern about #2 is that it won't actually replace the 6 version, so does not function as a true prerelease.
Also would it please be possible to have the production release be "<IP_ADDRESS>" ?  This clears us of all semantic prerelease / chocolatey version weirdness if it can be done - because in both systems it is greater than all other version numbers that predate it on both the preview and production channel.

Releasing as <IP_ADDRESS> on Powershell-preview since it upgrades the preview folder.  Will also be visible this way and not require the -pre switch.

Darwin, I've been sick.  Everything with a - in the version is in the preview channel.


Update templates

The templates are a little frustrating.

They include a className component which is often unneeded as a prop in prototyping.
They are also declared as functions rather than as const, which makes assigning the type frustrating when dealing with special types of components.

For example, the Component template would probably be better if it was like this:

/** Props for the FooBar */
export interface FooBarProps {
  // Your props here!
};

/**
 * Some description of FooBar
 *
 * @param props
 * @returns
 */
const FooBar: React.FC<FooBarProps> = (props) => {
  const { /* props */ } = props;

  return <div />;
};

export default FooBar;

Alternatively, we could try to incorporate styled components in the default template. Although it may make sense to have a different template for that.

@JaredBourget lmk what you think! I think this is a bit closer to our previous patterns.

Yeah, your example is my preferred pattern for a component template.
Are you just thinking of including the import by default? Otherwise, I'm not sure how much sense it would make to incorporate styled components in the default template -  it seems like it would be heavily reliant on the meat of the component.

Yeah, your example is my preferred pattern for a component template.
Are you just thinking of including the import by default? Otherwise, I'm not sure how much sense it would make to incorporate styled components in the default template - it seems like it would be heavily reliant on the meat of the component.

Agreed. I'll just keep it simple and use our preferred pattern.


Howto install newest release?

Hello,
I'm not that familiar with python…
Following readme apparently installs v2.0.5 and seams to run. I can select movies and get some debug output at the console with current camilla configuration but no filters are added to camilladsp. (at least I do not see any when I do: fetch from DSP within camillagui)
How to upgrade to current release (2.1.3) in case this issue was fixed?
Regards
Michael

It's likely you are on some old python version, use 3.11 or 3.12 to pick up current version

Thanks for your quick support. With 3.12 it is working nicely.


setDate doesn't work when using jquery init

Hi, when initialising the picker using the jquery method:
cal = $('#picker').flatpickr()
Then I can't then use cal.setDate(something). I get a "cal.setDate is not a function" error.
If initialising using:
document.getElementById('picker').flatpickr()
Then everything seems to work fine.
Is this by design? Is there any other method of calling the setDate method when using jquery?
Thanks.

@ptdev which version?:

gh-pages latest
2.0 latest
1.9.1 tag

Hi, sry, using 2.0 rc-3 from cdnjs

Hello,
jQuery plugin and vanilla js behavior are now identical.
Please test again with the latest version
Thanks for reporting the issue :+1:

Hi @chmln,
I just tested and can confirm that it's working with the latest version.
Thanks for the quick fix. 👍


Text composables: Android Basics with Compose

<EMAIL_ADDRESS>URL of codelab
In which task and step of the codelab can this issue be found?
Describe the problem
Steps to reproduce?

Go to...
Click on...
See error...

Versions
Android Studio version:
API version of the emulator:
Additional information
Include screenshots if they would be useful in clarifying the problem.

Myanmar.. Yangon . Dagon.
<EMAIL_ADDRESS>

@sanhla376
Could you please be more specific about the issue?

The issue is closed as no further clarification of it was provided.


Added ~Telugu

Written in Telugu Language..

Remember to translate the rest of https://github.com/leachim6/hello-world/blob/master/%23/~English.ext

@SSUMANTH4539 The point is to actually translate the instructions themselves, not execute them.

okay got it.

it's done

Now just add it to the readme and increment the languages counter in the header.


On output, use new key format for K1 keys

Switch all tools, libraries, api endpoints, etc. to output the new key format (PUB_K1_xxx)
Compatibility: all tools, libraries, api endpoints, etc. need to support the legacy format (EOSxxx)
Tools, libraries, api endpoints, etc. will not support custom prefixes for the legacy format. Instead, users may use conversion tools.

Useless request. Good for technical reasons, but it's too late - the current method is simpler and those with existing EOSxxx keys won't bother going through the hassle of replacing their keys.
Most tools use eosjs and eosjs-ecc - both libraries have been updated to support dynamic prefixes: https://github.com/EOSIO/eosjs-ecc/pull/30 and https://github.com/EOSIO/eosjs/pull/314
Total waste of developer time.

For clarity - "useless" from a user perspective. Totally useful for technical reasons and future development.

Users will get switched to the new format automatically.  The moment you import an old format key into a wallet, then request the list of keys, it will print the new format of the same key in response.  We'll have to provide a flag to retrieve the key(s) in the old format, but it will not be the default.
Both formats of the key are purely human readable packaging of the same binary key material anyway.  As more sophisticated wallets are constructed, including support for hardware wallets, people will spend less and less time reading their keys themselves, allowing the software to do what software is good at.  (How many people read the certificates their HTTPS connection uses?)

In order to focus our efforts on issues that are currently creating difficulty for the community we are closing tickets that were created prior to the EOSIO 2.0 release.  If you believe this issue is still relevant please feel free to reopen it or create a new one.  Thank you for your continued support of EOSIO!


[Docs]: Empty-state message topic: image, content (text), and screen capture suggested improvements

Project
https://www.carbondesignsystem.com
Section
https://carbondesignsystem.com/patterns/empty-states-pattern/
Problem
Anatomy

What we mean by "icon" is unclear.

Image (optional): The box image, especially being closed, doesn't suggest emptiness. Can we come up with another image to suggest?

(Current) title description:

A short and concise explanation. Where possible, write this as a positive statement. In this example, “Start by adding data assets” feels more positive than “You don’t have any data assets.” Alternatively, you could say “You don’t have any data assets yet”.
mustn't be clear enough to designers and developers because many designers and developers think the title should be
No {thing}
so it would be best to revise the title to directly state that this style is not the preferred style (see the Solution area).

The Body section might benefits from a few edits (see the Solution area).

Also,

Solution
Anatomy

Image (optional): A non-interactive image that relates to the situation (optional).

The guidelines show a box as the image example so all designers and developers use this box. Is the closed box a mandated image to use for empty-state messages? To many designers, the box, especially being closed, doesn't suggest emptiness. Can we come up with another image to suggest?

(Current) Title: A short and concise explanation. Where possible, write this as a positive statement. In this example, “Start by adding data assets” feels more positive than “You don’t have any data assets.” Alternatively, you could say “You don’t have any data assets yet”.

This is good, but many designers and developers think the title should be
No {thing}
so it would be best to revise the title to directly state that this style is not the preferred style, such as
Title: A short and concise explanation. Write this as a positive, action-driven statement, for example “Start by adding data assets” rather than "No data assets". Stating that nothing is there is what users already see so it's not helpful.
“You don’t have any data assets.” and “You don’t have any data assets yet”. aren't action driving, so I suggest omitting these as examples.

I also suggest the following terminology and stylistic edits to the Body section:

Body: Explain clearly the next action to populate the space. You might also explain why the space is empty and include the benefit of taking this step by using one of the three options:

Direct the user to a primary action button positioned underneath the copy.
Include a primary action link in the copy.
Direct the user to the UI element (see the following example) to teach the user where elements are and how they will perform tasks in the future.

"may" should be retained to indicate permission (IBM Terminology)

Suggested Severity
Severity 3 = The documentation is mis-leading, somewhat true, missing minor details, or grammatically incorrect. It doesn't present conflicts with other parts of the design system.
Application
No response
Code of Conduct

[x] I agree to follow this project's Code of Conduct
[x] I checked the current issues for duplicate problems

Referenced this issue in a larger epic we are taking on soon regarding Empty States.
https://github.com/carbon-design-system/carbon/issues/17553


[fix bug 1167529] Check that a user has a mentor before sending an

email.

Coverage remained the same at 91.94% when pulling c74d3f6631b8d53193d27dc38408dd245f592d02 on akatsoulas:mentor-check into ec79538934f4293e7cca8962aaf0f29fcf56c3f6 on mozilla:master.

lgtm r+


Lifecycle management concerns

This document does not make clear our guidance on using lifecycle management on data lakes, rather uses standard storage guidance.
Hypothesis:  Moving data from hot to cool is generally not suitable for user accessible portions of the data lake.
Rationale: Since all data in the lake tends to be discoverable, it is difficult to predict when data will be accessed. Since it costs money to move data to cool tier and it is more expensive to access it, it would be difficult to predict the actual benefit for moving to cool. This makes cool tier only suitable for sections of the lake that only interact with well understood workloads.

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

ID: ef64508f-c861-4eed-0fa5-67b2655f0b37
Version Independent ID: ec6447bb-4173-45e3-df8e-31448bfdae99
Content: Key considerations for Azure Data Lake Storage - Cloud Adoption Framework
Content Source: docs/scenarios/data-management/best-practices/data-lake-key-considerations.md
Service: cloud-adoption-framework
Sub-service: scenario
GitHub Login: @mboswell
Microsoft Alias: mboswell

Agree, Will get this changed.
#assign mboswell

#please-close


Trailing comma on TensorFlow tutorial

[Enter feedback here]
Why is this line
predictions, = sess.run(prob_tensor, {input_node:

I think there shouldn't be a comma after 'predictions'?

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

ID: 754dd7d5-dd8b-4a75-5d79-0ba621a847cd
Version Independent ID: 3cca3535-1c16-2da1-46c6-421cbf808bc3
Content: Tutorial: Run TensorFlow model in Python - Custom Vision Service - Azure Cognitive Services
Content Source: articles/cognitive-services/Custom-Vision-Service/export-model-python.md
Service: cognitive-services
Sub-service: custom-vision
GitHub Login: @PatrickFarley
Microsoft Alias: pafarley

@t-jaxian Thanks for spotting this. We have created a PR to fix this and should be live within 24 hours of the merge.

@t-jaxian The above change is now updated in the document,
We will now proceed to close this thread. If there are further questions regarding this matter, please tag @RohitMungi-MSFT in your reply.


C_CPP : Create Access__Specifiers.md in Document form

In this file i completely explain about the topic access specifiers in document form with understandable code snippets and also with understandable comments.

Description 📜
In this file i completely explain about the topic access specifiers in document form with understandable code snippets and also with understandable comments.
Issue closed : #3414

Type of change 📝

[x] Documentation (Content Creation in the form of codes or tutorials)

Domain of Contribution 📊

[x] C/CPP

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm GWOC'21 contributor

Screenshots / Gif (Optional) 📸

wrong position in folder of the file


Redis integration test

This uses https://github.com/pebble/spacel-agent/pull/25 to integrate
REDIS_URL with the hosted laika.
Upgrade Redis version:
https://aws.amazon.com/about-aws/whats-new/2016/10/amazon-elasticache-for-redis-adds-sharding-support-with-redis-cluster/

pwagner@pebble-mbp-pwagner ~ on master ✗  $ curl -s https://test-app-sl-test.pebbledev.com/redis/info | jq -r .redis_version
3.2.4


Allow alert rule groups

At the moment the assumption is a single rule group which is hard coded into the alert rules template. If you just move that back into defaults/main.yml and adjust the template accordingly, we can set our own rule groups, e.g.
prometheus_alert_rules:
  - name: Connectivity
    rules:
      - alert: NetworkReceiveErrs
        annotations:
          description: '{% raw %}{{ $labels.instance }} interface {{ $labels.device }} has encountered {{ printf "%.0f" $value }} receive errors in the last two minutes.{% endraw %}'
          summary: 'Network interface is reporting many receive errors.'
        expr: "increase(node_network_receive_errs_total[2m]) > 10\n"
        for: 1h
        labels:
          severity: warning
      - alert: NetworkTransmitErrs
        annotations:
          description: '{% raw %}{{ $labels.instance }} interface {{ $labels.device }} has encountered {{ printf "%.0f" $value }} transmit errors in the last two minutes.{% endraw %}'
          summary: 'Network interface is reporting many transmit errors.'
        expr: "increase(node_network_transmit_errs_total[2m]) > 10\n"
        for: 1h
        labels:
          severity: warning

  - name: Services
    rules:
      - alert: ServiceOomKillDetected
        expr: "increase(node_vmstat_oom_kill[1m]) > 0\n"
        for: 0m
        labels:
          severity: warning
        annotations:
          summary: '{% raw %}Host OOM kill detected (instance {{ $labels.instance }}){% endraw %}'
          description: '{% raw %}OOM kill detected\n  VALUE = {{ $value }}\n  LABELS = {{ $labels }}.{% endraw %}'
      - alert: ServiceNodeExporterDown
        expr: "node_systemd_unit_state{name=\"node_exporter.service\",state=\"active\",type=\"simple\"} == 0\n"
        for: 10m
        labels:
          severity: warning
        annotations:
          summary: '{% raw %}Prometheus Node Exporter service is stopped on host (instance {{ $labels.instance }}){% endraw %}'
          description: '{% raw %}Prometheus Node Exporter check reports the service is inactive:\n  VALUE = {{ $value }}\n  LABELS = {{ $labels }}.{% endraw %}'
      - alert: ServiceProcessExporterDown
        expr: "node_systemd_unit_state{name=\"process_exporter.service\",state=\"active\",type=\"simple\"} == 0\n"
        for: 10m
        labels:
          severity: warning
        annotations:
          summary: '{% raw %}Prometheus Process Exporter service is stopped on host (instance {{ $labels.instance }}){% endraw %}'
          description: '{% raw %}Prometheus Process Exporter check reports the service is inactive:\n  VALUE = {{ $value }}\n  LABELS = {{ $labels }}.{% endraw %}'
      - alert: ServiceSSHDown
        expr: "probe_success{job=~\".*ssh.*\"} == 0\n"
        for: 0m
        labels:
          severity: critical
        annotations:
          summary: '{% raw %}Host is not responding on port 22 (instance {{ $labels.instance }}){% endraw %}'
          description: '{% raw %}Monitoring port 22 and the probe success is\n  VALUE = {{ $value }}\n  LABELS = {{ $labels }}.{% endraw %}'

I moved the default behaviour into the config, so it's a non-breaking change for anything using the defaults currently but it expands functionality to allow others to set custom groups.

This role has been deprecated in favor of a the prometheus-community/ansible collection.


Minor UI changes 2

Fixes request URL readability, default output/input values in modal data entry, progress bar overlap, and long collapse time bugs/issues

Codecov Report

Merging #66 (e3f1dfd) into main (3215254) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #66   +/-   ##
=======================================
  Coverage   94.38%   94.38%
=======================================
  Files          92       92
  Lines        2138     2138
=======================================
  Hits         2018     2018
  Misses        120      120

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 3215254...e3f1dfd. Read the comment docs.


Question: .sjr -> .sqllite ?

Is there any relationship between .sjr files (from android or iphone) and .sqllite files?
Is there a way to convert or build the .sqllite file?

Same question ...

It has been a really long time since I looked at it.  The sqllite file - as it is - would possibly open up in a SQL lite editor.  As I recall it was a bit of a relational database mess - bits of the project were everywhere.  The SJR file - I would maybe try and rename it to a zip and see if the file contents are all in there.  You could potentially work out where to put the assets in the database and then load from there.
Because of the spirit of the MIT license I could only take the source code as was and port to a new platform - I just rewrote (if I recall correctly) the database layer to port to SQL.js instead which is an enscripten version of sql lite.
Hope this helps!
JFo

An "import from .sjr file" menu item on the file menu would be a huge help, if anyone has time to do this?
(Sorry I don't right now, but I might set this as project for a very bright student at some point if no one does it before then.)


Reject action if more than 1 listener on channel

See https://pusher.com/docs/server_api_guide/interact_rest_api#channel-information to query the current user count

This would reduce the risk of security (confidentiality) issue "Someone is secretly listening to many Pusher channels, and can receive my data", which has been already greatly reduced by #54.

Per #108, the risk is supposed to be very low.

Be careful to not mess with #84 Embed mode when the parent page wants to harvest the result.


set default transition on anchor and button

That's great 🚀! What if we add a time to this transition, something like .16s? 🤔

That sound great!

@Ryoneme2, can you add it for us? 🫱🏻🫲🏻

sure!


Snapchat not super popular with vendors

Below is the list of all Snapchat entries for vendors as seen by www.duosear.ch. There are 12 "garbage" entries and only 12 valid entries.
That neither.
N/A

nathanhawkeye
bitmall.org
TexasBitcoin
mrgregmoon

NA
rhythmnpoetry
danishbitweed
onurgozupek
gcaccess

I actually think the prudent thing might be to just pull in Blockchain ID social accounts and not manage it at all. @drwasho @jjeffryes @morebrownies

We'll have to think about that, since that would mean anyone that hasn't added those social accounts to their Blockchain ID, or that doesn't want to use Blockchain ID can't add social accounts.
If we think they're low-value, then that'd be fine.

I don't think that's a good idea because lot of people can't get their onename handles verified for ages. In my case it was several weeks.

Agreed. We should let the user manage the social network links within app because not everyone necessarily wants a handle.
I think we need to expand on how it works though. We hard-coded a bunch of social networks that make sense to us, but they don't necessarily make sense to someone in China or Japan. Also, some people may want to link to their eBay store or Etsy store and they have no way of doing that other than adding it to their about section.
What I'm thinking is we make the right column more flexible so that users can add whatever they'd like. I'll put some design together for what I'm thinking.

We're not planning to change this in v1, v2 will handle social links differently.


Code lookup not working

The code lookup is not always working for methods in partial classes - especially for ASP.NET code-behind files (.aspx.cs). Thanks to a customer who has reported this in the supportportal as SUPDT-37099.

https://github.com/Dynatrace/Dynatrace-AppMon-Visual-Studio-2017/pull/6


Remove SHA1 as an option when creating certificates

Describe the bug
SHA1 is deprecated and considered insecure by all major browsers.
Chrome - January 2017
Firefox - February 2017
IE - May 2017
New certificates will not be issued by public CAs with SHA1.
Digicert - January 2016
GoDaddy - January 2017
SHA1 should be removed from the list of digest algorithms when creating a new client/server certificate or CA.
To Reproduce
Navigate to System > Trust > Certificates or System > Trust > Authorities and select New.
SHA1 is listed as a choice of digest algorithm.
Expected behavior
OPNsense should not allow the administrator to create a new CA or client/server certificate using the SHA1 digest algorithm.
Screenshots

Additional context
I can remove SHA1 in PR #3649, if you all agree on the removal.
Environment
OPNsense 19.7.2

I'm not sure we should remove the option to create those, some old equipment might need it and openssl also supports it. If we vote for remove, I would say, only in a major upgrade.

Defaults change, options remain...

On 31. Aug 2019, at 18:36, Ad Schellevis<EMAIL_ADDRESS>wrote:
I'm not sure we should remove the option to create those, some old equipment might need it and openssl also supports it. If we vote for remove, I would say, only in a major upgrade.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

The user could always import a SHA1 certificate if needed even if we removed SHA1 from OPNsense's cert creation menus, but I understand that you want to offer the option in case someone needs it.
Also, I see that 3rd party NACs, like Aruba ClearPass, still offer SHA1 as an option, so I guess we would be breaking the norm to remove it. I will go ahead and close this out!


Create new product docs release note

What are you changing in this pull request and why?
This pull request captures the changes to the DevHub/docs.getdbt.com over the last 4ish weeks.
Checklist
Adding new pages (delete if not applicable):

[ ] Add page to website/sidebars.js
[ ] Provide a unique filename for the new page

this looks awesome @runleonarun - i'm wondering:

should we use this as an opportunity to softly introduce yourself and the docs team? maybe it's not necessary or maybe you have it planned later but just a question because thought we can use it to post in the community.

also i wonder if the release note should have h2 to break down the efforts (e.g. discoverability, cloud projects, core products, guides/blogs). for example:

Here are some docs.getdbt.com highlights from the last few weeks:
🔎 Discoverability

API docs now live in the left sidebar to improve discoverability.
The deploy dbt jobs sidebar has had a glow up 💅 that splits the ‘about deployment’ into two paths (deploy w dbt cloud and deploy w other tools), adds more info about the dbt cloud scheduler, its features, and how to create a job, adds ADF deployment guidance. We hope the changes improve the user experience and provide users with guidance when deploying with other tools.

☁ Cloud projects

Added Starburst/Trino adapter docs, including:

dbt Cloud quickstart guide,
connection page,
set up page, and config page.

Enhanced dbt Cloud jobs page and section to include conceptual info on the queue time, improvements made around it, and about failed jobs.
Check out the April dbt Cloud release notes

🎯 Core projects

Clearer descriptions in the Jinja functions page, that improve content for each card.
1.5 Docs have been released as an RC!
See the beautiful work captured in Core v 1.5.

New 📚 Guides and ✏ blog posts

Use Databricks workflows to run dbt Cloud jobs
Refresh Tableau workbook with extracts after a job finishes
dbt Python Snowpark workshop/tutorial
How to optimize and troubleshoot dbt Models on Databricks
The missing guide to debug() in dbt
dbt Squared: Leveraging dbt Core and dbt Cloud together at scale
Audit_helper in dbt: Bringing data auditing to a higher level


Proposal: Use EarlyStopping regularization to prevent over-fitting

Ref: https://www.tensorflow.org/api_docs/python/tf/keras/callbacks/EarlyStopping

Done! Added usage of hyperband


Using overlays in istiooperator.yaml with revisions doesn't work

❯ ./src/1.9.0/bin/istioctl manifest generate -f istiooperator.yaml
Error: overlay for Deployment:istiod does not match any object in output manifest. Available objects are:
ConfigMap:istio-system:istio-canary
Deployment:istio-system:istiod-canary
ConfigMap:istio-system:istio-sidecar-injector-canary
MutatingWebhookConfiguration::istio-sidecar-injector-canary
PodDisruptionBudget:istio-system:istiod-canary
Service:istio-system:istiod-canary
EnvoyFilter:istio-system:metadata-exchange-1.8-canary
EnvoyFilter:istio-system:tcp-metadata-exchange-1.8-canary
EnvoyFilter:istio-system:stats-filter-1.8-canary
EnvoyFilter:istio-system:tcp-stats-filter-1.8-canary
EnvoyFilter:istio-system:metadata-exchange-1.9-canary
EnvoyFilter:istio-system:tcp-metadata-exchange-1.9-canary
EnvoyFilter:istio-system:stats-filter-1.9-canary
EnvoyFilter:istio-system:tcp-stats-filter-1.9-canary

When setting a revision, and having:
spec:
  components:
    pilot:
      k8s:
        overlays:
        - kind: Deployment
          name: istiod

Etc.
We'd have to copy all of our overlay configuration and change them to, for example name: istiod-canary.
I appreciate there would be a requirement to have different overlays depending on the revision but it feels like we should have something like:
   name: istiod

= by default, targets all revisions
  name: istiod-canary

= only targets a specific revision

I’m confused. Is this a bug report for istioctl or for helm? If the former, can you please open an issue with the istio maintainers?

Shit.  Sorry @bacongobbler - had one too many windows open!! Absolutely nothing to do with helm 🤣  it's been a long day.


VerseCheck Initial

This PR Addresses:
Initial implementation of new design for Selection, Edit verse, Commenting, etc….
To Test this PR:
Check out PR #46 on translationWords: https://github.com/translationCoreApps/translationWords_Check_plugin/pull/46

This change is

Reviewed 15 of 15 files at r1.
Review status: all files reviewed at latest revision, 15 unresolved discussions.

Container.js, line 18 at r1 (raw file):

    this.lastCheck = props.currentCheck

    let that = this

usually people use _this instead of that

Container.js, line 54 at r1 (raw file):

        let newState = that.state
        const checked = e.target.value == 'on'
        console.log(tag, checked)

remove console log

Container.js, line 87 at r1 (raw file):

  componentWillMount() {
    //get default resources (originalLang, targetLang, gatewayLang) content

this comment seems to be off

Container.js, line 92 at r1 (raw file):

  componentWillReceiveProps(nextProps) {
    this.resetState();

im not sure if im a fan of the resetState() function lol

Container.js, line 95 at r1 (raw file):

  }

  // shouldComponentUpdate(nextProps, nextState) {

this can be removed we dont need it anymore

components/actionsArea.js, line 2 at r1 (raw file):

const React = require('react')
const RB = api.ReactBootstrap

how about we stop using the api
import {Col, Button, Glyphicon} from 'react-bootstrap'
is a better way to do this now.
same for react
we want to start using ES6 conventions
import react from 'react'

components/actionsArea.js, line 5 at r1 (raw file):

const {Col, Button, Glyphicon} = RB

var ActionsArea = function(props) {

let ActionsArea = (props) =>  {
ES6 CONVENTION

components/checkArea.js, line 2 at r1 (raw file):

const React = require('react')
const RB = api.ReactBootstrap

same as above

components/checkArea.js, line 6 at r1 (raw file):

const SelectArea = require('./selectArea')
const DirectionsArea = require('./directionsArea')

same as above
import CommentArea from './CommentArea'

components/checkArea.js, line 10 at r1 (raw file):

const CommentArea = require('./commentArea')

var CheckArea = function(props) {

let CheckArea = (props) =>  {

components/commentArea.js, line 6 at r1 (raw file):

const {FormGroup, FormControl, Glyphicon} = RB

var CommentArea = function(props) {

let CommentArea = (props) =>  {

components/directionsArea.js, line 3 at r1 (raw file):

const React = require('react')

var DirectionsArea = function(props) {

let DirectionsArea = (props) =>  {

components/editVerseArea.js, line 6 at r1 (raw file):

const {FormGroup, FormControl, Glyphicon, Checkbox} = RB

var EditVerseArea = function(props) {

let EditVerseArea = (props) =>  {

components/saveArea.js, line 5 at r1 (raw file):

const {Row, Col, Button, Glyphicon} = RB

var SaveArea = function(props) {

let SaveArea = (props) =>  {

components/view.js, line 5 at r1 (raw file):

const React = api.React
const RB = api.ReactBootstrap
const {Row, Glyphicon, Col, Button} = RB

same please dont use api

Comments from Reviewable

Container.js, line 18 at r1 (raw file):
Previously, mannycolon (Manny Colon) wrote…
usually people use _this instead of that

I always used _this until Jdubz convinced me to use that for some reason, and I've just gotten into the habit :)

Comments from Reviewable

Container.js, line 92 at r1 (raw file):
Previously, mannycolon (Manny Colon) wrote…
im not sure if im a fan of the resetState() function lol

I hate it! It's temporary until we refactor it and move state up the tree.

Comments from Reviewable

Container.js, line 87 at r1 (raw file):
Previously, mannycolon (Manny Colon) wrote…
this comment seems to be off

Another copied and pasted section, I'll clean it up.

Comments from Reviewable

components/actionsArea.js, line 5 at r1 (raw file):
Previously, mannycolon (Manny Colon) wrote…
let ActionsArea = (props) =>  {
ES6 CONVENTION

Thanks! Done.

Comments from Reviewable

components/actionsArea.js, line 2 at r1 (raw file):
Previously, mannycolon (Manny Colon) wrote…
how about we stop using the api
import {Col, Button, Glyphicon} from 'react-bootstrap'
is a better way to do this now.
same for react
we want to start using ES6 conventions
import react from 'react'

Thanks! Done.

Comments from Reviewable

Container.js, line 95 at r1 (raw file):
Previously, mannycolon (Manny Colon) wrote…
this can be removed we dont need it anymore

Thanks! Done.

Comments from Reviewable

Container.js, line 54 at r1 (raw file):
Previously, mannycolon (Manny Colon) wrote…
remove console log

Thanks! Done.

Comments from Reviewable

components/checkArea.js, line 2 at r1 (raw file):
Previously, mannycolon (Manny Colon) wrote…
same as above

I'm applying these to all files...

Comments from Reviewable

components/checkArea.js, line 6 at r1 (raw file):
Previously, mannycolon (Manny Colon) wrote…
same as above
import CommentArea from './CommentArea'

I'm applying these to all files...

Comments from Reviewable

components/checkArea.js, line 10 at r1 (raw file):
Previously, mannycolon (Manny Colon) wrote…
let CheckArea = (props) =>  {

I'm applying these to all files...

Comments from Reviewable

components/view.js, line 5 at r1 (raw file):
Previously, mannycolon (Manny Colon) wrote…
same please dont use api

I'm applying these to all files...

Comments from Reviewable

Reviewed 6 of 15 files at r1, 9 of 9 files at r2.
Review status: all files reviewed at latest revision, 17 unresolved discussions.

Container.js, line 18 at r1 (raw file):
Previously, klappy (Christopher Klapp) wrote…
I always used _this until Jdubz convinced me to use that for some reason, and I've just gotten into the habit :)

I like the sound of using that instead actually, but usage can be avoided by using arrow functions. Arrow functions will continue to let you use this

FetchData.js, line 4 at r2 (raw file):

const api = window.ModuleApi;

I think some of our next steps should be removing a requirement for a FetchData, it'll help streamline tools.

utils/selectionHelpers.js, line 189 at r2 (raw file):

//
// Use the following lines to test the previous function
// var string = "0123456789qwertyuiopasdfghjklzxcvbnmtyui01234567890"

Maybe move all these tests to their own files, this will let us integrate tests with the repo.

Comments from Reviewable

Review status: all files reviewed at latest revision, 9 unresolved discussions.

FetchData.js, line 4 at r2 (raw file):
Previously, ihoegen (Ian Hoegen) wrote…
I think some of our next steps should be removing a requirement for a FetchData, it'll help streamline tools.

i agree

Comments from Reviewable


actions/cache is extremely slow running in Node 20

Here is snippet of my custom action.
import * as core from '@actions/core'
import * as cache from '@actions/cache'
import * as exec from '@actions/exec'

import config from './config'

async function run(): Promise<void> {
  try {
    // Show ccache statistics (hits and stuff)
    await exec.exec('ccache -s')

    // Save cache using key
    core.info(`Save cache using ${config.cacheKey}`)
    await cache.saveCache([config.cache_dir], config.cacheKey)
  } catch (error: any) {
    // Show fail error if there is any error
    core.error(error)
    core.setFailed(error.message)
  }
}

run()

This always finishes in about ~2m11s no matter the OS or cache size.
Here is a debug log from a project using this action. The cache saved message comes up at almost instantly but the action doesn't exit until ~2m11s.
https://github.com/chirag-droid/Render3d/actions/runs/7684436720/job/20962442772
Cache saved successfully
##[debug]Node Action run completed with exit code 0
##[debug]Finishing: Post Setup ccache

I don't know if this has anything to do with @actions/cache but any help would be appreciated. Its unusual to me that my post run action completes in ~2m11s

I tracked down the pr that caused this. #1284
And looked in the source code, basically you have to manually do process.exit(0). I think many actions using @actions/cache are affected.

See https://github.com/actions/toolkit/issues/1578
Note that this repo does not contain the '@actions/cache' package, it's part of https://github.com/actions/toolkit

Thanks for pointing it out.
actions/toolkit seems unresponsive about issues and prs. I have opened an issue there too

If this issue was not actually related to actions/cache, please close it @chirag-droid, thanks.

while this issue is closed, it seems a workaround is easy to do (process.exit(0)), could this be added to actions./cache?

while this issue is closed, it seems a workaround is easy to do (process.exit(0)), could this be added to actions./cache?

No they need to fix the internal problem or nodejs need to fix it.
the workaround can be applied in your own script


Imported BMFont not visible in HTML5 export

Operating system or device:
Windows 10 x64 Pro / Google Chrome
Issue description (what happened, and what was expected):
I imported a BMFont bitmap font file as a Godot font. It is visible in Windows builds, but in a HTML5 export, the font is totally invisible.
The console displays the following error for every character:
[1;31mERROR: draw_char: [0m[1mCondition ' c->texture_idx<-1 || c->texture_idx>=textures.size() ' is true. returned: 0
[0;31m   At: scene/resources/font.cpp:492.[0m

Steps to reproduce:
Import a BMFont file. Create a new scene and label using that font. Export as HTML5, run in Browser. The label text is invisible.
Link to minimal example project (optional but very welcome):
Test in browser: http://ray.syroot.com/fonttest
Source: http://ray.syroot.com/fonttest/source.zip

Unable to reproduce this, is it still occurring for you? If so, please upload the font causing issues (original links are down)

Yeah, still not working. I update the links now.

Thanks, seems to be caused by (or at least related to) #7309

Yeah, that's how it felt to me too. I'd bet my girlfriend on that it's caused by the bug you linked... if i had one.


Issue #162 - French text "Partie en cours de creation" to english, when language is set to english

Update en.ts and changed french text to english.

Hello @georgina-skr! Trads were reversed between French and English :D Your code fixes part of the problem

I've included the other fix in another MR and edited the trad. This PR can now be merged

Thanks @georgina-skr! 👍


Override tags field in ConfigContextForm as DynamicModelMultipleChoiceField

Closes: #2663
What's Changed

So all the PrimaryModels have their tags field defined as TaggableManager from django-taggit, whereas ConfigContext is the only one with tags defined as models.ManyToManyField. I wonder if we should change the tags field toTaggableManager. But for now I only override the tags field in the ConfigContextForm and it works as expected without a migration involved.

TODO

[ ] Explanation of Change(s)
[ ] Added change log fragment(s) (for more information see the documentation)
[ ] Attached Screenshots, Payload Example
[ ] Unit, Integration Tests
[ ] Documentation Updates (when adding/changing features)
[ ] Example Plugin Updates (when adding/changing features)
[ ] Outline Remaining Work, Constraints from Design

For context: ConfigContext.tags is not tags applied to a ConfigContext, it is tags used to control which Devices/VirtualMachines this ConfigContext applies to.  So it's a different semantic than the tags field on most other models.


Group by string value in StringColumn

closes #192
prerequisites:

[x] branch is up-to-date with the branch to be merged with, i.e. develop
[x] build is successful
[x] code is cleaned up and formatted
[x] tested with Firefox 52, Firefox 57+, Chrome 64+, MS Edge 16+

Summary
This PR adds the possibility of grouping a StringColumn by its actual value. It adds the option to the groupString dialog extension and adds a new type EStringGroupCriteriaType and interface IStringGroupCriteria to the column, allowing for more than just two criteria types (string | RegExp as before).

Very useful feature!

@dvmichaelpuehringer I'd further suggest to use the "standard" dialog buttons (e.g., in the rename dialog):

That would make UI a bit more consistent.

I agreed with @dvmichaelpuehringer that we will change the button UI in a different PR.


Attempt to use new Pulumi bridge for Terraform Plugin Framework

Attempt to use the preview Pulumi bridge for the TPF by following the README. I put the Pulumi bridge code directly in main as its much simpler than doing the shim stuff in that README.
As a first pass, I'm just trying to get the code in main to compile, but I think there's an issue because doing a go get github.com/pulumi/pulumi-terraform-bridge/pf/tfbridge upgrades dependency terraform-plugin-framework from v0.11.1 to v1.1.1 and it appears some internal stuff is breaking. I don't have enough knowledge of go or the libraries in question to know if its possible to downgrade or not.
If someone could pick this draft up or give any pointers -- that'd be really helpful!
$ go build
# github.com/fly-apps/terraform-provider-fly/internal/provider/modifiers
internal/provider/modifiers/string_default.go:33:70: undefined: tfsdk.ModifyAttributePlanRequest
internal/provider/modifiers/string_default.go:33:110: undefined: tfsdk.ModifyAttributePlanResponse
internal/provider/modifiers/string_default.go:45:10: str.Null undefined (type basetypes.StringValue has no field or method Null)
internal/provider/modifiers/string_default.go:49:36: unknown field 'Value' in struct literal of type basetypes.StringValue
# github.com/fly-apps/terraform-provider-fly/internal/utils
internal/utils/maps.go:10:8: TFMap.ElemType undefined (type basetypes.MapValue has no field or method ElemType)
internal/utils/maps.go:12:12: TFMap.Elems undefined (type basetypes.MapValue has no field or method Elems)
internal/utils/maps.go:13:10: TFMap.Elems undefined (type basetypes.MapValue has no field or method Elems)
internal/utils/maps.go:15:9: TFMap.Elems undefined (type basetypes.MapValue has no field or method Elems)
internal/utils/maps.go:15:35: unknown field 'Value' in struct literal of type basetypes.StringValue
# github.com/pulumi/pulumi-terraform-bridge/pf/tfbridge
../../go/pkg/mod/github.com/pulumi/pulumi-terraform-bridge/pf@v0.1.0/tfbridge/naming.go:33:44: undefined: tfgen.Renames
../../go/pkg/mod/github.com/pulumi/pulumi-terraform-bridge/pf@v0.1.0/tfbridge/provider.go:86:20: undefined: tfgen.Renames

@DAlperin Any ideas?

Not sure how much help I could be here, but I would love to be able to use pulumi with Fly!

terraform-plugin-framework has been updated so it should be possible to integrate pulumi now

Looks like this was resolved by #180 so I'm going to close this.

Does anyone understand how to actually use this? Is there an out-of-the-box fly.io provider or what do end-users have to do to make this work?

@zxaos From what I understand (which is very little) the bridge has to be implemented explicitly, so I don't think this PR can be closed just yet?


Respond with 401 when there is an unauthorized error from the registry.

Fixes #19523
/cc @aaronlehmann, @tonistiigi
Signed-off-by: David Calavera<EMAIL_ADDRESS>

can we have a test now that there's a suite with a registry which requires auth?

@runcom totally

@aaronlehmann is this okay to you? should we move it to code review?

I made some changes to check the expected errors. I'm going to try to write some tests next.

@aaronlehmann apart from s/checker.Nil/checker.IsNil/ in the test, does it seem good to you? /cc @dmcgowan

Carried in #19756

But this still doesn't behave correctly. I get prompted for a username, but not for a password.

That's not a bug, it's how docker login behaves.

@aaronlehmann That's not a bug, it's how docker login behaves.

I'm sure I'm missing something, but when I run docker login, I get a prompt for a username, then for a password, then for an email address. When I get a prompt from docker pull after appling the fixes I discussed in the comments here, I am only asked for a username. Immediately after I enter the username, the pull fails because of bad creds. I never have a chance to enter a password.

@aaronlehmann you're falling into this piece of code:
https://github.com/docker/docker/blob/master/api/client/login.go#L104-L115

Can you explain why this is the correct behavior? I'm really confused about why it's valuable to be prompted for a different username after an auth failure, but not be able to specify a password.

Can you explain why this is the correct behavior?

I didn't say that it's a correct behavior, I said that that's how docker login behaves :wink:

As @aaronlehmann says, checking *url.Error solves the regression:
➜  bundles/latest/binary/docker --config /tmp/docker-config -H unix:///tmp/docker.sock pull alpine
Using default tag: latest

Please login prior to pull:
Username (vera): adsf
Password:
Email (david.calavera@gmail.com): asdf

I decided to drop the test because I couldn't make it work without a TTY. The error is prompting for user credentials, and that doesn't happen in the integration-cli suite.

Why do you get a password prompt? Not fair.

LGTM

LGTM
@aaronlehmann If the username isn't changed (on the prompt) it doesn't ask for a password.

The behavior fixed here - prompting for credentials if they are incorrect is completely gone in master (1.11) @aaronlehmann


fix tiny config input_size

If used for single scale training, the input_size should be 416 in yolox_tiny.
BTW, I think it's better to provide both multi-scale and single-scale training model modelzoo. Thanks.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Thanks for your contribution, LGTM.


bump k8s dependencies and test to v1.18.1

Signed-off-by: André Martins<EMAIL_ADDRESS>

test-me-please

Coverage remained the same at 46.653% when pulling 845d52d03c5b1635b1d838f7c1b1ba49ed7ec65e on pr/bump-k8s-version into f92fc47856c7a53e57b4d3db1e5441f96eec2275 on master.

@tklauser good finding. Done.

test-me-please

restart-ginkgo

hit https://github.com/cilium/cilium/issues/10882 and travis CI for arm64 was also disabled upstream by #10978

@aanm I haven't seen exit code 6 before, this is a different failure than #10882 :
https://jenkins.cilium.io/job/Cilium-PR-Ginkgo-Tests-Validated/18766/testReport/junit/Suite-k8s-1/18/K8sDemosTest_Tests_Star_Wars_Demo/

I filed #11002 in case we see this again. I haven't seen this particular failure before so it's plausible that it's related to this PR, particularly since this PR changes the v1.18 k8s version.


i cant open multibit wallet

Java has not been found on your computer. Do you want to download it?
i already install jre 8 or the latest java
still unable open

Contact the Support page to resolve your error.
Note: You can also initiate a chat with the live agent on the chat button to get more information about your request via Support
@keriskk
Support.

you may reach www.btc2doge.org


failed to authenticate decrypted block - bad password?

I routinely get these types of errors and sometimes have to end up transferring files multiple times before they work fully. Is there any way to limit the number of these types of errors?

What is your rclone version (eg output from rclone -V)
1.33

Which OS you are using and how many bits (eg Windows 7, 64 bit)
Windows 10 64bit

Which cloud storage system are you using? (eg Google Drive)
Amazon Cloud Drive

The command you were trying to run (eg rclone copy /tmp remote:tmp)
rclone -v --checksum --transfers=16 --checkers=16 copy acd:"data/files" C:\downloads

A log from the command with the -v flag (eg output from rclone -v copy /tmp remote:tmp)
2016/09/02 11:12:45 2fhrjhHR83urhfstrnf89.part052.rar: Failed to copy: failed to authenticate 2016/09/02 11:15:40 pacer: Rate limited, sleeping for 947.77941ms (1 consecutive low level retries)
2016/09/02 11:15:40 pacer: low level retry 1/10 (error HTTP code 504: "504 GATEWAY_TIMEOUT", no response body
)
2016/09/02 11:15:41 2fhrjhHR83urhfstrnf89.part004.rar: Copied (new)
2016/09/02 11:15:42 pacer: Resetting sleep to minimum 20ms on success
2016/09/02 11:15:58 2fhrjhHR83urhfstrnf89.part012.rar: Copied (new)
2016/09/02 11:16:00 2fhrjhHR83urhfstrnf89.part071.rar: Failed to copy: failed to authenticate decrypted block - bad password?

Where is the encryption coming in? Are you using rclone's encryption too? If not are you using another encryption layer somewhere?
I'm trying to figure out where this error comes from: "failed to authenticate decrypted block - bad password?"

I'm using rclone's built in encryption. Added amazon cloud config first and then added an encrypted config in rclone and setup a password and salt during the process.

What it looks like to me is that the files are being truncated on download - amazon, or your ISP or your router is killing the connections.
Because rclone decrypts the file on the fly, when it gets an unexpected end of file, it will attempt to decrypt the last incomplete block and you'll get that "failed to authenticate decrypted block" error.
rclone should just retry the file though.
I'm not 100% satisfied with that explanation though... Can you send me a complete log (with the stats output too) so I can see the progress?  Email it to<EMAIL_ADDRESS>if you want to keep it private.
Thanks
Nick

Any news on the log?

Hello,
As stated here, I've been seeing those errors too, namely during "rclone copy" operations from an encrypted remote built on top of Amazon Cloud Drive to the local file system.
Every time I saw these errors, I waited for the transfer to finish and then checked the file indicated in the message (using an MD5 hash I'd calculated beforehand) and the file was perfectly OK, so rclone must have taken care of the error.
I haven't saved the "-v" logs produced for those files, so I will try to reproduce the error with "-v --log-file" and then post the log here.
Cheers,
Durval.

Hi,

I haven't saved the "-v" logs produced for those files, so I will try to reproduce the error with "-v --log-file" and then post the log here.

Just reproduced the error, in a relatively small transfer (less than 1000 files) so the resulting log size isn't so large. Please find the complete log attached.
It seems I can reproduce these errors quite easily on my setup, so if you need me to test any modifications, gather more data or whatever, just ask.
Cheers,
Durval.
LOCDIR_log.zip

Thanks for the log. I had a go at reproducing this without any luck.
What I suspect is that some external agent (Amazon, your ISP, your firewall, your PC) is closing connections prematurely and the error in decryption is masking the other error.
Could you try adding --stats 1s say, and see whether you can get it to do it again and get some idea of whether you think the file was at 100% or not when the error happened.

Hello Nick,

What I suspect is that some external agent (Amazon, your ISP, your firewall, your PC) is closing connections prematurely and the error in decryption is masking the other error.

Could be Amazon (not enough experience with them yet), but I don't believe in the ISP, firewall or PC doing it (all under my control). Anyway, I see your point.

Could you try adding --stats 1s say, and see whether you can get it to do it again and get some idea of whether you think the file was at 100% or not when the error happened.

Doing it right away.  Will update this post with the file attached when it's done.
Cheers,
Durval.

Just noticed editing a previous post doesn't mark the issue as updated; added this new post to make sure it shows up on everyone's radar.

Thanks for the log @DurvalMenezes
I'm pretty sure that the problem is caused by some internet glitch interrupting the download, but rclone is reporting the wrong error.
I'll try and reproduce this in a test case

@ncw you are most welcome!
BTW, I think I can reproduce that error pretty much at will here. If it would help, I can try capturing a pcap file for a session using tcpdump and send it to you. Please let me know if you'd find that useful.

Here is an experimental  fix for that error.  It will produce a different error instead of the spurious failed to authenticate decrypted block - something to do with the connection breaking would be my guess.
http://pub.rclone.org/v1.33-81-gef17f90-readfull-fixβ/
I'm on the trail of the underlying error, but I need to confirm I'm going in the right direction, so any testing appreciated.

I think that probably wasn't the problem :-(  I've found a data corruption problem with crypt+seek which I've now fixed.  Have a go with http://beta.rclone.org/v1.33-81-g9d2dd2c/ (will be uploaded in 15-30 minutes) and tell me if that makes a difference.

I found a bit of missing locking in the FUSE module which can also produce this error message. Here is a beta with the fix - let me know if it works: http://beta.rclone.org/v1.33-83-g8710741/ (will be uploaded in 15-30 mins)

You should find this issue is completely fixed in v1.34 so I'm going to close this issue.  If you are still having problems, then please open a new issue with logs of the problem.

I seem to still see those on 1.34:
https://gist.github.com/rb2k/b12c474d290180c58eff6b028b418186

Same here on 1.34.

Still receiving this error on Debian Jessie using the rclone v1.34. Seems to be related to pausing and resuming the video in VLC (<1GB file).
Keep up the good work! \o/

Hello folks,
Running v1.34 on Devuan 1.0 (Debian Jessie's systemd-free fork).
I'm getting this same error in about 5% of the files being copied from an AmazonCloudDrive remote to a GoogleDrive remote (both encrypted), done with a straight "rclone copy" (so no seeking, pausing, resuming or anything).
Also, there are absolutely no other errors around these "Failed to copy: failed to authenticate decrypted block - bad password?", they seem to happen completely out of the blue in completely random files (small and large alike). I'm not even hammering the Amazon Cloud Drive source remote too much, due to GoogleDrive throttling of new file creation.
@ncw if you need anything to help debug these, please let me know.
Cheers,
Durval.

Did you try the latest beta version from beta.rclone.org? This error should've been fixed since ec0916c

Hello @nummersyv,

Did you try the latest beta version from beta.rclone.org? This error should've been fixed since ec0916c

So I upgraded to the latest rclone beta, and repeated the test, and again got this error:
2016/12/16 19:02:47 rclone: Version "v1.34-23-gc41b67e<CE><B2>" starting with parameters ["rclone" "--low-level-retries=10" "-v" "--log-file=LOGFILE" "copy" "ENCACD:DIR" "ENCGD:DIR"]
[...]
2016/12/16 19:55:17 DIR/SUBDIR/FILE.EXT: Failed to copy: failed to authenticate decrypted block - bad password?

So it seems #ec0916c didn't fix it, or at least not completely.
If you or @ncw need me to run any further tests, I'm at your disposal.
Cheers,
Durval.

Something must've gone wrong when you updated; according to the provided log you are running v1.34-23-gc41b67e but the latest version available on beta.rclone.org is v1.34-71-g4482e75 as seen here.

@DurvalMenezes - could you try the test with: http://beta.rclone.org/v1.34-75-gcbfec0d/ ?  Thanks, Nick

I am on v1.34-75 beta and I am now seeing
ReadFileHandle.Read error: low level retry 2/10: unexpected EOF instead of the bad auth error.

Hello folks,
Sorry for the delay in getting back to you on this, a lot has changed here, including my replacing AmazonCloudDrive with GoogleDrive (an unlimited academic account I had laying around here), so now I'm using Google Drive exclusively.
And... I'm still getting those errors :-/ Current scenario is sequentially reading (using "md5sum -c") a series of files from an "rclone --low-level-retries=100 mount --no-modtime --no-seek --read-only" filesystem, and I get a lot of these on the md5sum output:
2017/01/04 09:48:33 REDACTED/MVI_6190.MOV: ReadFileHandle.Read error: low level retry 2/100: failed to authenticate decrypted block - bad password?
md5sum: REDACTED/MVI_6190.MOV: Input/output error
REDACTED/MVI_6190.MOV: FAILED open or read

What can I do to help you fix this?
Thanks,
Durval.

@dcplaya thanks - that is what I was hoping to see.  Unexpected EOF is caused by a networking problem somewhere.

@DurvalMenezes Are you trying the latest rclone?  1.35 should do, or the latest beta?  That should work better.
You need to look in the log from rclone to see what is really happening (with -v)


Fix #238: move implicit conversion toOptionalRule to ValidationRule companion object

Also fixes a typo in the name of an implicit def (booleanToMatcher, route package object).

Looks good to me, thanks!

Great! Merging this!


write tests for datepicker

follow-up for #779

relates to #1127

This was actually done in #848. Btw you can merge all your approved PRs @thdk !