fix(Carousel): arrows & indicators in RTL

🔗 Linked issue
Resolves #1354
❓ Type of change

[ ] 📖 Documentation (updates to the documentation or readme)
[x] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description
This fixes most of the issues described in https://github.com/nuxt/ui/issues/1354. Only remaining issue is the active indicator styles are not applied correctly.
There's already an example in the playground that you can directly play with. playground/app.vue should be reverted to its initial state if this PR will be merged

Possible solution is requiring the user to pass a dir prop which we can reactively watch it for changes. But that adds a bit of overhead and Swiper for example don't need this
📝 Checklist

[x] I have linked an issue or discussion.
[ ] I have updated the documentation accordingly.

@MuhammadM1998 Can we not fix the active style indicator by using the multiplier inside currentPage computed?

Unfortunately not. refs in vue are not reactive so putting getComputedStyle(carouselRef.value) in a computed wont update when we change the dir (or any other attribute for that matter)

What should we do then? Not sure I want to merge this until everything's fixed.

You're right that's why I didnt make the PR. I can't think of a solution that automatically handles this. There's the dir solution mentioned in the PR description but don't know if that's okay with you

Not really a fan to add a dir prop as every component handles it automatically 😬

I thought so 😅 I'll convert this to draft then until one of us reaches a better solution

Closing this for now until we find a solution!


Issue #SB-9322 - Containerize the page content to match header width.

Related JIRA Tickets -

https://project-sunbird.atlassian.net/browse/SB-9322
https://project-sunbird.atlassian.net/browse/SB-9291 (Only filters UI changes, rest header comming from Prajwal's PR)

Changes done -

Containerized the UI ( Set max-width to the page so that UI do not break on big screens)
New Filters UI
Remove Sort button as mentioned in https://project-sunbird.atlassian.net/browse/SB-9291
Add a line between the cards carousel (left in the last release)

Before fix -

After fix -
Please watch this video
https://drive.google.com/file/d/1WLYMQNRftPt-5XNDGQz8C9qKPf_4Bpme/view?usp=sharing

@bvinayakumar Pls merge this pr.

@harishgilimi - Pls merge this pr.

@lakhanmandloi please resolve conflicts


Update block.sunappu.net size (now 193 KB)

That’s 5 KB saved!

This PR is:

[ ] Adding a new domain
[x] Updating existing domain size
[ ] Changing domain name
[ ] Removing existing domain from list
[ ] Website code changes (512kb.club site)
[ ] Other not listed

[x] I used the uncompressed size of the site
[x] I have included a link to the GTMetrix report
[x] The domain is in the correct alphabetical order
[x] This site is not a ultra lightweight site
[x] The following information is filled identical to the data file

I confirm that I have read the FAQ section, particularly the two red items around minimal pages and inappropriate content and I attest that this site is neither of these things.

[x] Check to confirm

- domain: block.sunappu.net
  url: https://block.sunappu.net/
  size: 193
  last_checked: 2022-04-16

GTMetrix Report: https://gtmetrix.com/reports/block.sunappu.net/rpMIiYoD/

Maybe I'm missing something, but the report says it's 4.67kb?

Maybe I'm missing something, but the report says it's 4.67kb?

Sorry, the link was wrong (other domain). I just edited it.


issue in use from pracmln in python

hello ....
I install pracmln with " pip install pracmln" but i could not use from it and see this error....please guide me .....
runfile('C:/Users/abbasi/.spyder/temp.py', wdir='C:/Users/abbasi/.spyder')
Traceback (most recent call last):
File "", line 1, in
runfile('C:/Users/abbasi/.spyder/temp.py', wdir='C:/Users/abbasi/.spyder')
File "C:\Users\abbasi\Anaconda2\lib\site-packages\spyder\utils\site\sitecustomize.py", line 705, in runfile
execfile(filename, namespace)
File "C:\Users\abbasi\Anaconda2\lib\site-packages\spyder\utils\site\sitecustomize.py", line 87, in execfile
exec(compile(scripttext, filename, 'exec'), glob, loc)
File "C:/Users/abbasi/.spyder/temp.py", line 7, in
from pracmln import MLN
File "C:\Users\abbasi\Anaconda2\lib\site-packages\pracmln_init_.py", line 21, in
from mln.base import MLN
File "C:\Users\abbasi\Anaconda2\lib\site-packages\pracmln\mln_init_.py", line 27, in
from base import MLN
File "C:\Users\abbasi\Anaconda2\lib\site-packages\pracmln\mln\base.py", line 27, in
from dnutils import logs, ifnone, out
File "C:\Users\abbasi\Anaconda2\lib\site-packages\dnutils_init_.py", line 10, in
from .signals import add_handler, rm_handler, enable_ctrlc
File "C:\Users\abbasi\Anaconda2\lib\site-packages\dnutils\signals.py", line 11, in
SIGKILL = signal_.SIGKILL
AttributeError: 'module' object has no attribute 'SIGKILL'

I get a related error after pip install pracmln
AttributeError: 'module' object has no attribute 'SIGCLD'

I solved this issue by replacing signals.py located in dnutils library with the following:
I am using windows 10 and python 3.5.4
`import signal as signal_
import threading
from collections import defaultdict
__usrreg = defaultdict(list)
__sysinit = defaultdict(list)
__systerm = defaultdict(list)
SIGINT = signal_.SIGINT
SIGTERM = signal_.SIGTERM
#SIGKILL = signal_.SIGKILL
SIGABRT = signal_.SIGABRT
#SIGALRM = signal_.SIGALRM
#SIGBUS = signal_.SIGBUS
#SIGCHLD = signal_.SIGCHLD
#SIGCLD = signal_.SIGCLD
#SIGCONT = signal_.SIGCONT
SIGFPE = signal_.SIGFPE
#SIGHUP = signal_.SIGHUP
SIGILL = signal_.SIGILL
SIGINT = signal_.SIGINT
#SIGIO = signal_.SIGIO
#SIGIOT = signal_.SIGIO
#SIGKILL = signal_.SIGKILL
#SIGPIPE = signal_.SIGPIPE
#SIGPOLL = signal_.SIGPOLL
#SIGPROF = signal_.SIGPROF
=============================================================================
SIGPWR = signal_.SIGPWR
SIGQUIT = signal_.SIGQUIT
SIGRTMAX = signal_.SIGRTMAX
SIGRTMIN = signal_.SIGRTMIN
SIGSEGV = signal_.SIGSEGV
SIGSTOP = signal_.SIGSTOP
SIGSYS = signal_.SIGSYS
SIGTERM = signal_.SIGTERM
SIGTRAP = signal_.SIGTRAP
SIGTSTP = signal_.SIGTSTP
SIGTTIN = signal_.SIGTTIN
SIGTTOU = signal_.SIGTTOU
SIGURG = signal_.SIGURG
SIGUSR1 = signal_.SIGUSR1
SIGUSR2 = signal_.SIGUSR2
SIGVTALRM = signal_.SIGVTALRM
SIGWINCH = signal_.SIGWINCH
SIGXCPU = signal_.SIGXCPU
SIGXFSZ = signal_.SIGXFSZ
=============================================================================
_lock = threading.Lock()
def _run_handlers(signal, args):
'''Executes all handlers that are registered for the given
siganl in the order of registration.'''
# run system handlers for preparing the signal handling
for handler in __sysinit[signal]:
handler(*args)
# run user defined signal handlers
for handler in __usrreg[signal]:
handler(*args)
# run system handlers for cleaning up the signal handling
for handler in __systerm[signal]:
handler(*args)
def add_handler(signal, handler):
'''
Add a handler to be executed on the signal signal
:param signal:  the signal to react to.
:param handler: a callable that will be called on the signal.
:return:
'''
_add_handler(signal, handler, __usrreg)

def _add_handler(signal, handler, registry):
'''
Add a handler to be executed on the signal signal
:param signal:  the signal to react to.
:param handler: a callable that will be called on the signal.
:return:
'''
with _lock:
    handlers_ = registry[signal]
    if not handlers_:
        signal_.signal(signal, lambda *args: _run_handlers(signal, args))
    if handler not in handlers_:
        handlers_.insert(0, handler)

def rm_handler(signal, handler):
'''
Remove a handler if it is registered to the given signal.
:param signal:  the signal that the handler is registered for
:param handler: the handler function.
:return:
'''
_rm_handler(signal, handler, __usrreg)
def _rm_handler(signal, handler, registry):
'''
Remove a handler if it is registered to the given signal.
:param signal:  the signal that the handler is registered for
:param handler: the handler function.
:return:
'''
with _lock:
try:
registry[signal].remove(handler)
except ValueError:
pass
def keyint(*_):
raise KeyboardInterrupt()
def enable_ctrlc():
'''
Allows to interrupt the main thread by pressing Ctrl-C.
:return:
'''
_add_handler(SIGINT, keyint, __systerm)
def disable_ctrlc():
'''
Disables interruption of the main thread by the Ctrl-C key combination.
:return:
'''
_rm_handler(SIGINT, keyint, __systerm)`

Fixed in dnutils release 0.2.12.

Hello! It seems that the error is still there. I am using macOS, python3.8.5. and dnutils 0.2.12.


Fix node-telegram-bot-api version

fixes #12

Please update this!! Spent a couple hours trying to track this bug down. Thanks :)

Omg, sorry!
Merged.

Thank you @pxr13 @shyr1punk
I am thinking of some CI solution to track this automatically.
Anyway, you rock. Thank you!


Standardize display of Infinity and NaN values

Now that the frontend UI displays strings such as Infinity, the plot fit value should be standardized as well. For example, T1 should say Infinity s in the plot label here:

For experiments that inherit from CurveAnalysis, the format is determined in a single location in the base class. T1 and T2Ramsey are about to inherit from CurveAnalysis.

@coruscating What do you mean by "not having the value, variance, and unit"? Has the discussion about it happened?


Create the ProspectCreateView so user can add new prospects

I can try this if it hasn't been assigned yet

solved the issue by creating a model view that will create a prospect by write the data to the core database and if successful
it will return a "SUCCESS MESSAGE" and if the core database is down it will return a "TRY AGAIN LATER" message for the user to wait for the core database write function to be up and running
the link for testing this is https://sales.zuri.chat/prospects/create and the first field (id field is not required)
the code/pr can be found here  #173

Okay :+1:


[Feature Request] Download free tracks when available

This would require a lot of work probably, but I noticed that some artists on Bandcamp charge money for the full album, however each individual track in the album is set to "name your price" as shown below:

It would be very handy if bandsnatch could automatically download each individual track from an album if the tracks are free. If this is out of scope, feel free to close this.

This is out of scope for Bandsnatch, it's just for downloading a user's collection. But also, this would require connecting to the user's email to catch the download links Bandcamp sends, and that's a rabbit hole I don't want to go down.

Thanks for the reply, and also thank you for this tool it has been very helpful!


ERROR in validation: ... validation error: '2023-03-23 13:35:45' is not a 'date-time' - 'timestamp'

Hello guys,
we just upgraded
openapi-schema-validator | 0.4.0 | 0.4.4
openapi-spec-validator | 0.3.0 | 0.5.6
and now in certain situations we get the following error:
[2023-04-12 10:55:55,777] ERROR in validation: http://<IP_ADDRESS>:9081/xrfapi/xrfDetectorCalibration/import validation error: '2023-03-23 13:35:45' is not a 'date-time' - 'timestamp'
when processing JSON data that looks like this:
{
  "uuid": "eea1804b-542f-4d6c-b88b-b2f2c0dc5915",
  ...
  "timestamp": "2023-03-23 13:35:45",
  ...
}

The identical json worked without issues before. Any troubleshooting hints?
It is possible that the upgrade pulled other upgrades that cause this effect or that it did not pull other upgrades it should have pulled -- anything to watch out for?

Upgrading openapi-schema-validator is not the root cause here. It works with both old and new versions of openapi-schema-validator. The important thing is that upgrading openapi-schema-validator pulls in rfc3339-validator. The presence of rfc3339-validator is a problem.

With rfc3339-validator installed '2023-03-23 13:35:45' is not a 'date-time'
With rfc3339-validator removed '2023-03-23 13:35:45' becomes a date-time with all versions of openapi-schema-validator

Can you explain that behavior? Can it be configured? Is removing rfc3339-validator the right thing for us? If so, why does openapi-schema-validator pull rfc3339-validator now in the first place?

Hi @niv-Inator
thanks for the report.
You get the validation error because your value is invalid date-time format based on RFC 3339, section 5.6
The issue was fixed in version 0.4.4 and when you choose to remove rfc3339-validator format checker skips date-time format validation.


Custom currency sign doesn't display correctly in private message

I have changed the currency sign to the :eggplant: emoji. However, when greeted by the $take command it will display the currency sign as letters instead of the icon because of the quote box feature.
One solution can be that the quotation box ends before the currency sign and the box starts again after it.

Show me your currency name in the config.json

That is the problem. Copy paste this
🍆
instead of
:eggplant:

Thank you

♥


Adds test for issue #1337

just a test that demonstrates #1337, no fix!
@astafan8

Closing this as #1359 has an equivalent test. Once again, thank you, @StefanD986, for reporting this issue.


ARROW-11121: [Developer] Use pull_request_target for PR JIRA integration

Demo: https://github.com/kou/arrow/pull/8

Demo: https://github.com/kou/arrow/pull/8


How to draw the attention map of ViTSTR?

Hello, thank you very much for being able to open the source code, it is a very rewarding work.
I am a graduate student and I want to do a little bit of my own experiment based on ViTSTR. Now I want to draw an attention map similar to the one shown in Fig. 9 of your paper, can you give me some help? Thank you very much!

Hi, Attention rollout: https://github.com/jacobgil/vit-explain


Rewrite Blade language to HTML

Adds at least partial support for Blade by rewriting it to HTML language.
See https://github.com/abdelhamiderrahmouni/filament-monaco-editor/issues/4

Thank you ❤🔥


mypy: Use Python 3 syntax for typing in multiple files.

Files modified:
zerver/webhooks/appfollow/view.py
zerver/webhooks/opsgenie/view.py
zerver/webhooks/gogs/view.py
zerver/webhooks/ifttt/view.py
zerver/views/typing.py
zerver/webhooks/pingdom/view.py
zerver/lib/avatar.py
zerver/webhooks/greenhouse/view.py
zerver/webhooks/beanstalk/view.py
zerver/tests/test_subdomains.py was already fully converted.

merged, thanks @picapi!


FYI Release 1.13

See
https://github.com/corona-warn-app/cwa-app-android/releases
https://github.com/corona-warn-app/cwa-app-android/releases/tag/v1.13.0-RC1
https://github.com/corona-warn-app/cwa-app-android/releases/tag/v1.13.0-RC2
Regarding release 1.14
https://github.com/corona-warn-app/cwa-app-android/pull/2375
https://github.com/corona-warn-app/cwa-app-android/pull/2484/files

EXPOSUREAPP-4879

@dsarkar
I'm not sure if the functionality was there all the time, but on GitHub you can selectively sign up for release notifications. Go to Watch / Unwatch (top right) > Custom and select "Releases".

@MikeMcC399 Thanks, didn't know that one. Watching all activity anyway. ;-)
The idea behind this issue is to direct users, which are not very familiar with the repo and GitHub to the release notes. Should increase the transparency for those who couldn't find the release notes and also might satisfy the curious ones. It's not a roadmap as discussed more than once in some wishlist issues, but it's something.

The release 1.13 has been announced today in the blog https://www.coronawarn.app/en/blog/2021-02-26-corona-warn-app-1-13-announcement/

Deutsche Telekom and SAP’s project team will release the Corona-Warn-App’s next update in the coming days. With version 1.13, users will be able to donate data to the Robert Koch-Institut voluntarily via the app. Based on that, the experts can further improve the app.
Additionally, users who have a red tile, i.e. an increased risk, will see a link to a scientific survey by the Robert Koch-Institut. By taking part in the survey, users can help the experts to analyze and better understand the app’s effectiveness. Detailed information on the update will be available here as soon as the project team has published version 1.13.

@dsarkar
Can you explain to us the reason for the deletion of the 1.13 announcement? Is the release:

delayed?
cancelled?
still due for release in the "coming days"?

@dsarkar
Can you explain to us the reason for the deletion of the 1.13 announcement? Is the release:

delayed?
cancelled?
still due for release in the "coming days"?

https://github.com/corona-warn-app/cwa-website/issues/921

CWA Android 1.13.2 has now been released.
See https://www.coronawarn.app/en/blog/2021-03-04-corona-warn-app-version-1-13/

@dsarkar
Do you need to keep this issue open?
1.14 and 1.15 have been released.


Stays IN_PROGRESS even if SUCCESS or has FAILED

I have this issue in my Ubuntu/.NET Core pipeline where it send IN_PROGRESS status on exit even if the job was SUCCESS or FAILURE.

There were no issue in my Ubuntu/Noje.js pipeline
TIA for your response on this.

I have forked this project and modified myself.
This works fine for my project.
Take a look

Hey @patrickpaulin I am not able to see the success message after using your forked. Here is my step. Let me know If anything is wrong in it.
 - name: MS Teams Notification Setup         uses: patrickpaulin/ms-teams-deploy-card@master         if: always()         with:           github-token: ${{ secrets.GITHUB_TOKEN }}           webhook-uri: ${{ secrets.MS_TEAMS_WEBHOOK_URI }}           environment: ${{ secrets.ENV }}           card-layout-exit: complete           show-on-failure: true           show-on-start: false           allowed-file-len: 10
`

Any update on this? @vimox-shah-genea  @patrickpaulin

Any update on this? @vimox-shah-genea @patrickpaulin

I am using @patrickpaulin's fork and it is working fine.


removing injury branch from under disease

I thought we had an issue for this, I can find ancillary tickets like #499
This places injury at root, it doesn't belong under disease
We can

leave at root and add an exception to the hardcoded list of roots in the test
place directly under material entity
place under ogms:disorder (which is a subclass of material entity)

I feel like we talk about this before too but I don't see any notes about it in our curation call minutes either.
I agree with moving it out the disease branch. I am not sure about the best place to put it. @sabrinatoro @matentzn, do you have opinions?

I agree that it makes sense to remove "injury" from the disease branch.
I remember we talked about it, but I don't remember what we said. I think it is important to find "why" we decided to have an "injury" branch in Mondo at all (I will also do some research in my notes).
Looking at the branch, there are "ulcer disease" and "acute radiation syndrome" which appear to be the only actual diseases. We should review and make sure that these, if we believe they are actual diseases, are somewhere else in the disease branch.

one thing at a time. This PR doesn't remove injury. It just changes the parent

I meant to say, if we remove injury from the disease branch. But the terms would still be accessible to users, right, in the same way as a disease susceptibility? I can't think of any objections to that. I agree with @sabrinatoro that we should move poisonings out of the disease branch too.

Just me being curious here. Does it make sense to have a common parent for diseases and injuries? Something like "affliction"?

If there are no objections from the community to this change, we'll implement these changes next week (on/after Monday, 2023-03-20)

no objections from the MedGen team to moving Injury outside of the disease concept. Thanks for the heads up.

no objections from the MedGen team to moving Injury outside of the disease concept. Thanks for the heads up.

I closed this by accident, reopening

We agreed that we will proceed with this PR. @matentzn could you adjust the QC check to allow the injury branch to be at root? Thanks!

@sabrinatoro can you fix what @nicolevasilevsky is asking for? This is the line: https://github.com/monarch-initiative/mondo/blob/master/src/sparql/qc/mondo/qc-no-superclass.sparql#L24

sparql looks good but there are conflicts in edit file

sparql looks good but there are conflicts in edit file

I will fix. Thanks @matentzn and @sabrinatoro!


Make consumer.Close() method race free

The select statement is not enough to guard against parallel calls
to Close. In case the "dying" channel is not closed, both goroutines
can go into default case resulting in double close of the channel.

@timoha could you pls quickly check #184, it's a bit simpler than using sync.Once


Updates to top bar

https://www.figma.com/file/wogMuUDylgOHgTEYhZ3jgp/Augur-Sportsbook?node-id=11045%3A9350
Reduce height of bar to 64px
Reduce size of logo to 32px height
Update font style - All caps and 16px bold
Dividers next to settings icon - make 64px (full height of bar)
Make full width 1440px

Could you make the dividers 50% @JohnDanz


Feat/map output names

Codecov Report
Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (501c814) 100.00% compared to head (37b3e98) 98.99%.

Files
Patch %
Lines

src/Attributes/MapOutputNames.php
0.00%
2 Missing :warning:

Additional details and impacted files
@@              Coverage Diff              @@
##                main      #60      +/-   ##
=============================================
- Coverage     100.00%   98.99%   -1.01%
- Complexity        71       86      +15
=============================================
  Files              8       11       +3
  Lines            163      199      +36
=============================================
+ Hits             163      197      +34
- Misses             0        2       +2

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Settings moving away from video canvas when scrolling

Mentioned in https://github.com/elan-ev/tobira/issues/1196, this was not solved yet, says https://tobira.opencast.org/!v/JePe-XwDwdz.

I opened an issue in the paella repo for that: https://github.com/polimediaupv/paella-core/issues/368


Sticky behavior guidance continued

Overview
The sticky behavior guidance is needed because we are seeing a lot of sticky components.  These sticky components may be conflicting each other and we should provide guidance on what gets to be sticky and when.
Additional Information

This issue is for the follow-on work to issue 971.

Acceptance Criteria

[ ] Sticky behavior guidance is reviewed by the design team members.  Final version is posted.

follow-on issue created.  Closing.


Fix version to bump minor

Fixes #20

 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.

Coverage remained the same at 100.0% when pulling cd8b1d441017c69f3ef4e498252b6e901010c21d on dnathe4th:dn-bump-version into<PHONE_NUMBER>3c39851adb64932a38320ce177b4e6 on uber-web:master.


The browser only shows "We're sorry, but something went wrong."

I tried with a couple of folders to run the command with the glob pattern in quotes.
tsuml --glob "./src/**/*.ts"

After showing on my terminal the list of files the glob matched, my browser automatically opens an address "https://yuml.me/diagram/plain/class/.svg". But the page merely shows this message in red characters: "We're sorry, but something went wrong."
There is no error message on the terminal.
I am using<EMAIL_ADDRESS>and I am logged in to yUML.

Same problem here.

Try to log out from yUML @trouve-antoine and reissue the command.

Same problem, solution suggested by @netanyahuyasser worked for me.

Didn't work for me.
I tried to issue the command, both logged in and logged out from yUML.
Perhaps it is important to note that the URL attempted at the en of tsumls execution was https://yuml.me/diagram/plain/class/%3C!DOCTYPE%20html%3E.
I'm running v0.0.1-alpha.8 on a Windows 7 machine.

Didn't work for me.
I tried to issue the command, both logged in and logged out from yUML.
Perhaps it is important to note that the URL attempted at the en of tsumls execution was https://yuml.me/diagram/plain/class/%3C!DOCTYPE%20html%3E.
I'm running v0.0.1-alpha.8 on a Windows 7 machine.

it happened to me too
I'm running '0.0.1-alpha.8' on Windows 10 machine

me too

I'm using bafolts/tplant and it's working

Same issue here

Same problem, solution suggested by @netanyahuyasser worked for me.

What was the solution?

Same problem, solution suggested by @netanyahuyasser worked for me.

Looks like the https://yuml.me/diagram/plain/class/ service is dead, and ask about the solution @egitura


Add an entry for Gogland

Not sure if it's better to wait for the official release of Gogland, or point it out now so more people test it.
In near daily use for the past few weeks, it's been better than using the IntelliJ plugin version.  Which itself was pretty good.


[github-check] Errors in unchanged files are filtered when upgrading from v0.9.17 to v0.10.0

When I upgraded from v0.9.17 to v0.10.0 I noticed that files that were not changed in a PR are now filtered and therefore don't fail anymore.
If I understand correctly, the goal of github-check was to also work for non-changed files.
It's basically same as -reporter=github-pr-check except it works not only for Pull Request but also for commit and it reports results outside Pull Request diff too.

Am I missing something?

Nevermind, this was caused by https://github.com/reviewdog/reviewdog#filter-mode

Thanks for the report. I should have documented it.
Update the README and CHANGELOG. #649


support for docker credentials helpers

I'm trying to use watchtower with an image hosted on ECR in AWS. ECR auth tokens expire every 12 hours, but there's a credentials helper that uses cli credentials or AWS_ACCESS_KEY_ID and AWS_SECRET_ACCESS_KEY environment variables to automatically refresh authentication when docker pull is used. I tried creating my own docker image from watchtower, adding the ecr credential helper, with Dockerfile:
FROM v2tec/watchtower
ADD docker-credential-ecr-login /bin/
ADD config.json /

config.json:
{
	"credHelpers": {
		"{aws_account_id}.dkr.ecr.us-west-2.amazonaws.com": "ecr-login"
	}
}

docker-credential-ecr-login was built based on https://github.com/awslabs/amazon-ecr-credential-helper.
I then run watchtower via docker-compose:
version: "3"
services:
  my-image:
    image: {aws_account_id}.dkr.ecr.us-west-2.amazonaws.com/my-image:latest
  watchtower:
    image: ecr-watchtower
    volumes:
      - /var/run/docker.sock:/var/run/docker.sock
    environment:
      AWS_ACCESS_KEY_ID: "{access_key}"
      AWS_SECRET_ACCESS_KEY: "{secret_key}"
    command: --interval 30 --debug

watchtower doesn't seem to respect the credential helper. logs:
watchtower_1  | time="2017-09-14T06:31:06Z" level=debug msg="Pulling {aws_account_id}.dkr.ecr.us-west-2.amazonaws.com/my-image:latest for /watchtower_my-image_1"
watchtower_1  | time="2017-09-14T06:31:06Z" level=debug msg="No credentials for {aws_account_id}.dkr.ecr.us-west-2.amazonaws.com/my-image:latest in /config.json"
watchtower_1  | time="2017-09-14T06:31:06Z" level=debug msg="No authentication credentials found for {aws_account_id}.dkr.ecr.us-west-2.amazonaws.com/my-image:latest"
watchtower_1  | time="2017-09-14T06:31:06Z" level=debug msg="Error pulling image {aws_account_id}.dkr.ecr.us-west-2.amazonaws.com/my-image:latest, Error response from daemon: Get https://{aws_account_id}.dkr.ecr.us-west-2.amazonaws.com/v2/my-image/manifests/latest: no basic auth credentials"
watchtower_1  | time="2017-09-14T06:31:06Z" level=info msg="Unable to update container /watchtower_my-image_1. Proceeding to next."
watchtower_1  | time="2017-09-14T06:31:06Z" level=debug msg="Error response from daemon: Get https://{aws_account_id}.dkr.ecr.us-west-2.amazonaws.com/v2/my-image/manifests/latest: no basic auth credentials"
watchtower_1  | time="2017-09-14T06:31:06Z" level=debug msg="Scheduled next run: 2017-09-14 06:31:35 +0000 UTC"

same goes for googles equivalent helper gcr.
btw you shouldn't need to build a new image to get this functionality just mounting the credential helper into $PATH as well as setting $HOME it knows where to lookup the config should be enough.
See the following snipped on how I made it work for compose:
docker run --name egm-compose -v /var/run/docker.sock:/var/run/docker.sock -v "/home/egm/easyguestmanagement:/rootfs/easyguestmanagement" -w="/rootfs/easyguestmanagement" -e HOME=/rootfs/easyguestmanagement -v /usr/bin/docker-credential-gcr:/bin/docker-credential-gcr docker/compose:1.16.1 up

FYI For a workaround/alternative setup for getting watchtower work with ECR see my setup at https://github.com/v2tec/watchtower/issues/38#issuecomment-331419620

@gabormay But if you are using cron and get login you don't need the credential helper or json? right? The whole point of the cred helper is to never have to run get login

@shadycuz Correct, in my setup I did not use the credential helper.
Btw, I would assume that the credential helper has to do a 'get login' or equivalent API call anyway, but indeed it would run it 'on-demand' as opposed to you (or cron) having to run it regularly.

I was able to get watchtower to play nice with ECR by following a similar approach (full explanation here), but I wasn't able to get my watchtower fork to see my AWS credentials until I changed it to start from alpine instead of from scratch. Once I changed it, I was able to use both environment variables and the ~/.aws/credentials file to authenticate.

adding a credential helper for aws ecr would, in my opinion, mean we'd need to add credential helpers for other popular cloud providers as well. we've yet to decide whether that is actually within the scope of watchtower or if some version of @gabormay's approach should be the recommended way of doing it.
care to weigh in anyone? how do you suggest we solve this in a vendor generic manner?

adding a credential helper for aws ecr would, in my opinion, mean we'd need to add credential helpers for other popular cloud providers as well. we've yet to decide whether that is actually within the scope of watchtower or if some version of @gabormay's approach should be the recommended way of doing it.
care to weigh in anyone? how do you suggest we solve this in a vendor generic manner?

you don't need to add specific support for each helper. from what I understand a .docker/config.json tells you which helper to use for each repository, and the helper's API is supposed to be consistent. ultimately, if you're using a docker client all of that should be handled for you...

Awesome! Would you be interested in having a stab at implementing this?

It seems like someone already gave it a shot - https://github.com/archetypesc/watchtower-ecr/blob/master/armhf/Dockerfile - however I couldn't get it to work either...
I've finally found pyouroboros/ouroboros which seems to cover this issue if you provide it with the creds helper and proper configuration:
  ouroboros:
    container_name: ouroboros
    image: pyouroboros/ouroboros
    restart: unless-stopped
    environment:
      - LOG_LEVEL=debug
      - SELF_UPDATE=true
      - INTERVAL=300
      - AWS_REGION=eu-west-1  # required for docker-credential-ecr-login
    volumes:
      - /var/run/docker.sock:/var/run/docker.sock
      - /bin/docker-credential-ecr-login:/bin/docker-credential-ecr-login
      - $HOME/.aws:/root/.aws
      - $HOME/.docker/config.json:/root/.docker/config.json  # contains '{"credsStore": "ecr-login"}'

I've been able to get it to work in a dev environment. As soon as I get around to it I'll try to integrate it into the image. However, that will likely not be until august.

I'm very happy to announce that this has been solved - and with no modifications to watchtower on top of that!
Documentation on how to do this is available at https://github.com/containrrr/watchtower/blob/master/docs/credential-helpers.md
Thanks for your patience!

It seems like someone already gave it a shot - https://github.com/archetypesc/watchtower-ecr/blob/master/armhf/Dockerfile - however I couldn't get it to work either...
I've finally found pyouroboros/ouroboros which seems to cover this issue if you provide it with the creds helper and proper configuration:
[...]

If you found a solution that works for you, then that's great! For me, and many others I've talked with, the size of the ouroboros image is way to large to make it usable in any real world scenarios.
Compared to watchtower, ouroboros weighs in at 7 times the image size, which quickly adds up to a large network utilization considering that image diffing at the moment's only possible by downloading the image and comparing the checksum.

If it makes a difference, I had the exact same  issue as the OP here.  I was able to fix it by adding 👍
"credStore": "ecr-login"
That one line in config.json fixed it all for me.  Now it works like a champ.

@chander Thank you for that explanation! I'll go ahead and lock this issue to prevent further discussions that might get lost in the ether, but if you don't mind, it'd be really great if you were to provide a PR that adds this to the docs.


multiple requests for multiple files...

Hi,
I understand your logic of uploading files from web client to S3. It is very clean! But there is one issue: you must create a ajax request to PHP server for each file you want to upload, and make one request to S3 for each file again. Is there any way to only request PHP server once to get the S3 credential and then upload all files to S3 based on this one-fit-all credential? I know that we can do it using another approach(example: https://www.designedbyaturtle.co.uk/2013/direct-upload-to-s3-with-a-little-help-from-jquery/), but I don't know if we can do it using pre-signed url...

each pre-signed url can be used to upload a specific file. you cannot re-use it (unless you want to overwrite the file you uploaded before).
if you don't want to play ping-pong with your server you could instead grant temporary credentials to the client but that opens up security issues.
only pre-signed urls give you full control over what is uploaded and where it is placed with which name.

Thanks for the explanation. I agree that each presigned url should be assigned to one file.
I am thinking, when user tries to upload multiple files, we can do just one ajax call to PHP server, which has all to-be-uploaded files' names and mime types, the PHP side should do a loop through the post data, create multiple presigned urls and return them to client. Now the client has one presigned url for each file to upload to S3.

obviously you could pre-select all files and try the upload only after the user pressed on an "Upload"-button. That way you could actually bulk query all the pre-signed urls.
You would however have to increase their TTL to deal with longer upload periods due to either a massive amount of files or massively big files (or both obviously). Otherwise the client might try to upload files with an already expired pre-signed url.
Another possible downside of this could be, that the AWS API sometimes is a bit slow creating pre-signed urls. Requesting, say, 1000 URLs in one bulk-call to your API could therefore make your PHP script or your XHR request to that PHP script time out.  (not tested)


Handle Errors in UI

Create an error service that can handle http and error messages. As part of this issue, you should improve the way they are currently handled. Consider using $mdToast to present simple messages to the user.

Merged into develop.


Support for debug draw

It would be very useful to have support for debug draw.
As far as I understand it, only this class has to be wrapped, and SetDebugDraw(), right?
https://github.com/rjanicek/liquidfun-rust/blob/master/liquidfun-cpp/Box2D/Common/b2Draw.h

I just added support for debug draw in my local fork, but I still have to test it..


Tap Only, unable to slide?

This is such a beautiful toggle.
I am trying it out, but it appears it's tap only? I can't get it to be slidable as the examples suggest.

Thanks. I got your point, you can use drag gesture to achieve exact output.


The stat init_fetch_timeout does not correctly increment when the timeout is reached

Description:
When init_fetch_timeout is reached (default of 15s) envoy gives up the xDS init process and proceeds to the next phase in initialization but does not increment the init_fetch_timeout counter.
Repro steps:
Define a xDS cluster that points to localhost:some_port
  - name: xds
    type: static
    hosts:
    - socket_address:
        address: <IP_ADDRESS>  # netcat listen on this address
        port_value: 50000
    connect_timeout: 1s
    http2_protocol_options: {}

Run nc -l 50000 and start envoy. You will see envoy connecting to the port and sending xDS requests. After about 15s envoy becomes LIVE.
$ curl localhost:8001/ready
LIVE

However the stat is never incremented
$ curl -s localhost:8001/stats | grep init_fetch_timeout
cluster_manager.cds.init_fetch_timeout: 0
listener_manager.lds.init_fetch_timeout: 0

Are you seeing config: initial fetch timed out for on the processes logs?

Looks like this is a regression introduced some where. Fixed in https://github.com/envoyproxy/envoy/pull/15114


Could you share the fine-tuned model?

Firstly, thank you for releasing the code.
Could you share the fine-tuned model?
Hope you reply.

你成功训练模型了吗？

Is there a successful training model?


Xcode 16 compatibility issue

Hello,
My project doesn't compile anymore, it is likely due to its dependency to Sentry,
They Updated to 8.29 so it can now compile on Xcode 16.
Could you please update to Sentry 8.29 in all of Lyra projects that depends on Sentry ?
Regards

Hi,
There is still an issue with Xcode 16 for Sentry below v8.33.0 (see this issue)
Could you please update to 8.33.0 (or higher)?

@ThomasBuresi just so you know the bump to Sentry 8.36.0 is on master now 😁🎉


Image .JPG non supportés mais .jpg oui

done; to be tested a bit more.

tested with PNG JPEG JPG GIF BMP: OK
TIF and TIFF image type not supported


ADD: Obol Charon Service, EventListener via IPC

Obol Charon Service
this marks the first step for the integration of the complete obol service
EventListener via IPC
added boilerplate code for EventListener which listen to events emited from the backend
background.js
//get right BrowserWindow object
let allWindows = BrowserWindow.getAllWindows()
let mainWindow = allWindows[allWindows.length - 1] // the last window is always the main (at least in our case)
mainWindow.webContents.send('ChannelToListenOn', data)

SomeComponent.vue
import ControlService from "@/store/ControlService";

// start listener (in created for example)
ControlService.addListener('ChannelToListenOn', this.someFunction)

// remove listener (in beforeUnmount() for example)
ControlService.removeListener('ChannelToListenOn', this.someFunction)

// function that updates this.counter in real time
someFunction(event, data){
  this.counter = data
}

Hey guys, loving that you guys are integrating Obol ❤  One small heads up, the Icons you guys have outlined, are our older versions (note the shape of the loop is much more asymetrical/fish like), you can get copies of our latest icon here. Let me know if you need any other resolutions/formats. :D

Hey guys, loving that you guys are integrating Obol ❤ One small heads up, the Icons you guys have outlined, are our older versions (note the shape of the loop is much more asymetrical/fish like), you can get copies of our latest icon here. Let me know if you need any other resolutions/formats. :D

Hey 👋
Already took care of it (I think)
These are the three icons we will use inside of Stereum, I changed them according to the page you linked:

Lets us know if these better fit your brand, or we should change something else! ❤

P.S.: because I stumbled upon it, there is a hex color code on there that doesn't work:

(Probably a zero, or? Didn't use it just to be sure 😄 )

P.S.: because I stumbled upon it, there is a hex color code on there that doesn't work:

(Probably a zero, or? Didn't use it just to be sure 😄 )

Fixed! Danke


Add the transport policy as a copy.

Transport policies should be stored by value so that users can reuse/release the structure.
Various bug fixes along the way:

Correct some verbs
Fixes to upload callback after the refactor

If Charlie comes up with a good pal build pattern, then this parameter will go away

Jeffrey Richter (watch Architecting Distributed Cloud Appshttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fazure.microsoft.com%2Ftraining%2Fdistributed-cloud-apps%2F&data=02|01|jeffreyr%40microsoft.com|82df4ff9e5ae46a5095908d7345e957b|72f988bf86f141af91ab2d7cd011db47|1|0|637035453166233741&sdata=4GnkpFUKxlZXGXFr6oQo5GB%2FFNIbDXIZDXUij0q4AEE%3D&reserved=0)

From: Victor Vazquez<EMAIL_ADDRESS>Sent: Tuesday, February 18, 2020 2:03 PM
To: Azure/azure-sdk-for-c
Cc: Jeffrey Richter; Review requested
Subject: Re: [Azure/azure-sdk-for-c] Add the transport policy as a copy. (#360)
@vhvb1989 commented on this pull request.

In sdk/core/core/src/az_http_policy.chttps://github.com/Azure/azure-sdk-for-c/pull/360#discussion_r380962897:

@@ -174,3 +174,12 @@ AZ_NODISCARD az_result az_http_pipeline_policy_transport(

return send_request(p_request, p_response);
}
+
+AZ_NODISCARD az_http_transport_options
+az_http_transport_options_default(az_http_client_send_request_fn send_request) {
it is just a kind of weird that it gives the default options but it only sets the value it gets as parameter XD.
Looks more like an init for its implementation.
But I got it. It is because of all other options we have with this default approach...
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHubhttps://github.com/Azure/azure-sdk-for-c/pull/360?email_source=notifications&email_token=AARLJP22WHFNIDBXRYCGYRDRDRLM3A5CNFSM4KXKM242YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCV77IGI#pullrequestreview-360707097, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AARLJP2Y2YSE56R7CYZ3EITRDRLM3ANCNFSM4KXKM24Q.

Yes, this is OK; it was actually a bug that we used the same one before – I can explain why in person.
However, all of this is temporary (I hope). Once we get the PLA stuff sorted out, I think we’ll hardcode that transport policy in Core so it will be 1 transport policy for the customer which ALL clients (and credentials will use.

Jeffrey Richter (watch Architecting Distributed Cloud Appshttps://azure.microsoft.com/training/distributed-cloud-apps/)

From: Victor Vazquez<EMAIL_ADDRESS>Sent: Thursday, February 20, 2020 9:14 AM
To: Azure/azure-sdk-for-c<EMAIL_ADDRESS>Cc: Jeffrey Richter<EMAIL_ADDRESS>Review requested<EMAIL_ADDRESS>Subject: Re: [Azure/azure-sdk-for-c] Add the transport policy as a copy. (#360)
@vhvb1989 approved this pull request.
LGTM.
However, I noticed that now we allow to use a different transport options for a credential and for an SDK client (like keyvault). Is that OK?
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHubhttps://github.com/Azure/azure-sdk-for-c/pull/360?email_source=notifications&email_token=AARLJP6U47J5TW7TBDHWGG3RD224VA5CNFSM4KXKM242YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCWKPF5A#pullrequestreview-362083060, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AARLJP3OKG3AWMUX3SWYAD3RD224VANCNFSM4KXKM24Q.


Try the new wagmi@alpha and test the fee currency support

After experimenting quite a bit and getting my hands dirty, it's safe to say the alpha is not ready and we should table that issue for now.
Outside of "cosmetic" changes (such as renaming the Provider and Context and some minor signatures), they also changed the way to declare which wallet connectors to use and that affects both rainbowkit and rainbowkit-celo. So will wait for documentation because the alpha one is very much in progress

cc @0xarthurxyz

Leaving this as blocked

Looks like we might be unblocked, wagmi v2 alpha is out it seems

Still very alpha, haven't managed a proper working example app

As of  2.0.0-alpha.10, the feecurrency feature is still not a thing as far as my trials go. Will reach out to their team to figure out if that's a me issue

Looks like they are at wagmi beta now: https://beta.wagmi.sh

Will reach out to their team to figure out if that's a me issue

Do they know about our fee currency issue?

https://github.com/celo-org/react-celo/issues/364

Wagmi 2.0 is in production (out of alpha and beta): https://github.com/wevm/wagmi/releases/tag/wagmi%402.0.3

I confirmed that
  const config = useConfig()
  const { data: hash, sendTransaction } = useSendTransaction({config: {...config, chains: [celo]}})
  const sendIt = () => {
    sendTransaction({
      feeCurrency: cUSDAddress.data,
      value: BigInt(100000000),
      to: '0x22579CA45eE22E2E16dDF72D955D6cf4c767B0eF',
  })

sends the request with fee currency all the way to

client.request({
            method: 'eth_sendTransaction',
            params: [request],

note that to get types working correctly i needed to set the celo chain on the config explicitly
useWriteContract also worked perfectly


display dependencies on demand in the dialog

fixed by https://github.com/zaggino/brackets-npm-registry/commit/d03fc85145db3a12d70dd88f8bbd53a5d752b493


feat: fabric operations console

Fabric Operations Console with all its incredible features (see 👇 ) is finally here! 🥳
This PR includes the following:

Support for all the features offered off-the-shelf by the console
One-command run - fabkit console start
Automatic creation of components artefacts starting from current network config (multi-org! 👯 )
Step-by-step guide to one-time configuration and getting started

The console provides the following high level function:

Ability to import and manage all Hyperledger Fabric Components from a single web console, no matter where they are located.

Join Peers to Channels and view channel membership as well as individual transactions and channel details.
Register, view, delete, and re-enroll CA Users.
View Ordering cluster and node information as well as view and modify consortium and channel membership.
View and modify channel capabilities and ordering service parameters.
Install and Instantiate chaincode.  Supports both 1.x and 2.x Lifecycle.
View, Create, Import and Export Organizations and Identities.
Role Based Access Control in UI to tightly control which Console users can perform which operations.

https://github.com/everledger/fabkit/blob/4a081baee2f6a1a7a2d06bacb0ed8c8c23ac2bb9/fabkit#L26 - this line makes it hard for people to run fabkit when they don't have or have poor internet.
Can you add a TODO to implement a caching mechanism for this?

https://github.com/everledger/fabkit/blob/4a081baee2f6a1a7a2d06bacb0ed8c8c23ac2bb9/fabkit#L26

this line makes it hard for people to run fabkit when they don't have or have poor internet.
Can you add a TODO to implement a caching mechanism for this?

I thought about it in the past and then opted for this on-the-fly approach after realising that a peer will always require an internet connection in order to deploy a chaincode container.

I removed the check dep version from the main script because it was unnecessarily increasing the overhead time. We don't need to check everytime we run Fabkit whether they have the required dependencies.

@gtpaulose I reverted one commit which was removing the dependency check. While I think it is a good suggestion, I would rather prefer to do this in another ticket (and perhaps architecting a good caching mechanism). Moreover, the __check_dep_version would have been reachable from the install.sh script when run on the fly.


Erlang 19

Anyone have luck getting this to compile with Erlang 19?

I've got things compiling with Erlang 19. Check out #47
The exometer_core package on hex probably needs updating before it can be merged, though.

@mootpointer does master work for you? We've had luck on our end.

We also had compilation errors with OTP 19.
include/amqp_gen_consumer_spec.hrl:30: syntax error before: '/'
include/amqp_gen_consumer_spec.hrl:31: syntax error before: '/'
include/amqp_gen_consumer_spec.hrl:32: syntax error before: '/'
include/amqp_gen_consumer_spec.hrl:34: syntax error before: '/'
include/amqp_gen_consumer_spec.hrl:35: syntax error before: '/'
include/amqp_gen_consumer_spec.hrl:36: syntax error before: '/'
include/amqp_gen_consumer_spec.hrl:37: syntax error before: '/'
include/amqp_gen_consumer_spec.hrl:38: syntax error before: '/'
include/amqp_gen_consumer_spec.hrl:39: syntax error before: '/'
include/amqp_gen_consumer_spec.hrl:42: syntax error before: '/'
include/amqp_gen_consumer_spec.hrl:30: syntax error before: '/'
include/amqp_gen_consumer_spec.hrl:31: syntax error before: '/'
include/amqp_gen_consumer_spec.hrl:32: syntax error before: '/'
include/amqp_gen_consumer_spec.hrl:34: syntax error before: '/'
include/amqp_gen_consumer_spec.hrl:35: syntax error before: '/'
include/amqp_gen_consumer_spec.hrl:36: syntax error before: '/'
include/amqp_gen_consumer_spec.hrl:37: syntax error before: '/'
include/amqp_gen_consumer_spec.hrl:38: syntax error before: '/'
include/amqp_gen_consumer_spec.hrl:39: syntax error before: '/'
include/amqp_gen_consumer_spec.hrl:42: syntax error before: '/'
Compiling src/amqp_selective_consumer.erl failed:
ERROR: compile failed while processing /Users/zorbash/dev/some_app/deps/amqp_client: rebar_abort
** (Mix) Could not compile dependency :amqp_client, "/Users/zorbash/.asdf/installs/elixir/1.3.3/.mix/rebar compile skip_deps=true deps_dir="/Users/zorbash/dev/some_application/_build/dev/lib"" command failed. You can recompile this dependency with "mix deps.compile amqp_client", update it with "mix deps.update amqp_client" or clean it with "mix deps.clean amqp_client"

It compiles successfully if you do:
EXOMETER_PACKAGES=minimal
mix deps.get
mix compile

amqp_client is listed in the rebar config of Exometer (see: https://github.com/Feuerlabs/exometer/blob/master/rebar.config#L20).
Exometer uses env variables to configure its dependencies (see: https://github.com/Feuerlabs/exometer#dependency-management)
My understanding is that the rebar compilation script of Exometer =>
https://github.com/Feuerlabs/exometer/blob/master/rebar.config.script#L40
fetches amqp_client and a few more dependencies by default and compiles them.
The version of amqp_client on which Exometer depends on, seems to not be compatible with OTP 19.

Master builds under OTP 18-20 now, and Elixir 1.3 through 1.5 are currently supported.


Determine Tracing Level Defaults

Tracing should have a configuration that determines what is traced. There are four common scenarios that should be supported:

Disabled
Wire requests (HTTP calls, etc)
Wire requests and long running operations (operations that could result in multiple service calls)
Everything

From these a default configuration should be determined.

Combining all open tracing issues to #3823


Update Terragrunt to work with Terraform 0.14

Add support for copying lock files as necessary. Fixes #1454.
Fix issue where Terragrunt would re-init every time you ran it. Fixes #1423.

@infraredgirl We need to remember to look into https://github.com/gruntwork-io/terragrunt/issues/1423 also  as part of supporting 0.14!

Tests passed! This is now ready for review!

Thank you for the review! Merging now.

Thank you for the review! Merging now.

Released in: https://github.com/gruntwork-io/terragrunt/releases/tag/v0.27.0

Released in: https://github.com/gruntwork-io/terragrunt/releases/tag/v0.27.0


Record an introductory demo

Description of problem
There have been multiple requests and also an outstanding one since Cartographer launch, to have a recorded demo where we introduce users on how to actually use Carto.
It could be something like 'a day in the life of a workload through a supply chain which is choreographed by Cartographer'
As described by one of the users who requested a demo, the expected outcome could look like:

How to create a workload, label it and run it through a choreographed supply chain

Tagging  @ciberkleid and @waciumawanjohi to know your thoughts around this.

Also related to #275

Done as part of TGIK episode:
https://www.youtube.com/watch?v=TJPGn0-hpPY&t=1957s


switch to new guessit version

Motivation for changes:
dateutil is no more restricted to >= 2.5.2 in guessit >= 2.1.0

Duplicate of #1398

oops, sorry


NET 6 support on Linux

Hello, is NET 6 currently supported on Linux? If not, are there any plans in this regard? System.Drawing supports only Windows for NET 6.

FastReport Open Source works fine on Linux with .NET 6 (with libgdiplus). Or did you mean UI?

We will work to support a different drawing engine

Thanks for the detailed answer!


(Bug/Suggestion) Items get Smelted with Silk Touch

Hey, thanks for your amazing plugin! I have a request, wich might make the plugin even better.
So currently, any item that gets mined will instantly be smelted. My suggestion is, that if theres an Silk Touch enchantement on the tool, the item will not get smeltet (maybe turnable on/off in config). If this is possible, it would be a great addition to the plugin!
Thanks and a good day!

Ow ! Of course ! I'm just waiting for the publishing on poggit and after i can do it !


Possible character encoding issue, in browser tab title?

The tab title has incorrect characters in it, see screenshot.  Seems to be true, whichever module I go to.  It's OK when on the Dashboard though.
I don't know how long this issue has been there - several versions, I think, just got round to reporting.
This is the same in Google Chrome latest, running in ChromeOS, Windows XP and WIndows 7, and also IE in Windows 7.
Many thanks!

Sounds like very possible. I'll take a look!

Do you have language set to UTF-8?

Hi.  Just checking...   I have the encoding is set to 'Western (Windows 1252)'.  Behaviour is the same if I switch to UTF-8 too.
thanks

I can't confirm your issue! I think I know why.
In case you went to Webmin->Webmin Configuration->Language and set there proper language (like .UTF-8), you must also make sure that you have two things set-up correctly:

Module Webmin->Change Language and Theme can override global settings!
Make sure that your browser really detects encoding as UTF8, if not set it manually!

Do you have other options set in Webmin->Webmin Configuration->Language, like Character set for HTML pages and Use language specified by browser?
With the following settings it works fine in all browsers:

Same should be done with Webmin->Change Language and Theme:

Anyway, first, you must make sure that the UTF-8 is set in both cases and that your browser detects it correctly. Go to browser tools, encoding options and in case it's set to Windows 1252 or automatically, try forcing to UTF-8 and reload the page.
P.S. While I was writing it, I was many years ago also a Windows user (:() and I remember there were locale settings that affected the way plain text documents encodings are shown. For example, I always used English as system language and at the same time, when I was trying to view/edit plain text in Russian things always didn't work. Make sure to make system settings correct as well.
I believe it is your system specific thus non theme related issue. Closed.

Hi, just checked on the settings you mentioned, and all is now sorted, many thanks!
On Webmin->Webmin Configuration->Language, it was 'English US (EN)', changed that to 'English UK (EN_GB.UTF-8)'.
And then Webmin->Change Language and Theme, was Personal Choice, 'English UK (EN_GB)', changed that to 'Global language'.
I didn't select any of these on purpose, but who cares, making the changes you suggested has sorted it.
thanks, Andy

Hi,
Just thought I'd chime in. I am seeing an error specifically when I am viewing the "Virtualmin" side of the interface, Webmin side works without any issues. I've confirmed that all the encoding settings as described here have been setup correct including both Webmin and browser.

-Peter
*** BTW: I dumped my chat server which is why you haven't seen me online. Perhaps I'll either setup a new one, or get an account somewhere so we can resume our chats. ***

Peter, I will re-post this issue to Joe, as I can't reproduce it.

Hi,
It should be noted, I've only noticed the issue in a Virtualmin Pro installation as my GPL copies do not present the same issue. Also, the system in question I DID NOT originally do the install so it could be related to something that I haven't thought of yet.
-Peter


I can edit other students' replies in threads

Describe the bug
I clicked on "reply" in a thread, and a window in which apparently I can edit another student's reply appeared. I don't think this is intended behaviour

After a bit of research, it looks like it happens only from the Exercises page and not from the Communication page
To Reproduce

Go to the Communication sidebar on any exercise
Click on 'Show reply' (1)
Click on 'Reply' (2)
See error

Expected behavior
I should reply to a student and not edit their message
Screenshots
No response
Which version of Artemis are you seeing the problem on?
7.7.1
What browsers are you seeing the problem on?
Firefox
Additional context
No response
Relevant log output
No response

This will be fixed by this PR https://github.com/ls1intum/Artemis/pull/9815


BorderRect maintenance

Objective
The doc comments and function namings for BorderRect feel imprecise to me. Particularly the square function which is used to define a uniform BorderRect with equal widths on each edge. But this is potentially confusing since this "square" border could be around an oblong shape.
Using "padding" to refer to the border extents seems undesirable too since "padding" is typically used to refer to the area between border and content, not the border itself.
Solution

Rename square  to all (this matches the name of the similar method on UiRect).
Rename rectangle to axes (this matches the name of the similar method on UiRect).
Update doc comments.

Migration Guide
The square and rectangle functions belonging to BorderRect have been renamed to all and axes.

I like these changes, but seconding the request not to lose the information about pixels :)

CI failure seems real and related: I think there's a conditional compilation issue here.

I think it's just new stuff from #16693 that needs to be updated.


Dem fixing tangential overlap

Description of the problem

The normal overlap and tangential overlap vectors needs to be perpendicular through out a collision, which is not the case in the current implementation.

Description of the solution

We subtract the projection of the tangential overlap on the normal vector from the tangential overlap at every time step.
This is not done for particle-wall contacts, since the normal vector remains the same through out a collision with a flat wall (which is the only type of wall currently implemented).

How Has This Been Tested?

With the application test. Some results have slightly changed

Why do you have so many changed files that are unrelated to DEM????


Update updated_at without redeeming sig

Pull Request Test Coverage Report for Build 726777516

7 of 7 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.03%) to 25.657%

Totals

Change from base Build 726295600:
0.03%

Covered Lines:
1739

Relevant Lines:
6778

💛  - Coveralls


Message recipient dropdown behaviors

Close when clicking outside the menu
Close on hitting
Animate the chevron upside-down when open
Move "view recipients" function to count pill

Done.


Coverage fix

Pull Request Description
What issue does this change request address? (Use "#" before the issue to link it, i.e., #42.)
Closes #1076
What are the significant changes in functionality due to this change request?
Adjusts paths in check_py_coverage.sh to align with new rook testfinding behavior as of #1056.

For Change Control Board: Change Request Review
The following review must be completed by an authorized member of the Change Control Board.

[x] 1. Review all computer code.
[x] 2. If any changes occur to the input syntax, there must be an accompanying change to the user manual and xsd schema. If the input syntax change deprecates existing input files, a conversion script needs to be added (see Conversion Scripts).
[x] 3. Make sure the Python code and commenting standards are respected (camelBack, etc.) - See on the wiki for details.
[x] 4. Automated Tests should pass, including run_tests, pylint, manual building and xsd tests. If there are changes to Simulation.py or JobHandler.py the qsub tests must pass.
[x] 5. If significant functionality is added, there must  be tests added to check this. Tests should cover all possible options.  Multiple short tests are preferred over one large test. If new development on the internal JobHandler parallel system is performed, a cluster test must be added setting, in  XML block, the node <internalParallel> to True.
[x] 6. If the change modifies or adds a requirement or a requirement based test case, the Change Control Board's Chair or designee also needs to approve the change.  The requirements and the requirements test shall be in sync.
[x] 7. The merge request must reference an issue.  If the issue is closed, the issue close checklist shall be done.
[x] 8. If an analytic test is changed/added is the the analytic documentation updated/added?
[x] 9. If any test used as a basis for documentation examples (currently found in raven/tests/framework/user_guide and raven/docs/workshop) have been changed, the associated documentation must be reviewed and assured the text matches the example.

I see two other files are removed from this; those are extraneous files that should have been removed in the dataobject rework, as they don't apply anymore. I'm surprised they're still in devel.

Checklist passed...
PR can be accepted.
Andrea


added newid if pure multibyte character set

Summary
If the incident name has only multibyte characters its does not create the channel, this change will create a new id and it will allow channel creation.
Ticket Link
https://github.com/mattermost/mattermost-server/issues/16834#event-4282247227

Hello @arjunagl,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

thanks @lieut-data, I have included an unit test as well into the .go code.

Just a heads up that this change may find an edge case in the bug filed over at https://mattermost.atlassian.net/browse/MM-32912, but it's something for the Web Platform team to investigate.


Colab - ImportError

I launched colab notebooks and tried to run cells one by one.
But I encountered the ImportError.
(1)

(2)

Thanks @dyuliu for pointing this issue.
The problem here is that you need to restart runtime after finishing the installation and run the block again, because you are uninstalling default pandas and urllib versions in colab.
# install dependencies then restart kernel and run again
! pip install orion-ml
! pip install 'urllib3!=1.25.0,!=1.25.1,<1.26,>=1.21.1'

! git clone https://github.com/signals-dev/Orion.git
! mv Orion/notebooks/tulog/* .

The second part of cloning the repo and copying files is causing the issue for you (because it has not been run). I am wondering if you have an idea of a better installation mechanism than basic pip install and git clone commands. Ideally, I wish to have a persistent environment that colab always runs from but I don't think this is possible.
It's a similar issue pointed to in #153.

Main issue
When you run !pip install orion-ml cell, orion installs some versions of libraries, for example pandas, that are counter to the default version in google colab. Thus, colab sends a subtle message of asking the user to restart runtime. If this task is ignored, the user will run into an error in the rest of the notebooks (as demonstrated by @dyuliu).
Discussion of possible solutions

add an exit() command immediately after the pip install command to ensure the kernel would be reset.
put an error message to inform users that you should be installing orion then restarting kernel.
catch the exception caused by the import section cell, indicating to the user to restart runtime.

Considerations to keep in mind

If colab is reminding the user, is it our responsibility to also remind them?
Binder is a better setup for our use case, should we also support colab or keep it at the discretion of the user?

Improvements to the current setup

Colab actually asks the user to restart runtime, but it's really hidden (buried) under the installation output, to solve this we can add -q to pip installations
Break the cell into (1) intalllation cell (2) cloning cell.
Include urllib installation in orion installation -- ensure this in the new release.
Show a successful installation message to the user by displaying orion version.


New recipes from other awesome cookbook

Some examples I find:

https://github.com/dabeaz/python-cookbook
https://golangcookbook.com/

I will add an example in 15. Text Processing similar to the Turning an Array into a Sentence and Processing a String One Word or Character at a Time in the Go Cookbook.

@horochx Those two example seems tool simple to me, I don't know if it's necessary to add them.

@jiacai2050 I'm currently learning Zig and I got quite a collection of Zig code snippets...however how to know if something is too simple? I mean I've been struggling to find reference to learn Zig, hence I made the snippet collection...and I think this Zig cookbook is gonna be amazingly helpful for Zig newcomers.
For example i see there's Generate Random Numbers in the cookbook already, but it's using crypto.random. I have snippet that uses rand.DefaultPrng:
const RndGen = std.rand.DefaultPrng;
const tMilli = std.time.milliTimestamp;
const PageAlloc = std.heap.page_allocator;

fn gen_randNum(minVal: u32, maxVal: u32, arraySize: u32) ![]u32 {
    var seed = tMilli();
    var rnd = RndGen.init(@intCast(seed));
    var randNums = try PageAlloc.alloc(u32, arraySize);

    for (randNums) |*num| {
        num.* = rnd.random().intRangeAtMost(u32, minVal, maxVal);
    }
    return randNums;
}

I also got plenty of Sorting and Search Algorithms written in Zig.
To be honest I have been looking for an avenue to contribute to Zig learning for newcomers...so I would love to be involved in the Zig Cookbook. Cheers!

but it's using crypto.random. I have snippet that uses rand.DefaultPrng

I don't see much difference between those, so I would suggest you try another recipe, thanks.


Fix markdown table in bare-metal reference

Copied header from Packet guide.

https://github.com/kinvolk/lokomotive/pull/1245#issuecomment-739704539


Decoding parameters used in evaluation (Flan Collection)

First off, thank you for open-sourcing the collection of templates and data, etc. it can enable a lot more research!
Could you please share the parameters that were used in decoding and any other details around decoding that could be useful in playing with Flan-T5? @shayne-longpre Thank you!

Hi ioana-blue,
There's no critical decoding parameters as far as I can remember. You can use greedy decoding (temperature=0.0). And it's pretty obvious to set dropout=0.0
Let us know if you want to know the value of a specific parameter.
Thx

@lehougoogle Thanks for the prompt reply! It would be nice to add this detail to the model card in the paper (I couldn't see this, it may be already in there).


Barcode implementation

I'm working on a (partial, for now) implementation of barcode printing based on your other function implementations,, but it's not actually printing a barcode when I try to use it.  Any chance you could look it over and see what I might be missing?
BarCode.php.txt
In my PHP page that prints the label, this is the line I use to write the barcode:
$printer->addCommand(new Command\BarCode($detailID,48,Command\BarCode::WIDTH_MEDIUM,"code39",false,2.5));

Hello,
A (long) while ago I also tried to get barcodes to work, I made this: https://github.com/RamyTalal/Label-Printer/pull/5/files#diff-1f38cad15689411667485a54ba9d2025 but I also can't get it to work.

I got Code39 barcode working for me.  I took a look at your implementation, and with just changing the preference order of 3 options (basically moving Parenthesis ahead of Height and shifting Height and Width down by 1) I have it working for me with your code too.  I added suggestions on the pull request showing the change.
My only further comment would be that modifying the options could be tough for an end user, because they may not be able to read the code well enough to figure out which order preference they should be modifying when it's just a number.  Since there's a static number of options, and they seem to need to go in a specific order, it seems more accessible to use alphabetic array keys instead of numeric keys, and then have the parseOptions function read them in the specific order they're needed.  That would make input of an options array at construction easier to create. For example:
$printer = new Printer(new Escp($stream));
$options = array("height"=>80,"parenthesis"=>Command\BarCode\Parenthesis::OFF,"ratio"=>Command\BarCode\Ratio::TWO_BY_ONE);
$printer->addCommand(new Command\BarCode("123456",Command\BarCode\Format::CODE39,$options));
$printer->printLabel();

Oh, I also had to remove some extra characters on a couple options that I'm not sure why they were in there. Adding those as code suggestions also.


Build is failing on Darwin

Problem
The gn_build.sh is failing on Mac OS
Summary of Changes

Updated the CHIPClustersObjc.mm file
Updated the LockTracker.h file

Test

Used the ./gn_build.sh  to verify the building is successful

The fix is to update the templates. It looks like the code has been hand rolled here. I have included the fix into: https://github.com/project-chip/connectedhomeip/pull/6596/commits/9b8a43a8076c1ea0add6fd2b245dfca314c0fd90

@bzbarsky-apple Updated the PR and removed the generated code chage

Please try #6748 instead

Doesn't appear to be needed anymore, after @andy31415 's changes. Closing - but please re-open if that's incorrect!


Banner and release notes v 1.2

Updated banner and release notes

@EricSoroos Any updates on this?


Upgrade Bitcasa to <IP_ADDRESS>

Thank you for the contribution. It was merged directly as ead1cd1145dd06a6d238b35c6f881c57888c947f to keep commit history cleaner. Your contribution is still credited to you.


webservice not accepting uri notation

I'm querying a webservice which, for some reason works with
http.request({host:"www.myhost.com",path:"/some/path?{query:options}", method:"GET"});

But when I use the
var someURI = "http://www.myhost.com/some/path?{query:options}";
request(someURI)

syntax, the webservice hates it and won't return any data for exactly the same request.
Is it possible to make the URI notation optional? ... apologies if this has been asked previously.

You can try to encode it first
var someURI = "http://www.myhost.com/some/path?"
  + encodeURIComponent(JSON.stringify({query:options}));
request(someURI)

Thanks @simov, but this doesn't seem to work.
There appears to be something going on with the specific server. I say this because other organisations have the same webservice and I can use both http.request(), and request() and they both work. However with this one agency http.request() works and request() doesn't for exactly the same call. Here are the working and not working calls.
Any help on using the request() for streaming rather than the old http.request->response  would be fantastic.
Parameters
var query = {
    "function":"get_site_list",
    "version":"1",
    "params":{
        "site_list":"11000*"
    }
}

var options = {
          "host":"watermonitoring.dnrm.qld.gov.au",
          "path":"/cgi/webservice.server.pl?",
          "method":"GET"
      }

Not Working
var request = require('request');

//Either:
1. options.path = options.path + JSON.stringify(query);
//Or
2. options.path = options.path + encodeURIComponent(JSON.stringify(query));

var uri = 'http://'+options.host + options.path;

request.get(uri)
        .pipe(process.stdout)

Working

var http = require('http');

options.path = options.path + JSON.stringify(query);

http.request(options,function(res){
        res
        .pipe(process.stdout)
    })
    .on('error', function(err) {
        console.error('Error with the request:', err.message);
        cb(err);
    })
    .end();

I was looking at implementing my own stream of http.request() as a fork of request(), however it looks complicated.
Maybe I've found an edge case which should be covered with request() itself.

Ok so on further investigation the particular service doesn't like the encodeURI which request() appears to do internally.
Put the following addresses into a Google Chrome address bar:
Un-encoded Works
http://watermonitoring.dnrm.qld.gov.au/cgi/webservice.server.pl{"function":"get_site_list","version":"1","params":{"site_list":"11000*"}}

request()'s Encoded Output Doesn't Works
http://watermonitoring.dnrm.qld.gov.au/cgi/webservice.server.pl%7B%22function%22:%22get_site_list%22,%22version%22:%221%22,%22params%22:%7B%22site_list%22:%2211000*%22%7D%7D

Confirmed.
If I do the following decode in request.js it works.
So the question is, how do I check for whether the webservice wants a coded or decoded path?
(starting at around line 960 in request.js)

// We have a method named auth, which is completely different from the http.request
  // auth option.  If we don't remove it, we're gonna have a bad time.
  var pat = self.path;
  self.path = decodeURIComponent(pat);

  var reqOptions = copy(self)
  delete reqOptions.auth

  console.log('self path',self.path);
  debug('make request', self.uri.href)
  self.req = self.httpModule.request(reqOptions)

The thing is that with these webservices, if you don't get the query right, it doesn't return an error. It returns a valid html document telling you how to access the webservice & wsdl.
So what's the best way to handle this edge case? Fork?

rfc3986() seems to be the culprit.

Thanks for the thorough bug report.
Request uses the standard url module to parse the url, but you can pass your own pre-parsed object
var request = require('request')
var url = require('url')

var query = {function:'get_site_list', version:1, params:{site_list:'11000*'}}
var uri = url.parse('http://watermonitoring.dnrm.qld.gov.au/cgi/webservice.server.pl?'
  + JSON.stringify(query))
uri.path = decodeURIComponent(uri.path)

request({
  uri: uri
}, function (err, res, body) {})

{"buff_required": 351, "buff_supplied": 50000, "error_num": 0, "_return": {"sites": ["11000056A", "11000060A", "11000062A", "11000064A", "11000066A", "11000068A", "11000104A", "11000138A", "11000146A", "11000147A", "11000148A", "110001A", "110001B", "110001C", "110001D", "110002A", "110003A", "110004A", "110004B", "110005A", "110006A", "110006B", "110006C", "110007A", "110009A"]}};

No idea if that's the correct result, but at least it's not HTML :)

@simov - that's the correct result.
Brilliant. The pre-parsed option seems to work. I'll close this one.
Thanks - much appreciated.


51 feature ad view and api call

Description

added view for ad details
added api request and caching

Reference
Issues #51

The picture needs to be fetched in a seperate api request


imgbot seems not work

Link:https://github.com/dromara/fast-request
I have install imgbot to my project and have add the config,  but imgbot seems can not give pr to me!

Hello, to make it work please remove comments from the configuration file, also note that configurable pull request title and body are only available for paid plans. Remove also the schedulefield if you want the pr to be instant and not weekly as you set it up. The ignoreFiles can also be removed if not used. I tested here and everything is working as expected: https://github.com/GrigoreMihai/fast-request/pull/1 . Also change the minKBReduced to 0 to be sure you have the minimum optimization to open a pull request, eg: https://github.com/GrigoreMihai/fast-request/blob/master/.imgbotconfig
Let us know if you receive the optimization after doing the changes in the .imgbotconfig file. Thank you.

I have change the config like example
https://github.com/dromara/fast-request/blob/master/.imgbotconfig
And have click this button

It seems still no PR, does it take too long to provide PR?

Hello, is there any other solution?

Need help

Six days, still no pr!!!

Hello, we checked this issue and there is a limit on how many images we can optimize in a single repository depending on the number of images, size and format. We are currently reviewing this issue and we will do the necessary updates so  we can handle bigger repositories. We will let you know when we fix this.

Hello @kings1990  , we did some updates and got a pr on a fork of your repository https://github.com/GrigoreMihai/fast-request/pulls . You can request two optimization (half of hour apart from each other) for your repository from here: https://imgbot.net/ and you should get a pr in a few hours depending on the current workload.

I see that you got a pr here: https://github.com/dromara/fast-request/pull/105 .


Add a .in file for config.py

Don't use hard-coded default paths; derive them from $(PREFIX).

Fixed in 7492d561f4f2c4604122f890a5c4f4a875b28550


Handle conversion from and to sets

It is rather annoying convert an existing Set[Tree] into a Set[Structurally[Tree]]
We could:
Add a pimp for Set[Tree].
I'm unsure of names at this point...
But we need to have something like
set.usingEquality[Structurally]
or perhaps if we ignore the typeclass approach
set.structurally and set.syntactically which might be easier on most users

Contrib has been stable for a while now and is working great. Let's defer feature requests like this ticket until we have someone driving the work.


Added EMPD2 samples contributed by Tantau

Hi! I'm your friendly automated EMPD-admin bot!
Thank you very much for your data submission! I will now run some tests on your data.
In the meantime: please review your test data on the EMPD viewer using
this link: https://EMPD2.github.io/?repo=EMPD2/EMPD-data&branch=Tantau&meta=Tantau.tsv
This bot is helping you merging your data. I am checking for common issues and I can fix your climate, country or elevation data.
Type @EMPD-admin help in a new comment in this PR for usage information.
Make sure, that all the commands for the @EMPD-admin are on one single line and nothing else. Only
@EMPD-admin help
will work, not something like
<other words> @EMPD-admin help
or
@EMPD-admin help <other words>
Note that you can also run these tests on your local machine with
pytest.

Hi! I'm your friendly automated EMPD-admin bot!
This is just to inform you that I tested your data submission in your PR (Tantau.tsv) and found it in an excellent condition!

@EMPD-admin finish --commit

Hi! I'm your friendly automated EMPD-admin bot!
I processed your command and hope that I can help you!

finish --commit

Finished the PR!
You may want to have a final look into the viewer (https://EMPD2.github.io/?repo=EMPD2/EMPD-data&branch=Tantau) and then merge it.


Improve KF authentication handling

This PR

enables the user to choose an authentication type for Kubeflow Runtime configurations
provides support for an initial set of authentication types
additional authentication type contributions can be provided by the community

Not that this implementation does not provide BYO authentication type support (no plug-and-play like we provide in https://github.com/elyra-ai/elyra/pull/2241 for component catalog types)
Closes #2240
Closes #2107

What changes were proposed in this pull request?

Manual testing using

KF secured using DEX/static pwd
KF secured using DEX/LDAP
KF secured using DEX/static pwd + DEX/LDAP

Current authentication type selections (Auto is the default because it appears to be the most flexible one):

Known limitations:

Irrespective of how many authentication inputs an authenticator requires (e.g. 0 for No Authentication, or 2 for Auto) the UI currently always displays two input fields with static labels (username and password). That said, the authenticators ignore any values a user might enter if they are irrelevant.

Additional infrastructure work is required (and outside the scope of this PR) to make this more flexible and improve the user experience.

How was this pull request tested?
TBD

Developer's Certificate of Origin 1.1
   By making a contribution to this project, I certify that:

   (a) The contribution was created in whole or in part by me and I
       have the right to submit it under the Apache License 2.0; or

   (b) The contribution is based upon previous work that, to the best
       of my knowledge, is covered under an appropriate open source
       license and I have the right under that license to submit that
       work with modifications, whether created in whole or in part
       by me, under the same open source license (unless I am
       permitted to submit under a different license), as indicated
       in the file; or

   (c) The contribution was provided directly to me by some other
       person who certified (a), (b) or (c) and I have not modified
       it.

   (d) I understand and agree that this project and the contribution
       are public and that a record of the contribution (including all
       personal information I submit with it, including my sign-off) is
       maintained indefinitely and may be redistributed consistent with
       this project or the open source license(s) involved.

The latest code drop

Fixes a bug in the NONE/not secured authenticator
Removes the authenticator that was introduced in 3.2
Includes static password and ldap authenticators based on @thesuperzapper's  code (with small modifications). Both authenticators raise an error if KF is not secured or secured using a provider other than the one the user selected.

I haven't had a chance yet to look at the token scenario.

the PR right now would be very confusing to users, the auth providers all overlap in features, and don't actually tell you what kind of authentication they will use.

Agreed and resolved. Wherever possible error messages will indicate if a mismatch between selected auth provider and configured auth provider was detected.

The main motivation for introducing "Auto" was to achieve backward compatibility (users can continue to use their existing KF runtime configurations as is) and to support authentication types that are not explicitly supported otherwise (LDAP and static password) or might not work as expected (for the supported types).
I believe there is a solution that avoids the need for breaking changes and aligns the behavior of "Auto" (perhaps a better name -> "Generic") with the behavior of the other authenticators:

Legacy KF runtime configurations will be implicitly converted to "No authentication" (if no credentials are stored in the configuration) or "Generic" (if id/password are stored in the configuration) using schema filters (need to confirm with @kevin-bates).
Users can still explicitly choose any other authentication type in the runtime configuration as desired. Once they do the selection is persisted and no more implicit auth type conversion as outlined above is performed.
The "Generic" auth type raises an auth error if the KF endpoint is not secured (same behavior as other auth types) and tries to authenticate using the provided credentials without making any changes to the auth flow that KF initiates. The latter is an important differentiator to the LDAP/static password approach: if, for example, DEX responds with a request to choose an authentication type, "Generic" authentication won't pick one.

Legacy KF runtime configurations will be implicitly converted to "No authentication" (if no credentials are stored in the configuration) or "Generic" (if id/password are stored in the configuration) using schema filters (need to confirm with @kevin-bates).

Since "KF runtime configurations" are instances of the kfp schema, these kinds of changes would need to be applied using the metadata_class_name reference such that when an instance of the kfp-schema is used, that instance will actually be an instance of metadata_class_name where we could probably use the on_load() method to make these adjustments and, essentially, migrate on the fly.
Although not immediately necessary at this time, it would be good to introduce a class hierarchy that introduces a base class from which all Runtimes schemas would at least reference.  Then, for cases like this, the kfp subclass of that base would be used to apply kfp-specific changes.
Something like this this:
class Metadata(object):  # today's base instance
    pass

class RuntimesMetadata(Metadata):  # schemaspace-specific metadata class
    pass

class KfpMetadata(RuntimesMetadata):  # applies changes specific to kfp schema
    def on_load(self, **kwargs: Any) -> None:
        pass

Then the kfp schema would add:
"metadata_class_name": "elyra.pipeline.kfp.kfp_metadata.KfpMetadata",

Examples of this will be available once changes for BYO catalogs and RuntimeProcessorType are merged, as we're doing this kind of thing there.

Still working on a solution for Python DSL export.

Latest update

incorporates review feedback (no connectivity check for service account tokens, etc)
adds convenience method to_dict to SupportedAuthProviders enum in support of https://github.com/elyra-ai/elyra/pull/2257#discussion_r741094757
standardizes spelling of  "service account" (not "serviceaccount") to align with Kubernetes naming convention (enum member name is now KUBERNETES_SERVICE_ACCOUNT_TOKEN and all messages and comments should now be consistent)
adds defensive code that properly handles the scenario where a developer adds a new provider to SupportedAuthProviders but forgets to implement it

@thesuperzapper @portellaa @andre-lx Thank you so much for your feedback. Much appreciated! We will soon be cutting a first 3.3 release candidate (waiting for other PRs to be approved). If there's anything that should not/cannot be deferred to a subsequent [patch] release please let me know.


[Bug]: IHaskell syntax highlighting

Current Behavior
It seems like Haskell syntax highlighting is not currently working?
See #126
Expected Behavior

Steps To Reproduce

OS
-

Version
83eafd0b78ee51d0b61b046ac79b1cc90a6226a6
Additional Context
No response
Relevant log output
No response

It looks like syntax highlighting is supposed to work.
https://github.com/tweag/jupyenv/blob/83eafd0b78ee51d0b61b046ac79b1cc90a6226a6/kernels/available/haskell/default.nix#L44

We've solved this problem many times but it won't stay solved. https://github.com/IHaskell/IHaskell/pull/1270


Ensure both node and S3 mismatches with the console data are detected and removed periodically

Part of https://github.com/neondatabase/neon/issues/2606
Buckets with layers could be shared with multiple pageserver nodes, any of them potentially having locally the timeline that should be removed.
If any working pageserver has a branch that's being synced to S3, it has to be deleted from the pageserver first.
After https://github.com/neondatabase/neon/issues/3723, such mismatches should be detectable and are possible to fix in semi-automated way.
We only care about the extra timelines that are present on the nodes: as soon as they are removed, the S3 cleansing could start, no need to wait for the timelines that are missing on pageservers to be downloaded.
This way should be applied for a background task in the console, that finds a bucket, cleans all related pageservers first.
After https://github.com/neondatabase/neon/issues/3724 , there should be a way to do that based on the console management API output.
This logic could be added to a periodic job in the console: with the current design, it's run per node (which presumably would require more DB changes in the console to support the periodic job runs) and does not need to be run frequently: S3 operations might take a while and might cost extra.

https://github.com/SomeoneToIgnore/s3-deleter automates it for the time being and https://github.com/neondatabase/neon/issues/3889 should improve the situation with deletion from pageserver nodes.
Possible future cleanups with the same tool will handle the mismatch checks also.


Add a figure and fix typos

I didn't check if the photo can be shown. Please check this

set the pic


AWSCognitoNetworkPreferences cannot be found after upgrading to 2.x

Describe the bug
We are upgrading our library from 1.x to 2.11.5
Our existing code will not build, as AWSCognitoNetworkPreferences can no longer be found.
We've read the upgrade instructions and amplify codebase but can't find references to the network preferences or an alternative configuration.
import Amplify
import AWSCognitoAuthPlugin

try Amplify.add(plugin: AWSCognitoAuthPlugin(networkPreferences: AWSCognitoNetworkPreferences(
        // Limit number of retries and timeout to prevent spamming of backend after iPad reconnects to internet
        maxRetryCount: 2,
        timeoutIntervalForRequest: 60,
        timeoutIntervalForResource: 60
)))
try Amplify.configure()

Steps To Reproduce
Run the above code

Expected behavior
It continues to work
Amplify Framework Version
2.11.5
Amplify Categories
Auth
Dependency manager
Swift PM
Swift version
5
CLI version
n/a
Xcode version
14.2
Relevant log output
No response
Is this a regression?
No
Regression additional context
No response
Device
iPad 10 simulator
iOS Version
15
Specific to simulators
No response
Additional context
No response

Hi please could you explain why this is tagged as a feature request and not a bug? This code worked in version 1.X

Generally, adding a new API into a major version would be considered new feature. But you're right, it looks like v2 has removed the ability to configure these values, and with that has removed support for this use case.

OK thank you for correcting the label. Do you have an estimate of when this will be prioritised? This bug blocks us from upgrading the library - so it would be good if it was fixed with ample buffer before V1 of the library left maintenance (May 31, 2024)

@grutts Our team is reviewing the effort and changes that would needed to support this feature in Amplify V2. We will post an update on the issue in the next couple of weeks.

Hi @harsh62 do you have an update please?

Hello, do you have an estimate when this will be fixed now that https://github.com/awslabs/aws-sdk-swift/issues/1065 has been merged?

Thanks @atierian - that's a very helpful update

Hi @harsh62 thanks for the update. We should be able to test this w/c 4 December

The fix PR has been released in version 2.24.0. Feel open to reopen this ticket if you still having the issue.


Add AUR To Readme

Why yay instead of pacman?

Why yay instead of pacman?

because pacman is not used for aur, yay is a aur wrapper and it builds packages for pacman


rrule doesn't work with event-recurrence plugin

I'm trying to use the rrule property that comes with the event-recurrence plugin. Maybe there's something I'm missing because when I set my rrule with FREQ=WEEKLY I expected to see the event repeated every week.
I have the following code :
const app = createCalendar({
    ...CALENDAR_CONFIG,
    plugins: [createEventRecurrencePlugin(), createEventModalPlugin()],
    dayBoundaries: {
        start: '07:00',
        end: '00:00',
    },
    views: [viewWeek, viewDay, viewMonthGrid],
    defaultView: viewWeek.name,
    locale: 'fr-FR',
    events: [],
});

function formatTimeEvent(date: string, hours?: string): string {
    let string = date;
    if (hours) {
        string += ` ${hours}`;
    }
    return string;
}

onMounted(async () => {
    const planning = await fetchPlanning();
    for (const event of planning) {
        const start = formatTimeEvent(event.startDate, event.startHours);
        const end = formatTimeEvent(event.endDate, event.endHours);

        app.events.add({
            id: event.id,
            start,
            end,
            title: event.title,
            description: event.description,
            location: event.location,
            calendarId: event.calendarId,
            rrule: event.rrule,
        });
    }
});

And for an example, here an event from my db :
{
    title: 'example',
    startDate: '2023-08-30',
    startHours: '17:45',
    endDate: '2023-08-30',
    endHours: '19:45',
    rrule: 'FREQ=WEEKLY',
}

I read the doc on the events facade part that I had misunderstood, now it works. I close this issue

Great :) I see that you add the events in a loop one after another. How is the performance when doing that?
I plan on implementing an events.set for overriding the whole list of previous events with a new list. Maybe this would be something for you too: https://github.com/schedule-x/schedule-x/issues/268


:gift: Support multiple Golang modules in a single project

Changes

:gift: Support multiple Golang modules in a single project

/kind enhancement
Fixes #85

/assign @knative/productivity-leads
/cc @knative/productivity-reviewers
/assign @knative/productivity-writers

/assign @upodroid
/assign @kvmware
/assign @psschwei
/cc @mgencur
/cc @evankanderson
/cc @shinigambit

+1
We need to have a dummy github repo that we execute a our complete release process on to test these changes.
Changes in this repo broke the 1.7 release which didn't flag up in the nightlies release process.

You also don't want to be patching hack repo on release day as every repo needs to be bumped then resynced properly.

We need to have a dummy github repo that we execute a our complete release process on to test these changes.

There is a test/ directory that serves as such a dummy project. There are some tests, and I'm adding some with this PR as well. But, that's not enough, as you said @upodroid. We can add more tests there... (Writing them in Go, instead of bash could also help)

@cardil: We can add more tests there... (Writing them in Go, instead of bash could also help)

The follow-up PR #215 is doing just that...

This PR looks good to me. I was first confused why there are changes in schema/third_party and schema/vendor but @cardil told me it was generated by running update-deps on this same repo. Personally, I would include it at least in a separate commit with  the title "Run update-deps.sh for this repo", or maybe even include this in a follow up PR. But having this here is ok as well.

/lgtm
/approve
/hold
Unhold when you ready to observe the changes @cardil

/unhold
Let's merge. It should be fine. Will react if otherwise...


⚠ gorzdrav.org has degraded performance

In d7ebaef, gorzdrav.org (https://gorzdrav.org) experienced degraded performance:

HTTP code: 200
Response time: 1617 ms

Resolved: gorzdrav.org performance has improved in b29b5d2.


Major release

Alaska Airlines Pull Request
This update is a BREAKING CHANGE update that totally removes any support for Orion or Classic token support.
Per this release, the npm package has been renamed to @alaskaairux/design-tokens. This will allow for the install of the new Tokens package while having the legacy package installed at the same time. The intent is that this will make it easier to move over to the new conventions and systematically remove all legacy Classic or Orion references.
The following is an updated list of currently supported Design Token resources.
└── tokens
   ├── CSSCustomProperties.css
   ├── CSSCustomPropertiesColorRGB.css
   ├── CSSOpacityProperties.scss
   ├── CSSSizeCustomProperties.css
   ├── JSData--color.js
   ├── JSObject--allTokens.js
   ├── JSVariables--color.js
   ├── SCSSOpacityVariables.scss
   ├── SCSSVariableMap.scss
   ├── SCSSVariables.scss
   ├── SassCustomProperties.scss
   ├── SassCustomPropertiesColorRGB.scss
   └── SassSizeCustomProperties.scss

Other notable changes
The following files have been updated to assume the auro prefix.
   ├── JSData--color.js
   ├── JSObject--allTokens.js
   ├── JSVariables--color.js

In regards to JSVariables--color.js, Issue #31 reported that the names of variables were incorrect. E.g.
export const color-background-lighter = "#f8f8f8";

This was corrected and the output from the previous version is now
export const ColorBackgroundLighter = "#f8f8f8";

In this MAJOR release, these same variables will be output with the Auro prefix. Also noted is that any variable that was for use with Orion or Classic, they are deleted per this release. See the Auro docsite for a complete list of deprecated/deleted tokens.
export const AuroColorBackgroundLighter = "#f8f8f8";

In regards to the JSObject--allTokens.js and JSData--color.js files, the convention is the following:
"name": "auro-color-alert-notification-on-light",

All other Auro tokens are already using the auro prefix, so there are no changes there.
$auro-color-alert-notification-on-light: #0074cb;
--auro-color-alert-notification-on-light: #0074cb;

Type of change:
Please delete options that are not relevant.

[ ] New capability
[x] Revision of an existing capability
[ ] Infrastructure change (automation, etc.)
[x] BREAKING CHANGE

Checklist:

[x] My update follows the CONTRIBUTING guidelines of this project
[x] I have performed a self-review of my own update

By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Pull Requests will be evaluated by their quality of update and whether it is consistent with the goals and values of this project. Any submission is to be considered a conversation between the submitter and the maintainers of this project and may require changes to your submission.
Thank you for your submission!
-- Orion Design System Team

:tada: This PR is included in version 3.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Phaser renderer floating text remove pixi

https://github.com/moddio/taro/blob/phaser-renderer-floating-text-remove-pixi/engine/core/IgeEntity.js#L4278-L4283
this needs to be addressed, probably replaced with events to show/hide name label that get handled on phaser rendering side