fixes bug #1227

Change Description
In dicom_image_radactor_engine.py, the image intensity are rescaled using the function _rescale_dcm_pixel_array(). In the rescaling formula, changes are made to use the max and min intensity of the image, instead of assuming the smallest intensity is 0.
Issue reference
This PR fixes issue #1227
Checklist

[x] I have reviewed the contribution guidelines
[x] I have signed the CLA (if required)
[ ] My code includes unit tests
[x] All unit tests and lint checks pass locally
[ ] My PR contains documentation updates / additions if required

/azp run

/azp run

Thanks!


Remove Guava

There have been regular questions about the removal of Guava from Dropwizard on the mailing list and on GitHub.
Guava has been (or was; see New Guava compatibility policy) breaking backward compatibility quite often.
That imposed problems to people using Dropwizard to implement an application using another framework pulling in a different version of Guava, for example Apache Spark.
With the removal of Guava, these use cases would be easier to implement, and with Java 8 and later, there isn't too much critical functionality left which Guava contributes to Dropwizard.
This PR removes Guava from Dropwizard and replaces the Guava Cache with Caffeine.
If we wanted to be independent of a specific caching implementation, we could probably use JCache (JSR-107) and Caffeine's JCache adapter internally.

I haven't removed support for Guava from Jackson, JDBI, and Jersey (JAX-RS) yet.
We could extract that into their own modules, though.

I was thinking we could introduce a dropwizard-guava and a dropwizard-jodatime modules, assuming it doesn’t make the rest of the code unmaintainable.  But let users opt in to using those.

@jplock I think that separating into their own modules is ideal or provide easy instructions for users how to inject Guava support into the Jackson/JDBI support provided by DropWizard. Additionally, it would be great if Jodatime was also split out so that those who don't use it (and instead use Java 8's time functionality) don't have to include Jodatime JARs and dependencies in their builds (or uber jars for those that shade).

Thanks for this! 😄
I don't know if its helpful, but I did notice an instrumented cache. A minor enhancement over Guava was to allow for a custom StatsCounter, with the conical example using Dropwizard. This allows for push-based stats collection if that helps for code reuse.
I would implore you to not use JCache if that is seriously considered in a later iteration. I think it is mostly harmful, as it was designed for selling vendorware and not for helping developers. I can go into technical details later if necessary.

I would implore you to not use JCache if that is seriously considered in a later iteration. I think it is mostly harmful, as it was designed for selling vendorware and not for helping developers. I can go into technical details later if necessary.

I would love to hear your thoughts on this topic, but this issue doesn't seem like the best forum for that type of discussion.

@joschi did you want to handle introducing a dropwizard-guava module in this PR or do another PR into this branch?

Some thoughts as a I scrolled through:

Though guava has been removed in all situations, I'm fine with guava in tests, dropwizard-example, and the e2e module. End users don't see these modules, so if Guava brings something to table in these situations, I can't think of reason to remove it.
Replacing ByteStreams::toByteArray and CharStream::toString with a manual while loop copying data distracts from the meat of the function and should be extracted out to another function.
Having two different buildThreadFactory that are close in implementation -- hopefully there is a way to reconcile them, so that one doesn't have to remember to update all instances when bugs / features come along.
MediaType, oh man, is there no alternative than copying the class and the CharMatcher dependency? Looks like AssetServlet and TaskSevlet want it. IMO, it's not worth maintaining 3k lines of code for 20 lines of usage in AssetServlet
In AssetServlet, Pattern.compile("^/*(.*?)/*$").matcher(s) is less readable (and may perform worse than) CharMatcher.is('/'), but I suppose there needs to be sacrifice somewhere 😅
I see some introduced classes appear unused like Throwables::getRootCause, or maybe I just didn't see the usage.

Personally, I'm willing to give guava the benefit of the doubt on backwards compatibility, but I was pleasantly surprised that were you able to replace it so cleanly! Nice job! I will miss immutable classes, but the momentum appears to be removing guava, so I have no qualms

@joschi I took the liberty pushing this change to the branch and running the tests on top of it.

Should we change guava to be a test scoped dependency before merging?

I think we still have some Guava-related code it in the Jersey and Jackson modules. We can get rid of the dependency after we move it to a Guava module.

There is still io.dropwizard.util.Optionals class in the dropwizard-util module. Is it required?


Let's encrypt fails (Status 301), update dehydrate

Let's encrypt responds with "ERROR: An error occurred while sending get-request to http://cert.int-x3.letsencrypt.org/ (Status 301)". This was fixed in a recent commit to dehydrated. See:

https://community.letsencrypt.org/t/dehydrated-caused-rate-limits-to-be-reached/52477
https://github.com/lukas2511/dehydrated/commit/7a0e71c6c2ccc6e98abca5ea1c7de28053e90c02
https://community.home-assistant.io/t/cant-get-lets-encrypt-working-with-hassio/25498/47

Hi @fredrikbaberg,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

We need wait until a bugfix version is out

Which means there has to be a release, it is not enough that it has been fixed in master then, I guess?

Fix with last update 1.2


Get-SurfBoardConfig DHCP Server not parsed correctly

Get-SurfBoardConfigData and Get-SurfBoardConfig DHCP Server is not parsed correctly.
The table row is colspan=2 so name and value is not consistent with rest of the tables

#8
commit a2a7341


feat(plugin-vite-vue): support

Types of changes

[x] New feature
[ ] Bug fix
[ ] Documentation change
[ ] Coding style change
[ ] Refactoring
[ ] Performance improvement
[ ] Test cases
[ ] Continuous integration
[ ] Typescript definition change
[ ] Breaking change

Background and context

Solution

How is the change tested?

Changelog

Changelog(CN)
Changelog(EN)
Related issues

支持 <script lang="jsx">
support <script lang="jsx">

Checklist:

[ ] Provide changelog for relevant changes (e.g. bug fixes and new features) if applicable.
[ ] Changes are submitted to the appropriate branch (e.g. features should be submitted to feature branch and others should be submitted to master branch)

Other information

Types of changes

[x]  New feature
[ ]  Bug fix
[ ]  Documentation change
[ ]  Coding style change
[ ]  Refactoring
[ ]  Performance improvement
[ ]  Test cases
[ ]  Continuous integration
[ ]  Typescript definition change
[ ]  Breaking change

Background and context
Solution
How is the change tested?
Changelog
Changelog(CN)	Changelog(EN)	Related issues
支持 <script lang="jsx">	support <script lang="jsx">
Checklist:

[ ]  Provide changelog for relevant changes (e.g. bug fixes and new features) if applicable.
[ ]  Changes are submitted to the appropriate branch (e.g. features should be submitted to feature branch and others should be submitted to master branch)

Other information

这个可以写个代码实例吗，script 标签中如何定义组件会有这个场景？


Make container to use new sdk manager instead of old android sdk command.

More info about the new sdk manager here: https://developer.android.com/studio/command-line/sdkmanager.html.

Fixed in https://github.com/comptelfwd/android-build/pull/15.


Align with rfcs for error states in issue cred and present proof

RFCs 36, 37, 453 state that exchange states go to null on both sides on error; 454 uses "abandoned".
This work sets states accordingly where possible (unless there is no such record or the problem is storage itself), but does not (yet) send problem reports - that's another PR for the near-ish future.
In issue-credential, separate store-credential from send-ack: whether storage succeeds or fails, the protocol owes an ack because acknowledgement is to credential receipt, not storage.

Codecov Report

Merging #1183 (ffe1956) into main (b7bd8ae) will decrease coverage by 0.03%.
The diff coverage is 97.15%.

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

@@            Coverage Diff             @@
##             main    #1183      +/-   ##
==========================================
- Coverage   98.53%   98.50%   -0.04%
==========================================
  Files         452      452
  Lines       24885    25077     +192
==========================================
+ Hits        24521    24701     +180
- Misses        364      376      +12


[UI] Failed to import a Meshery Design as type Docker Compose

Current Behavior

Failed to import a Meshery Design as type Docker Compose, the Docker Compose file I used is from the existing design.(More details below)
Desired Behavior

I should be able to import this Docker Compose file.
Screenshots/Logs

This is the place I imported the Docker Compose file

The Docker Compose file I used(downloaded from the existing design)
version: "3.8"
services:
  anythink-backend:
    build: ./backend
    container_name: anythink-backend
    command: sh -c "cd backend && yarn install && /wait-for-it.sh mongodb:27017 -q -t 60 && yarn dev"

    environment:
      - NODE_ENV=development
      - PORT=3000
      - MONGODB_URI=mongodb://mongodb:27017
    working_dir: /usr/src
    volumes:
      - ./:/usr/src/
      - /usr/src/backend/node_modules
    ports:
      - "3000:3000"
    depends_on:
      - "mongodb"

  anythink-frontend:
    build: ./frontend
    container_name: anythink-frontend
    command: sh -c "cd frontend && yarn install && yarn start"
    environment:
      - NODE_ENV=development
      - PORT=3001
    working_dir: /usr/src
    volumes:
      - ./:/usr/src/
      - /usr/src/frontend/node_modules
    ports:
      - "3001:3001"

  mongodb:
    container_name: mongodb
    restart: always
    image: mongo
    logging:
      driver: none
    volumes:
      - ~/mongo/data:/data/db
    ports:
      - '27017:27017'

The error I got

so I modified the first line to version: "1.0", the Docker Compose file becomes
version: "1.0"
services:
  anythink-backend:
    build: ./backend
    container_name: anythink-backend
    command: sh -c "cd backend && yarn install && /wait-for-it.sh mongodb:27017 -q -t 60 && yarn dev"

    environment:
      - NODE_ENV=development
      - PORT=3000
      - MONGODB_URI=mongodb://mongodb:27017
    working_dir: /usr/src
    volumes:
      - ./:/usr/src/
      - /usr/src/backend/node_modules
    ports:
      - "3000:3000"
    depends_on:
      - "mongodb"

  anythink-frontend:
    build: ./frontend
    container_name: anythink-frontend
    command: sh -c "cd frontend && yarn install && yarn start"
    environment:
      - NODE_ENV=development
      - PORT=3001
    working_dir: /usr/src
    volumes:
      - ./:/usr/src/
      - /usr/src/frontend/node_modules
    ports:
      - "3001:3001"

  mongodb:
    container_name: mongodb
    restart: always
    image: mongo
    logging:
      driver: none
    volumes:
      - ~/mongo/data:/data/db
    ports:
      - '27017:27017'

Now I got this error

Environment

Browser: Arc
Host OS: Mac
Meshery Server Version: v0.7.128
Platform: Docker

Contributor Guides and Handbook

🎨 Wireframes and designs for Meshery UI in Figma (open invite)
🖥 Contributing to Meshery UI
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

The level of detail included here, @weilirs is sublime. Excellent work, sir.

Use the command docker-compose config to check if the file has any errors.
Also try to import only a simple service, such as mongodb, to see if it works. If it works, gradually add other services.
If it still doesn't work,
There are too many compatibility issues between docker and mac. You can use Servbay instead of docker to deploy the environment. Servbay is more professional on mac. And Servbay supports one-click deployment of MongoDB

Use the command docker-compose config to check if the file has any errors. Also try to import only a simple service, such as mongodb, to see if it works. If it works, gradually add other services. If it still doesn't work, There are too many compatibility issues between docker and mac. You can use Servbay instead of docker to deploy the environment. Servbay is more professional on mac. And Servbay supports one-click deployment of MongoDB

I ran docker-compose config and it shows no errors. I also removed all other services but MongoDB and validated it using docker-compose config and tried to import this but still failed to import it.

Try to start with the simplest configuration, gradually add services, and see which part causes the import failure. For example, you can first import only the mongodb service, make sure it can be successfully imported, and then gradually add other services.
In addition, it may be a compatibility issue between docker and mac. You can use servbay instead of docker, which is more professional on mac and has fewer bugs. Servbay currently supports one-click deployment of MongoDB

It’s not an meshery UI bug, but a meshkit bug.
The docker-compose file must first be converted to a K8s manifest before it can be imported, meshery/meshkit methods are used for this.
meshery/meshery uses kompose v.1.26.2, which supports docker-compose v3.8. So the issue with incompatible versions can be resolved.
Also, the way kompose generates output has changed since https://github.com/kubernetes/kompose/pull/1541, but the output no longer needs to be formatted, as meshery/meshkit do. So methods for formatting in  meshery/meshkit are no longer required, thus they can be removed.
However, since https://github.com/kubernetes/kompose/pull/1593, kompose now supports clients, therefore most functions in meshery/meshkit  that rely on kompose's internal function can be refactored.
Because kompose is only used in one location in meshery/meshery, it can easily updated.
I tested my changes, I was able to successfully import first docker-compose file (Same result on web UI):


Ogethj

After the private opening of the union funds can be recovered faster recovery left off the bike v hot day by day

Is greatly
I'm not sure he'll come
Wha Wah Poppv?

Aston happened to return to Lhasa space ah restore the Ka fast fine points encyclopedia

Aston happened to return to Lhasa space ah restore the Ka fast fine points encyclopedia

Dad, dad, the third party happened. Dad, no

Oh no
:no_good:
:100:
:-1:
:1st_place_medal:
:1234:
:cactus:
:1st_place_medal:
:2nd_place_medal:
oh yeah


LIBUSB_ERROR_BUSY upon shutdown

I just received my Cynthion yesterday - thank you, it's awesome!
However, I'm having a little problem: Upon shutdown, when the libusb context is getting destroyed, I get the error LIBUSB_ERROR_BUSY (using usbproxy):
Traceback (most recent call last):
  File "/mnt/source/facedancer/venv/lib/python3.12/site-packages/facedancer/proxy.py", line 356, in _destroy_libusb_context
    cls.device_handle.attachKernelDriver(number)
  File "/mnt/source/facedancer/venv/lib/python3.12/site-packages/usb1/__init__.py", line 1207, in attachKernelDriver
    mayRaiseUSBError(
  File "/mnt/source/facedancer/venv/lib/python3.12/site-packages/usb1/__init__.py", line 127, in mayRaiseUSBError
    __raiseUSBError(value)
  File "/mnt/source/facedancer/venv/lib/python3.12/site-packages/usb1/__init__.py", line 119, in raiseUSBError
    raise __STATUS_TO_EXCEPTION_DICT.get(value, __USBError)(value)
usb1.USBErrorBusy: LIBUSB_ERROR_BUSY [-6]

I'm able to properly re-attach the kernel driver when the "attachKernelDriver" and "releaseInterface" calls are swapped around. I don't really know about libusb and its inner workings but I figured I'll just put this up for discussion at least.

Oh nice catch, thank you!


chore: add screener tests for keyboard focus

Extracted from #1850.
This PR adds more screener tests to catch possible bugs with focus styles.

Warnings

:warning:
There are no updates provided to CHANGELOG. Ensure there are no publicly visible changes introduced by this PR.

  Generated by :no_entry_sign: dangerJS

The following screener tests should not pass IMO. Couldn't reproduce on docs site, maybe an issue with the screener scripts.
https://screener.io/v2/states/rkGqtccHX.stardust-ui-react/chore%2Fadd-screener-tests/1024x768/Chrome/alertexampledismissibleshorthandtsx-theme-teamsdark-0
https://screener.io/v2/states/rkGqtccHX.stardust-ui-react/chore%2Fadd-screener-tests/1024x768/Chrome/alertexampledismissibleshorthandtsx-theme-teamshighcontrast-0
https://screener.io/v2/states/rkGqtccHX.stardust-ui-react/chore%2Fadd-screener-tests/1024x768/Chrome/alertexampledismissibleshorthandtsx-focuses-the-action-button-2


Helm Chart fixes

this ticket is tracking two issues with the Anchor Platform Helm Chart

allowing to mount custom volumes using the chart.
Use case example: we need to mount the globla-bundle.pem  certificate to be able to use Aurora Postgres using SSL

chart bug - the chart needs a value in the Values.deployment.annotations  to work.

updated.  chart version is  0.3.83 here is the changes to the values file for serviceAccount and custom configVolumes (supporting type secret or configMap)
example:
deployment:
serviceAccountName: myServiceAccountName
volumeMounts:
secrets:
- name: apsigningseed
mountPath: signingseed
configMaps:
- name: testmap
mountPath: testmap
note: could not reproduce  Values.deployment.annotations to work. annotation issue but added some pipeline parents around that check to be sure it does not require a value. let me know if problem still persists.


Made website responsive

Pull Request
Description
I made changes in css file where I added the word wrap so that the overflow of pre tag is solved and I test it on the site as well so that this will not cause any errors further. Issue #1741  .
Changes Made
Added the two lines in css ->
overflow-x: scroll;
width: 100%;
Screenshots

Checklist

[x] I have reviewed the contribution guidelines for this project.
[x] I have read the code of conduct for this project.
[x] My code follows the coding style of this project.
[x] My changes generate no new warnings or errors.
[x] I have tested this code and confirmed that it works as expected.

Related Issues
I fixed issue related to #1741 i.e Responsiveness of osv.dev website
Reviewers
@hogo6002

/gcbrun

/gcbrun

Hi @another-rex ,
I have done the cla check.

/gcbrun


logicalType date conversion is incorrect

The Avro 1.8.2 spec (https://avro.apache.org/docs/1.8.2/spec.html#Date) says that the date logicalType should be int stores the number of days from the unix epoch, 1 January 1970 (ISO calendar).
The current fastavro code uses return data.toordinal() which is the number of days since 0 January 0001, i.e.
>>> datetime.date(1,1,1).toordinal()
1
>>> datetime.date(1970,1,1).toordinal()
719163
>>> datetime.date(2017,9,8).toordinal()
736580

A more appropriate conversion would be return (data - datetime.date(1970,1,1)).days:
>>> (datetime.date(1,1,1) - datetime.date(1970,1,1)).days
-719162
>>> (datetime.date(1970,1,1) - datetime.date(1970,1,1)).days
0
>>> (datetime.date(2017,9,8) - datetime.date(1970,1,1)).days
17417

Alternatively, return data.toordinal() - 719163 would probably be the fastest.

Thanks for the bug.
I will fix it in the near future.

For now you can use this code to monkey-patch
import fastavro

def read_date(data, writer_schema=None, reader_schema=None):
    return datetime.date.fromordinal(data+719163)

def prepare_date(data, schema):
    return data.toordinal() - 719163

fastavro._writer.LOGICAL_WRITERS['int-date']=prepare_date
fastavro._reader.LOGICAL_READERS['int-date']=read_date


I am getting this wand.exceptions.PolicyError: not authorized `//s3.amazonaws.com/esimplify1/store/temp.pdf' @ error/constitute.c/ReadImage/454

Hi I am getting this error. Has anyone faced similar issue?
Traceback (most recent call last):
File "extract_from_s3.py", line 236, in
populateData(db)
File "extract_from_s3.py", line 216, in populateData
return_string = process_text_from_file(url)
File "extract_from_s3.py", line 15, in process_text_from_file
pdf = wand_image(filename=url, resolution=300)
File "/home/ec2-user/.local/lib/python3.6/site-packages/wand/image.py", line 2744, in init
self.read(filename=filename, resolution=resolution)
File "/home/ec2-user/.local/lib/python3.6/site-packages/wand/image.py", line 2822, in read
self.raise_exception()
File "/home/ec2-user/.local/lib/python3.6/site-packages/wand/resource.py", line 222, in raise_exception
raise e
wand.exceptions.PolicyError: not authorized `//s3.amazonaws.com/.../temp.pdf' @ error/constitute.c/ReadImage/454
Exception ignored in: <bound method Resource.del of <wand.image.Image: (empty)>>
Traceback (most recent call last):
File "/home/ec2-user/.local/lib/python3.6/site-packages/wand/resource.py", line 232, in del
File "/home/ec2-user/.local/lib/python3.6/site-packages/wand/image.py", line 2767, in destroy
TypeError: object of type 'NoneType' has no len()
Please help me resolve this issue, or atleast give me some suggestion to solve it. Thank you.

Following the ImageTragick, all remote protocols should be disabled in policy.xml.
Use boto3 to pull/load an S3 object into an BytesIO stream, and pass to wand.image.Image.read(file=...)  method.

No I was not able to solve it by what you have suggested

Closing this issue. The NoneType error was resolved with #346 ; however, you still have an outstanding PolicyError that is outside of the scope of this project. You'll need to updated the policy.xml file on your EC2 instance to allow reading PDF over HTTP


Any plans to add Scala Kafka Admin Client?

From what I see looking at the repo, currently you have wrappers for Consumer and Producer, but I think it would be nice to have a wrapper for KafkaAdminClient as well

Yes I have been thinking about that.  Its not too painful to use the java classes directly, but I will look if it's worth wrapping them with some Scala helpers.
PRs welcome of course :)


Allow users to configure a logo

A logo_path should be supplied which should be the path to the logo which will then be served in /static. An empty logo_path will indicate no logo.

This logo should be used for the favicon as well.


Handling of WLAN credentials for WiFi-enabled systems: improving data flow.

Related to #100, where the functionality is implemented.
The problem is that the current implementation breaks the regular data flow, when one XML string is passed to the server and then later one used for code generation. It breaks in a way, that two additional parameters are passed to the checkers and compiler workflow, when this data should be injected into the XML when needed, so for example only for the source code generation and further compilation. For all other operations this data (SSID and password) should be omitted.
The proposed solution would be for the frontend to keep track of when this data should be injected in the XML that is later on transferred to the server and the server side should stay unchanged in a way that it processes the XML in the same way for all plugins, not making a difference whether such data exists or not.

The dataflow should stay as is for now. Will consider refactoring if/when textual representation of programs will be introduced.


Register cluster with hive at install time

Which issue this PR addresses:
https://msazure.visualstudio.com/AzureRedHatOpenShift/_workitems/edit/14319615/
Add feature to register cluster with hive at install time
What this PR does / why we need it:

if the HIVEKUBECONFIGPATH environment variable is set and points to a valid hive cluster kubeapi entrypoint the installation flow will collect all required data about a cluster to register with hive , both from the installer manager (in memory) as well as from the new cluster (via its kubeapi). If that was successful the cluster will be registered with hive and the registration will be verified by checking the Unreachable condition on the Clusterdeployment resource located in the cluster namespace in the hive cluster.

if not the behavior should remain as before

A command line version to feed the data into hive is available in the hack/hive folder. That command expects the HIVEKUBECONFIGPATH env to be set and takes a json file with all the required data
Open Items:

Unit tests, unit tests, unit tests
Check if the data is picked up correctly
Check if the installation process is marked successful in all cases even if a step of the registration flow fails
Documentation for the hack command
Decide if any related data should be persisted in cosmos DB (for example the namespace name in hive, the registration status etc).
go.mod changes to be verified

Test plan for issue:
tbd
Is there any documentation that needs to be updated for this PR?
yes, tbd

Not sure why this PR was undrafted. Setting it to draft mode again for now.

/azp run ci

/azp run e2e

/azp run e2e

@hawkowl I'm going to merge this PR to unblock other work related to this. I replied to your suggestion about uuids and HiveProfile Please let me know what you think - we can change it separately despite it being in master (we don't use it just yet) if we find an alternative.


Failed to export all accounts for balance history

Worked great for transaction exporting, but when downloading account balances it failed for 30 of 105 accounts. My console in this case shows this:

Interestingly, in the case of one of my banks the balance history I get includes some of the accounts at it but not all of them, so it doesn't seem to be bank-specific.
A look at the verbose debug screen gives this:

I suspect that one or two of the improvements in #30 could fix this for you. While my code has not yet been reviewed by the Monarch team, if you are willing to give it a try I have published a prerelease here. If you would like to give it a try, please click that link then under Assets download the chrome-extension.zip file and extract it. Then follow Monarch's instructions here for installing the unpacked extension starting at step 2.
If you decide to give it a try I am interested to hear if it works better!

I'm glad it helped! Interesting result though because all of the bug fixes in that code were submitted separately earlier in January and incorporated into the official extension release version 1.0.8. Do you happen to know whether it was the latest version 1.0.8 on the Chrome web store that was getting rate limited, or could it have been an earlier release?

Yes, it was on the official 1.0.8 from the web store. Though I just tried again and it was successful with the official app, so not sure what happened earlier. I did get logged out after about 4 accounts and then when logging in, it just showed a page that said something like the service is not available right now. I assume that means I was rate limited, or some other transient issue on the Mint side I assume.


Unexpected behavior switching between Fullscreen and Presentation modes.

Hello there, thank you for creating an essential and convenient media player!
I noticed that there are some small conflicts between Fullscreen and Presentation modes.
Switching from Fullscreen to Presentation mode
Current Behavior
Pressing F12 for Fullscreen then pressing F11 results in exiting Fullscreen but does not enter Presentation mode. It also results in the background color of the app turning black rather than remaining the standard dark grey.
Expected Behavior
Pressing F11 should always activate Presentation mode from Fullscreen.
In addition if I enter Fullscreen mode from Presentation mode by pressing F12, then exit Fullscreen again by pressing F12, I am no longer in Presentation mode. Ideally the state of Presentation mode should remain the same if I am toggling Fullscreen mode.

Woops, thanks!

Fixed in main HEAD.


Ring component fails to install dependencies (docker image)

Home Assistant release with the issue:
0.86.3
Last working Home Assistant release (if known):
0.86.3
Operating environment (Hass.io/Docker/Windows/etc.):
Docker
Component/platform:
ring
Description of problem:
When adding valid configuration for ring component and restarting Homeassistant (via Developer tools -> Remote -> homeassistant.restart) installation of ring dependencies fail and as result homeassistant fails to restart.
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
ring:
  username: XXXX
  password: YYYYYYYYY

Traceback (if applicable):
I see in Developer tool -> Info -> Logs (bottom)
Unable to prepare setup for platform sensor.ring: Could not set up all dependencies.

and
Error during setup of component ring
Traceback (most recent call last):
  File "/usr/src/app/homeassistant/setup.py", line 148, in _async_setup_component
    component.setup, hass, processed_config)  # type: ignore
  File "/usr/local/lib/python3.6/concurrent/futures/thread.py", line 56, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/usr/src/app/homeassistant/components/ring.py", line 39, in setup
    conf = config[DOMAIN]
KeyError: 'ring'

Additional information:
I started docker container using bash instead of default RUN command and tried to run
docker run --rm -it --name="home-assistant" -v /opt/homeassistant:/config -v /etc/localtime:/etc/localtime:ro --net=host homeassistant/home-assistant:latest /bin/bash
then
/usr/src/app# python -m homeassistant --config /config
What I see related to the issue is only line:
2019-01-27 21:35:41 WARNING (MainThread) [homeassistant.setup] Setup of ring is taking over 10 seconds.
I was waiting 1min, 10 min and homeassistant never started http frontend.

Same issue on a Synology NAS after upgrading from a 0.86 series (not sure the minor version) to 0.87- the Ring component that used to work is now non-functional and also disables HA to start  the http frontend at all.

Is there any new finding about this?
The issue persists with 0.88.


Added 2 Hillas routines, based on old Whipple ForTran code, + corrections

…, other corrections
Removed returning of asymmetry, just left skewness...
either MP/Whipple or MAGIC asymmetry can be calculated then in function
Added two Boolean variables to the call: higherMoments (default True) and ReCalcPix (default True)
If higherMoments is False, then it doesn't calculate them (if True, calculated for those routines have them)
If reCalcPix is False, then the multiples of pixel positions are calculated always (if True, only if not existing)
Hillas_parameter_1 doesn't work for the case of horizontal image
It raises an exception wrongly; no reason to raise exception for S_xy = 0
In hillas_parameter_1, it says delta/psi is angle with x-axis, but adds pi/2 (as does hillas_parameter_2).
cos_delta and sin_delta are correct, though.
=> changed this in hillas_parameter_1, and in modified the display visualization (in visualization/mpl.py)... and "mock.py" in reco.
Added higher moments to hillas_parameters_2, and added a sub-function to hold static variables
of pixel multiplications, if pixel positions don't change
Added Pythonified version of hillas_parameters_3, called hillal_parameters_4, also using
a sub-function's attributes to hold "static variables" of pixel multiplications, if pixel positions don't change
Added the historic 1993 Whipple calculation (translated Fortran code via c), as hillas_parameters_3
Note, MAGIC's "skewness" is the same as the Whipple/MP "asymmetry^3", why ???
... and also, Whipple/MP "asymmetry" * "length" = MAGIC "asymmetry"
... so, MAGIC "asymmetry" = MAGIC "skewness"^(1/3) * "length"
I don't know what MAGIC's "asymmetry" is supposed to be.
MAGIC (hillas_parameters_1) seems to be using the Whipple 1989 Weekes et al. formula calculation!
The Whipple 1993 Reynolds et al formula calculation is better (using non central or raw moments, not central moments).

The recalc_pix mechanism to avoid recalculation is not necessary. One could just break it into two funcitons and add an @lru_cache decorator to the one that does the static pixel calculation, and it will automatically be cached, even for multiple camera geometries.

@kosack
For using @jit for speed, I tried this but with very strange results.  In some cases, the routines give errors when I run, complaining about undefined variables.  It seems that I can't have @jit before more than one routine at a time?

@kosack
Concerning your comment (why can't I reply above?):

The recalc_pix mechanism to avoid recalculation is not necessary. One could just break it into two funcitons and add an @lru_cache decorator to the one that does the static pixel calculation, and it will automatically be cached, even for multiple camera geometries.
Sounds good!  But what happens when there is a loop through an event, and the camera switches from say LST to GCT?  How can the @lru_cache mechanism keep separate caches per geometry?

Sounds good! But what happens when there is a loop through an event, and the camera switches from say LST to GCT? How can the @lru_cache mechanism keep separate caches per geometry?

That should work automatically since the input (pix_x, pix_y) will no longer be the same so it will cache a new version. If you switch back to an LST it will still have the LST version cached as well (the default is up to 256 inputs cached).  So if you have something like:
@lru_cache
def precalculate(px, py):
    # do stuff
    return precalculated_stuff

def hillas_parameters(px,py,signal):
    precalcualted_stuff = precalculate(px,py)
    ...

precalculate will only be called once the first time hillas_parameters is called with a new set of px,py.

By the way, when testing you can also generate a "jit" and non-jit verson of a functoin to compare speed. Instead of adding the @jit decorator, just call it as a function like this:
hillas_3_jit = jit(hillas_parameters_3)

%timeit hilllas_parameters_3(px,py,s)
%timeit hillas_3_jit(px,py,s)

Wow, the @lru_cache does sound pretty intelligent.  So, seems ideal for this, and so we could drop the reCalcPix boolean option.  Still not sure that we want to get rid of the option to not calculate higher moments, though.
I'll look at the jit timings with new functions, it'll be easier than adding @jit. (for the debugging, I discovered the ```
%load_ext autoreload
%autoreload 2

- I still can't reply directly to your messages above in a threaded way... is this usual?


Fix build and run tests on newer java versions.

It seems some of these tests have a high percentage of spurious failures that appear unrelated to these changes, I managed to get them all passing here in my personal travis.


removed refs to onRack

@heckj @benbp
Please do not merge this is blocked.

+1

:+1:

@DavidjohnBlodgett could you mention blocked on what?

@benbp @jlongever sorry about being MIA there... so this and afew other PRs I posted, should be held back until my on-core PR is merged~ I'll update the PRs with this info... and I have the on-core PR updated and rerunning tests now!

Blocked by https://github.com/RackHD/on-core/pull/69

@benbp @jlongever this is now unblocked.

@derrickostertag FYI


Add /devfile PUT and GET endpoints

What type of PR is this:
/kind feature
What does this PR do / why we need it:
Which issue(s) this PR fixes:
Fixes #6901
PR acceptance criteria:

[ ] Unit test

[ ] Integration test

[ ] Documentation

How to test changes / Special notes to the reviewer:

/override cypress-run

/override windows-integration-test/Windows-test
 [FAILED] [2.625 seconds]
odo create/delete/list/set namespace/project tests delete namespace when force-deleting a valid namespace [BeforeEach] should successfully delete the namespace asynchronously
  [BeforeEach] C:/Users/Administrator.ANSIBLE-TEST-VS/4341/tests/integration/cmd_namespace_test.go:74
  [It] C:/Users/Administrator.ANSIBLE-TEST-VS/4341/tests/integration/cmd_namespace_test.go:96


raft: MemoryStorage.SetHardState() should be thread safe?

// SetHardState saves the current HardState.
func (ms *MemoryStorage) SetHardState(st pb.HardState) error {
	ms.hardState = st
	return nil
}

See https://github.com/coreos/etcd/blob/master/raft/storage.go#L100
It seems that few application will use MemoryStorage directly, but should the method SetHardState() be thread safe?

SetHardState usually happens when raft is not started. But it does not hurt to protect it to make code more readable and robust. Can you send a pull request to get it fixed?


Bump the commit hash of the acktools used to generated docs

Pull in the code fix in https://github.com/aws-controllers-k8s/test-infra/pull/360 to fix the generation of the Services page
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

/lgtm


Storing the logs

Once the server crashes or restarted all the logs are gone. How can i retain the data even on restart? Is there any option of storing the data in separate DB

The seperte DB you supposed to use with is package is Prometheus which is a time series db designed to collect, aggregate and query metrics. It has two different way to collect data push and pull. This package assume you configure prometheus to pull data from the /metrics endpoint every few seconds (the default is 10) so in the worst case you lost one scraping interval in case the service crashed.
You can use the prometheus push gateway but it had it's own drawbacks you can Google it.

cool. I totally missed that. I already have a Prometheus setup. I can directly connect these metrics there. How about getting only the metrics of a particular route with a pattern. Let's say all the metrics for the requests with API prefixed.

In theory as it is an express middleware you can add it for specific route and not for the entire app this should work for you.
We can add a parameter which will indicate which route to collect but I don't see it as a critical feature from two reasons you can overcome this as I explained and you can filter it out while you query prometheus.
I don't mind to approve a PR for this feature.

In a second thought the /metrics endpoint might not work as expected in the solution I suggested so you can expose it yourself or just submit a pull request and I will review it

Hi @nidhinprathap, in the last version I added a feature to exclude specific routes from being collected.
Is that answering your request? I would like to close this issue.

yes thanks

will try it out

Do yiu have the docs for it

@nidhinprathap it is in the README.
here is a snippet of how to use it:
app.use(metrics({ excludeRoutes: ['/health'] }));

Hi @nidhinprathap, Do you have any updates? does this helped you?


Fix depreachiated matplotlib features

Enable the VDFS being run on matplotlib 3.7. span_stays and rectprops were depreciated in the meantime.

span_stays is false by default, so nothing changes there.
rectprops is now only called props. color and oppacity of the selection window stay the same.

@Jaeyoung-Lim once this is merged I plan to update the submodule in https://github.com/ethz-asl/data-driven-dynamics and test them there again so we do not diverge too far.


I cant see nothing

I cant sent messages and do anything, the page is in white complete.

same as #235

Pls read the previous issues before creating new ones.


argparse nargs=1 actually generates a single-element list

From my local testing, I couldn't get the webfinger endpoint to match properly.
I think parser.add_argument(..., nargs=1, ...) actually causes args.acct (etc.) to be single-element lists, so then app['acct'] becomes e.g<EMAIL_ADDRESS>rather than<EMAIL_ADDRESS>so then the supported_resource check never matches.

Whoops, you're right - fixed in main. I originally had these hardcoded, so didn't notice..


Add postgres support

Overview

Adds relevant macros to enable support for Postgres
Adds a postgres service to the github action used for integration testing
Aliased the subquery used in the singular test as it's necessary within Postgres (should work across all other supported DWs)

Update type - breaking / non-breaking

[ ] Minor bug fix
[ ] Documentation improvements
[ ] Quality of Life improvements
[x] New features (non-breaking change)
[ ] New features (breaking change)
[ ] Other (non-breaking change)
[ ] Other (breaking change)

What does this solve?

Support for a new platform, Postgres
Outstanding questions

What databases have you tested with?

[ ] Snowflake
[ ] Google BigQuery
[ ] Databricks
[ ] Spark
[x] Postgres
[ ] N/A

Passing postgres integration tests - https://github.com/dpguthrie/dbt_artifacts/actions/runs/5538393942

Hi @dpguthrie . Thanks for taking the time to add this functionality in.
One of the team will spend some time reviewing it and get back to you.

Would love to see this in the next release!


Added the Lycée Anna De Noailles

The domain name for the students email is : lyceenoaillesevian.org
The official URL is : anna-de-noailles.ent.auvergnerhonealpes.fr
Since this school is in France, nearly every lycée (equivalent to high school in the USA) has a course called "NSI" (cours de Numérique Sciences Informatiques), translated it means "IT and digital course". Here is a link to prove that we offer this lesson: https://anna-de-noailles.ent.auvergnerhonealpes.fr/presentation-du-lycee/les-formations/ . This IT course can last for 1 or 2 years (depending if you drop it or not after the first year).
Here is a screenshot of the link but translated:

Here is a screenshot to prove that my email is affliated to a legitimate organization (my school):

And that's it I think you have everything!

@SkyrixFr Thank you for your patience while waiting for a review and processing your request. Pull request merged.


websocket::send() default callback

In my opinion there are plenty of cases where one might want to send something through a websocket but not necessarily care about the callback. Currently, this requires the user to specify an empty callback handler.
@0x00002a what do you think about providing a default callback handler (an empty one) so a user might just do this:
conn->send(malloy::buffer("foo"));

instead of:
conn->send(malloy::buffer("foo"), [](auto, auto){});

Agreed, that would be better. There does need to be a way to report errors though, I don't like the idea of letting the user easily (potentially accidentially even) ignore them, which I guess is more #40.

I agree. Lets close this down as the future of this will mainly be dictated by the outcome of #40.


Guidance agent

@iGavroche please take a look at the updated README - please note that this version does not support CPU usage. We need the llamacpp bindings version that @Karajan421  is working on for that.

We could also add feature flags but might be a lot of effort to revert the changes now behind a flag.

Marvelous!

We could also add feature flags but might be a lot of effort to revert the changes now behind a flag.

I'll leave the decision up to you, @paolorechia, as long as it doesn't get stale in this branch 😊

Let’s merge it, then, even if it’s not ideal yet - we can improve it over time. I’ll just wait for @Karajan421 ’s opinion in Discord


The behaviour of sendSequence changed

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: 1.71
OS Version: Linux Mint 20

Earlier the following keybinding used to work.
{
    "key": "ctrl+q c",
    "command": "workbench.action.terminal.sendSequence",
    "when": "editorTextFocus",
    "args": {
        "text": "echo ${fileDirname}/${selectedText} | xsel -i --clipboard \u000D"
    }
}

Supposed a file is located in /path/to/file/filename.txt. If i selected the text abcd.png then press ctrl+q c then /path/to/file/abcd.png would be copied to clipboard.
Now this shortcut stopped working.
If I use
{
    "key": "ctrl+q c",
    "command": "workbench.action.terminal.sendSequence",
    "when": "editorTextFocus",
    "args": {
        "text": "echo sample123 | xsel -i --clipboard \u000D"
    }
}

Even it does not copy sample123 to clipboard.

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.71.2. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

The issue is also in 1.71.2

@blueray453 do you mean that variables in the string used to work but don't anymore? Also the simple example echo sample123 | xsel -i --clipboard \u000D should send that text to the shell, it wouldn't copy it until it's run by the shell, what runs instead?

what runs instead?

I am not sure. If the command ran, sample123 would be in the clipboard. So I can just use ctrl+v to paste it. But nothing is placed in the clipboard when I invoke the above shortcut.
What I can say is, the keybinding used to work (the day before i posted the issue). Now it is not working. I used Timeshift to go to previous day's state, still it did not work.
Please let me know what info you need and how can i get those info (the debug steps).

This works fine for me in the latest, cannot reproduce 🤷

Removing from milestone as it wasn't reproducible.


Markup for {e }

is a span with style='font-style: italic' attribute. It should be a em tag.

Looking at the context of https://github.com/ocaml-doc/doc-ock-html/commit/ffcd3907f050b1232e255234a8ed478c4e28d7e4 it seems there are quite a few remaining style attributes. Since we no longer live in the 90s we should get rid of them:

Bold can be translated to strong or b
Italic can be translated to  i
Right, Center, Left should be translated to divs with a class of left, center or right.
Custom should be a span (or div I don't know when this happens) with a class of the given string.


[Snyk] Upgrade react-native from 0.70.5 to 0.71.6

Snyk has created this PR to upgrade react-native from 0.70.5 to 0.71.6.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 18 versions ahead of your current version.
The recommended version was released 23 days ago, on 2023-04-03.

Release notes

Package name: react-native

0.71.6 - 2023-04-03Fixed
iOS specific

Fix React Codegen podspec to build on Xcode 14.3 (0010c3807d by @ cipolleschi)

You can participate in the conversation on the status of this release in this discussion.

To help you upgrade to this version, you can use the upgrade helper ⚛

You can find the whole changelog history in the changelog.md file.

0.71.5 - 2023-03-29Changed

Bump CLI to 10.2.2 and Metro to 0.73.9 (4c3bc24893 by @ kelset), contains:

CLI fix: correctly list ios devices and simulators (relevant PR)
Metro fix: fix watching contents of new directories in NodeWatcher (ab86982 by @ robhogan)

Android specific

Bump react-native-gradle-plugin to 0.71.17 (bf490d379f by @ kelset), contains:

Fix patch for codegen for 0.71 (ec3681143e by @ kelset)

iOS specific

Remove ruby-version from 0.71 (1d22e29146 by @ cipolleschi)

Fixed
Android specific

Fix race condition in ReadableNativeMap (9aac13d by @ rshest)

iOS specific

Give precedence to textContentType property for backwards compat as mentioned in #36229 (comment) (c0abff11b6 by @ lunaleaps)
Blob data is no longer prematurely deallocated when using blob.slice (36cc71ab36 by @ awinograd)

You can participate in the conversation on the status of this release in this discussion.

To help you upgrade to this version, you can use the upgrade helper ⚛

You can find the whole changelog history in the changelog.md file.

0.71.4 - 2023-03-08

0.71.3 - 2023-02-14

0.71.2 - 2023-02-01

0.71.1 - 2023-01-19

0.71.0 - 2023-01-12

0.71.0-rc.6 - 2023-01-09

0.71.0-rc.5 - 2022-12-19

0.71.0-rc.4 - 2022-12-14

0.71.0-rc.3 - 2022-11-30

0.71.0-rc.2 - 2022-11-24

0.71.0-rc.1 - 2022-11-23

0.71.0-rc.0 - 2022-11-04

0.70.9 - 2023-04-19This release is primarily aimed at addressing issues with ANRs using multiline TextInput on Samsung devices (#35936, #35590).
Changed

Update Hermes to hermes-2023-04-13-RNv0.70.8-c9b539bf3d7bfa4143ff1a5751886c7b2dd728a2 (7b1441730b), contains:

Remove register stack size override in hermes.cpp (03f2df)
Increase default max stack size (1b759f4)

Fixed
Android specific

Resolved bug with Text components in new arch losing text alignment state. (31a8e92cad by @ javache)
Mimimize EditText Spans 9/9: Remove addSpansForMeasurement() (92b8981499 by @ NickGerleman)
Minimize EditText Spans 8/N: CustomStyleSpan (b384bb613b by @ NickGerleman)
Minimize EditText Spans 6/N: letterSpacing (5791cf1f7b by @ NickGerleman)
Minimize Spans 5/N: Strikethrough and Underline (0869ea29db by @ NickGerleman)
Minimize Spans 4/N: ReactForegroundColorSpan (8c9c8ba5ad by @ NickGerleman)
Minimize Spans 3/N: ReactBackgroundColorSpan (cc0ba57ea4 by @ NickGerleman)
Minimize Spans 1/N: Fix precedence (1743dd7ab4 by @ NickGerleman)
Fix measurement of uncontrolled TextInput after edit (8a0fe30591 by @ NickGerleman)

iOS specific

Address Hermes performance regression (1df92c6 by @ kelset)

You can participate in the conversation on the status of this release in this discussion.

To help you upgrade to this version, you can use the upgrade helper ⚛

You can find the whole changelog history in the changelog.md file.

0.70.8 - 2023-04-04Changed
iOS specific

Relax Ruby requirements (e3a5fbe72f by @ cipolleschi)

Fixed
iOS specific

Fix React Codegen podspec to build on Xcode 14.3 (34f3794f18 by @ cipolleschi)
Blob data is no longer prematurely deallocated when using blob.slice (36cc71ab36 by @ awinograd)

You can participate in the conversation on the status of this release in this discussion.

To help you upgrade to this version, you can use the upgrade helper ⚛

You can find the whole changelog history in the changelog.md file.

0.70.7 - 2023-01-31

0.70.6 - 2022-11-15

0.70.5 - 2022-11-06

    from react-native GitHub release notes

Commit messages

Package name: react-native

030ab8e [0.71.6] Bump version numbers
509f8f2 [LOCAL] fix definition of local mock for ruby test
0010c38 fix(xcode): backport Xcode 14.3 fix to 71 (#36769)
5162e0b Use packaged react-native in test-e2e-local script (#36703)
e137040 [LOCAL] update rntester podlock post 0.71.5 release
1d282da [0.71.5] Bump version numbers
e492858 [LOCAL] update snapshot test to fix CI
98f346b [LOCAL] update post 0.71.4 release
bf490d3 [LOCAL] bump RNGP to 0.71.17
8c20bb7 bumped packages versions
4c3bc24 [LOCAL] bump CLI to 10.2.2 and Metro to 0.73.9
ec36811 [LOCAL] fix patch for codegen for 0.71
0d3c70b Fix race condition in ReadableNativeMap (#36201)
a413e3e Precedent textContentType when set
96dc2c1 Prevent native blob resource from being de-allocated prematurely (#31392)
7488e9a Add option to commit with generic message (#36421)
1d22e29 chore: Remove ruby-version from 0.71 (#36423)
e0d92c6 [0.71.4] Bump version numbers
be199f8 [LOCAL] enforce iPhone 14 simulator on run ios E2E script
291cc0a [LOCAL] Bump Hermes Version
0f8d942 fix: conditional to include rn-tester and react-native-gradle-plugin in settings.gradle.kts (#36188)
41a633d Do not use a mixture of plugins{} and buildscript{}
f4546bf [LOCAL] Bump hermes version
56f2aa4 [LOCAL] update podlock

Compare

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Codecov Report

Merging #95 (474d040) into main (bfcebef) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #95   +/-   ##
=======================================
  Coverage   91.95%   91.95%
=======================================
  Files          10       10
  Lines         373      373
  Branches       75       75
=======================================
  Hits          343      343
  Misses         25       25
  Partials        5        5

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


Rich Text: Text disappear after copy/paste text or adding image

Category
[ ] Enhancement
[x] Bug
[ ] Question
Version
Please specify what version of the library you are using: [ 2.6.0 ]
Expected / Desired Behavior / Question
When using a rich text control, copy/paste text and image doesn't work.
Observed Behavior
Console error occurred when copy/paste any content or add a image.
Steps to Reproduce
1: Add Rich text webpart
2: Copy paste any text and click outside of your webpart.
3: Text will disappear with below error in console.

Debug

4: Its working fine if I'm manually writing any text instead of copy/paste.

Thank you @pk091990 for reporting the issue!
It has been fixed and the fix will be included in the next release.

@AJIXuMuK  Approx when we can expect the next release?  since this is kind of blocking us from choosing the editor.

@AnandhaprabhuSp - I would expect new release in the next 2 weeks.
But you can install beta version from npm right away if that's a blocker. V2 beta already contains the needed changes.

Hi I installed  "@pnp/pnpjs": "^2.5.0-beta2" where as previosly I had pnp version 2.5.0 but still copy and paste does not work.


[azure-cli-ml] az extension add --name azure-cli-ml  --version does not find most older extension versions

If the issue is to do with Azure CLI 2.0 in-particular, create an issue here at Azure/azure-cli

Extension name (the extension in question)
azure-cli-ml
Description of issue (in as much detail as possible)
Describe the bug
It seems like az extension add --name azure-cli-ml --version does not find most older extension versions.
To Reproduce
az version
{
  "azure-cli": "2.19.1",
  "azure-cli-core": "2.19.1",
  "azure-cli-telemetry": "1.0.6",
  "extensions": {}
}

az extension add --name azure-cli-ml --version 1.21.0 --debug
cli.knack.cli: Command arguments: ['extension', 'add', '--name', 'azure-cli-ml', '--version', '1.21.0', '--debug']
cli.knack.cli: __init__ debug log:
Enable color in terminal.
cli.knack.cli: Event: Cli.PreExecute []
cli.knack.cli: Event: CommandParser.OnGlobalArgumentsCreate [<function CLILogging.on_global_arguments at 0x7f42923e2840>, <function OutputProducer.on_global_arguments at 0x7f4291f35620>, <function CLIQuery.on_global_arguments at 0x7f4291cca8c8>]
cli.knack.cli: Event: CommandInvoker.OnPreCommandTableCreate []
cli.azure.cli.core: Modules found from index for 'extension': ['azure.cli.command_modules.extension']
cli.azure.cli.core: Loading command modules:
cli.azure.cli.core: Name                  Load Time    Groups  Commands
cli.azure.cli.core: extension                 0.003         1         7
cli.azure.cli.core: Total (1)                 0.003         1         7
cli.azure.cli.core: Loaded 1 groups, 7 commands.
cli.azure.cli.core: Found a match in the command table.
cli.azure.cli.core: Raw command  : extension add
cli.azure.cli.core: Command table: extension add
cli.knack.cli: Event: CommandInvoker.OnPreCommandTableTruncate [<function AzCliLogging.init_command_file_logging at 0x7f42910d9598>]
cli.knack.cli: Event: CommandInvoker.OnPreArgumentLoad [<function register_global_subscription_argument.<locals>.add_subscription_parameter at 0x7f4291109620>, <function register_global_query_examples_argument.<locals>.register_query_examples at 0x7f4291068d08>]
cli.knack.cli: Event: CommandInvoker.OnPostArgumentLoad []
cli.knack.cli: Event: CommandInvoker.OnPostCommandTableCreate [<function register_ids_argument.<locals>.add_ids_arguments at 0x7f4291068d90>, <function register_cache_arguments.<locals>.add_cache_arguments at 0x7f4291068ea0>]
cli.knack.cli: Event: CommandInvoker.OnCommandTableLoaded []
cli.knack.cli: Event: CommandInvoker.OnPreParseArgs []
cli.knack.cli: Event: CommandInvoker.OnPostParseArgs [<function OutputProducer.handle_output_argument at 0x7f4291f356a8>, <function CLIQuery.handle_query_parameter at 0x7f4291cca950>, <function register_global_query_examples_argument.<locals>.handle_example_parameter at 0x7f4291068c80>, <function register_ids_argument.<locals>.parse_ids_arguments at 0x7f4291068e18>]
urllib3.connectionpool: Starting new HTTPS connection (1): aka.ms:443
urllib3.connectionpool: https://aka.ms:443 "GET /azure-cli-extension-index-v1 HTTP/1.1" 301 0
urllib3.connectionpool: Starting new HTTPS connection (1): azcliextensionsync.blob.core.windows.net:443
urllib3.connectionpool: https://azcliextensionsync.blob.core.windows.net:443 "GET /index1/index.json HTTP/1.1" 200 763607
cli.azure.cli.core.extension._resolve: Candidates ['azure_cli_ml-1.23.0-py3-none-any.whl', 'azure_cli_ml-1.5.0-py2.py3-none-any.whl']
cli.azure.cli.core.extension._resolve: Candidates ['azure_cli_ml-1.23.0-py3-none-any.whl', 'azure_cli_ml-1.5.0-py2.py3-none-any.whl']
cli.azure.cli.core.extension._resolve: Candidates ['azure_cli_ml-1.23.0-py3-none-any.whl', 'azure_cli_ml-1.5.0-py2.py3-none-any.whl']
cli.azure.cli.core.extension.operations: Extension with version 1.21.0 not found
cli.azure.cli.core.util: azure.cli.core.util.handle_exception is called with an exception:
cli.azure.cli.core.util: Traceback (most recent call last):
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/extension/_resolve.py", line 82, in resolve_from_index
    chosen = [c for c in candidates_sorted if c['metadata']['version'] == target_version][0]
IndexError: list index out of range

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/extension/operations.py", line 307, in add_extension
    source, ext_sha256 = resolve_from_index(extension_name, index_url=index_url, target_version=version)
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/extension/_resolve.py", line 84, in resolve_from_index
    raise NoExtensionCandidatesError('Extension with version {} not found'.format(target_version))
azure.cli.core.extension._resolve.NoExtensionCandidatesError: Extension with version 1.21.0 not found

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/opt/az/lib/python3.6/site-packages/knack/cli.py", line 233, in invoke
    cmd_result = self.invocation.execute(args)
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 664, in execute
    raise ex
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 727, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 720, in _run_job
    six.reraise(*sys.exc_info())
  File "/opt/az/lib/python3.6/site-packages/six.py", line 703, in reraise
    raise value
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 698, in _run_job
    result = cmd_copy(params)
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 331, in __call__
    return self.handler(*args, **kwargs)
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/__init__.py", line 816, in default_command_handler
    return op(**command_args)
  File "/opt/az/lib/python3.6/site-packages/azure/cli/command_modules/extension/custom.py", line 18, in add_extension_cmd
    version=version, upgrade=upgrade)
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/extension/operations.py", line 315, in add_extension
    raise CLIError(err)
knack.util.CLIError: No matching extensions for 'azure-cli-ml (1.21.0)'. Use --debug for more information.

cli.azure.cli.core.azclierror: No matching extensions for 'azure-cli-ml (1.21.0)'. Use --debug for more information.
cli.azure.cli.core.azclierror: No matching extensions for 'azure-cli-ml (1.21.0)'. Use --debug for more information.
cli.knack.cli: Event: Cli.PostExecute [<function AzCliLogging.deinit_cmd_metadata_logging at 0x7f42910d97b8>]

Installing the latest extension using the --version value works
az extension add --name azure-cli-ml --version 1.23.0
az version
{
  "azure-cli": "2.19.1",
  "azure-cli-core": "2.19.1",
  "azure-cli-telemetry": "1.0.6",
  "extensions": {
    "azure-cli-ml": "1.23.0"
  }
}

Expected behavior
The old versions are installable using the --version parameter.
Environment summary
cat /etc/*release | grep PRETTY
PRETTY_NAME="Ubuntu 18.04.5 LTS"

python --version
Python 3.7.10

/opt/az/bin/python3 --version
Python 3.6.10

Additional context
Most old extension versions are missing as candidates in the debug output
az extension add --name azure-cli-ml --version 1.21.0
...
urllib3.connectionpool: https://azcliextensionsync.blob.core.windows.net:443 "GET /index1/index.json HTTP/1.1" 200 763607
cli.azure.cli.core.extension._resolve: Candidates ['azure_cli_ml-1.23.0-py3-none-any.whl', 'azure_cli_ml-1.5.0-py2.py3-none-any.whl']
...

The installation of the other available version az extension add --name azure-cli-ml --version 1.5.0 fails due to a different issue https://github.com/Azure/azure-cli/issues/16856

azure-cli-ml

@marcindulak why do you need version 1.21.0? azure-cli-ml versions are manully updated and they only keep the latest version and an old version that is compatible with older versions of azure-cli. @j-so for awareness.

The old extension versions are still available at https://azurecliext.blob.core.windows.net/release
Version pinning, of azure-cli as well, is used for ci as an attempt to minimize potential breaking changes.
For pinning, an explicit installation from an url with az extension add --source is available https://github.com/Azure/azure-cli-extensions/issues/2992#issuecomment-777849355, but since the az extension add --version option exists it may support all versions as well.


MWPW-160015: M@S: support for mnemonic multifield

during card hydration.
Resolves:
MWPW-160015
MWPW-160079
Test URLs:

Before: https://main--milo--adobecom.hlx.page/drafts/ilyas/merch-card?martech=off
After: https://mwpw-160015-final--milo--yesil.hlx.page/drafts/ilyas/merch-card?martech=off

MAS Studio link:
https://main--mas--adobecom.hlx.live/studio.html?milolibs=mwpw-160015-final--milo--yesil

@yesil  The authored target link is not getting opened up from studio - 3rd AC - Can you plz check ?
https://main--mas--adobecom.hlx.live/studio.html?milolibs=mwpw-160015-final--milo--yesil
Marked comment in https://jira.corp.adobe.com/browse/MWPW-160015
Meanwhile marking as Ready for stage, since covered regressions on Milo.

@Roycethan yes it is by design. It is to prevent Studio users from leaving after unintended link clicks.


[Read me] Adding hero-image (including new icon) and module screenshots

Summary of the Pull Request
Visually improved the read me file, so it looks more appealing to (new) users:

Added hero image at the top with the new PowerToys logo
Added screenshots for all modules
Added Keyboard Manager and PowerToys Run (@crutkas proper descriptions are still needed there) for version 0.18.
Added 3 'shortcuts' after the project description (Download / Release notes / Privacy statement

References
#2766
PR Checklist

[X] Applies to #2766
[X] CLA signed. If not, go over here and sign the CLA

if you move this to a non-master branch, i can directly help here too versus your fork

Can we change the text to be only left from the image? The list dot under preview pane image doesn't looks good.

@crutkas @riverar Thanks for your comments. I have adopted most of them. For any of the actual content / descriptions I'd like to as @crutkas to make the changes as he sees fit.
Apologies for setting this up as a fork. Forgot I got write access now so created a new branch with the changes mentioned by you. I have created a new PR here: #2946


Test neo4j notebooks on CI

This uses docker-compose to add a Neo4j instance running on CI. This is created by building an image from the official neo4j images that includes the "APOC" plugin that we need. Building and running this image is easy and fast, only a few seconds if the neo4j image is already available on the agent; as such, for simplicity, this unconditionally runs the image neo4j for every step that uses one of the 3.x runners, even though most do not use neo4j.
The notebooks are tested as a single step, that runs them in series, because the directed GraphSAGE notebooks rely on having Cora already available in the database, and thus the load-cora-into-neo4j.ipynb notebook needs to run first.
This makes some minor edits to the notebooks so that they can work on CI, with the major one restructuring how the connection to neo4j works, by using Neo4j/py2neo's defaults for everything except the host, which is read from the STELLARGRAPH_NEO4J_HOST environment variable if it exists. This is used on CI to tell to the code to connect to the neo4j container which is available at neo4j, not localhost since it's a separate container. An alternative would be to specify network_mode: "service:runner-3_6" in neo4j, so that it's available under the runner container's localhost and thus need no custom code in the notebook at all. Thoughts?
This tests on Neo4j 3.5, which is not the latest release, but (a) it's probably good to support a bit of a range, and (b) the notebooks fail on Neo4j 4.0 (https://github.com/stellargraph/stellargraph/issues/1055). Once that issue is fixed, I believe we should be able to extend this to test against multiple versions of Neo4j using build args.
See: #849

The readme for the demos could probably be updated too, and direct users to use pip install stellargraph[neo4j] but that could be done as a separate task to redo the readme in general

That's a good idea, but I think we shouldn't do it quite yet: pip install stellargraph[neo4j] won't currently work, since the 0.10.0 release didn't have the neo4j extra.


clientv3 compile fail

It looks like grps.Balancer modify interface.
# github.com/coreos/etcd/clientv3
../../coreos/etcd/clientv3/client.go:290: cannot use client.balancer (type *simpleBalancer) as type grpc.Balancer in argument to grpc.WithBalancer:
	*simpleBalancer does not implement grpc.Balancer (wrong type for Start method)
		have Start(string) error
		want Start(string, grpc.BalancerConfig) error
github.com/stretchr/testify/mock

This was introduced via https://github.com/grpc/grpc-go/pull/899.
/cc @heyitsanthony

@gyuho gRPC does not have any release after 1.0. We should not update the dependency at the moment.

I had to do this.
etcd> git diff                                                                     nmuppalaneni@nmuppalaneni-vm etcd> git diff
diff --git a/clientv3/balancer.go b/clientv3/balancer.go                                                        diff --git a/clientv3/balancer.go b/clientv3/balancer.go
index 011f6cf..16df457 100644                                                                                index 011f6cf..16df457 100644
--- a/clientv3/balancer.go                                                                                --- a/clientv3/balancer.go
+++ b/clientv3/balancer.go                                                                                +++ b/clientv3/balancer.go
@@ -72,7 +72,9 @@ func newSimpleBalancer(eps []string) *simpleBalancer {                                        @@ -72,7 +72,9 @@ func newSimpleBalancer(eps []string) *simpleBalancer {
return sb                                                                                return sb
}                                                                                }
-func (b *simpleBalancer) Start(target string) error { return nil }                                             -func (b *simpleBalancer) Start(target string) error { return nil }
+//func (b *simpleBalancer) Start(target string) error { return nil }                                           +//func (b *simpleBalancer) Start(target string) error { return nil }

                                                                                +

+func (b *simpleBalancer) Start(target string, cfg grpc.BalancerConfig) error { return nil }                    +func (b *simpleBalancer) Start(target string, cfg grpc.BalancerConfig) error { return nil }
func (b *simpleBalancer) ConnectNotify() <-chan struct{} {                                                      func (b *simpleBalancer) ConnectNotify() <-chan struct{} {
b.mu.Lock()

cc @SuhasAnand

Any moves? This code on godoc. You are released it. Four days! Are you absolutely do not respect yourself?

@akaspin You totally do not understand the problem and do not read my previous reply. You should respect others in the community.
As I mentioned, gRPC broke their API and has not released a version with this API. Users should vendor the correct version of gRPC. There is nothing we can do much now.

@xiang90 Oh no! I totally understand the problem. What I see? I see totally dead code. Four day. Four working days. I see go get github.com/coreos/etcd/clientv3 in docs. What I don't see? I don't see any mentions or workarounds. No reaction. I see completely disregard. "This is introduced ... Google not released new version ...". This is a joke? There are two versions. Last - in August. Who here does not respect the community?

@akaspin
Your last response confirmed that you do not understand how go vendor/dependency management works at the moment. Your project needs to vendor all dependencies. Or any indirect dependencies breakage can break the library you depend on (like gRPC broke their API and your etcd client broke). You need to vendor your dependencies. This is not workaround, this is something you have to do.
Again, we are not going to update etcd/clientv3 until gRPC release a version with the new API. The current API change is in the unreleased version. And you are getting that unreleased version for your project, which you probably should not do as I mentioned.

@xiang90 Wow! I'm right enlightened! "You need to vendor your dependencies ..." Wow! Really! (however, that vendor breaks the types and it can not be excluded if the test directory. and much better to override GOPATH. never mind) But ... It ... Uh ... go get github.com/coreos/etcd/clientv3. Hour of my life is trashed.

@akaspin
In the go get doc:
Get downloads the packages named by the import paths, along with their
dependencies. It then installs the named packages,

So it downloads both package itself and the missing dependencies. This is not ideal since it is not versioning aware. But this is not an etcd specific issue. Go developers should be aware of that.
If you believe we should improve the doc, can you please open another issue? Probably we can make clear that go get also download dependencies, which might not be what you want. Or you can help to improve it directly by opening a pull request? Thanks.

We are stuck right now. Would you recommend any work around? I am using godep too. But if the got get fails then how I can use godep to save the vendor package anyway. I already roll back to version 3.0.8 without any luck.

@Son-Lam go get doesn't handle the dependency as far as I know.
You should try go get -d that skips compiling. And vendor your project with etcd's gRPC git SHA.

This problem seems to cause a lot of trouble to etcd newer
Need to pay attention to this bad influence

Hi,
To be clear, it is not coreos/etcd' problem. The problem is the gRPC guys changed interfaces. It affecting new users. I got it work by vendoring my dependencies to the exact snapshot of grpc (SHA 231b4cfea0e79843053a33f5fe90bd4d84b23cd3).
Maybe, coreos/etcd should put up some work-around instruction and/or update to match up with grpc 1.1.0 interface.
Thanks,

many people are stuck right now!
i think u should gix this problem!

If you are using the gvt vendoring tool just ensure your manifest has the following:
{
    "importpath": "google.golang.org/grpc",
    "repository": "https://github.com/grpc/grpc-go",
    "vcs": "git",
    "revision": "231b4cfea0e79843053a33f5fe90bd4d84b23cd3",
    "branch": "master",
    "notests": true
},

The importat part being the revision hash obviously.

Sorry for the delay.  Should work now.
Again we recommend:

etcd uses cmd/vendor directory to store external dependencies, which are to be compiled into etcd release binaries. client can be imported without vendoring. For full compatibility, it is recommended to vendor builds using etcd's vendored packages, using tools like godep, as in vendor directories. For more detail, please read Go vendor design.

Thanks,

Could you please backport the modification (https://github.com/coreos/etcd/pull/6617/files#diff-3ac56ebeb8ecf6f62729390be572416eR75) to 3.0.x branch? Otherwise build of the v3.0.12 version does not work. I would rather vendor v3.0.x in our code base than 3.1.x-alpha with the fix.
Thank you.

@zaa Sorry, unlikely.
Our policy is to only backport bug patches. Is there any reason that your project does not use the same gRPC as release-3.0 branch?


Stop rewriting 404s

What steps will reproduce the problem?
1. Go to http://www.modpagespeed.com/mod_pagespeed_beacon
2. Look at the source and notice that it was rewritten even though it's a 404
page

My first thought was that a 404 page should not be rewritten. But maybe it
doesn't hurt. In this case we are adding instrumentation to the 404 page, which
seems like a bad idea because it seems like a 404 page shouldn't have resources.

What does everyone else think?

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 30 Dec 2010 at 9:28

It may make sense to revisit this.  404 pages are often relatively rich, with lots to optimize.  Neither the introduction of PLT beacons nor the DOS attack reasoning makes much sense to me.  Is there any other reason to keep this restriction?

@sligocki @jmaessen

My 4 year old memory is a bit fuzzy on this :) but the reasoning I remember is: A) We do not expect much content on 404s, B) We don't expect high percentage of traffic to be for 404, C) We really don't want to break 404 pages (ex: somehow obfuscate the error message).
But if we're not worried about (C), then maybe there's no harm. I certainly don't feel strongly and think it would be fine if we changed this back.

On Mon, Jun 15, 2015 at 5:39 PM, Shawn Ligocki<EMAIL_ADDRESS>wrote:

My 4 year old memory is a bit fuzzy on this :) but the reasoning I
remember is: A) We do not expect much content on 404s, B) We don't expect
high percentage of traffic to be for 404, C) We really don't want to break
404 pages (ex: somehow obfuscate the error message).
But if we're not worried about (C), then maybe there's no harm. I
certainly don't feel strongly and think it would be fine if we changed this
back.
I seem to recall in particular that folks serving huge 404 pages and sites
with lots of missing links (or just a missing favicon.ico) ran into
problems whenever (say) googlebot came a-visiting.

Arguably we should actually be replacing some 404 pages with
content-length: 0 404s, as this has produced high returns for Chrome mobile
bandwidth reduction.

—
Reply to this email directly or view it on GitHub
https://github.com/pagespeed/mod_pagespeed/issues/170#issuecomment-112218952
.

Arguably we should actually be replacing some 404 pages with
content-length: 0 404s, as this has produced high returns for Chrome mobile
bandwidth reduction.

I don't think this is something we could do by default.  Many sites use their 404 pages to offer people suggestions about how to find what they're looking for.
I do think 0-length 404s would be good for favicon.ico, apple-touch-icon.png, and apple-touch-icon-precomposed.png.  On mobile browsers that fetch these by default (not Chrome anymore) they were accounting for 3%-4% of bytes downloaded: https://bugs.webkit.org/show_bug.cgi?id=104585

Note that there are practical difficulties rewriting 404s on Apache 2.4
with our current filter and the way it's configured.
I don't think it's unsolvable, but config backwards compat may be tricky.
IIRC it implements AddOutputFilterByType  in mod_filter (2.2 had a
different impl in core.c), which eventually dispatches to:
static apr_status_t filter_harness(ap_filter_t *f, apr_bucket_brigade *bb)
{
apr_status_t ret;
#ifndef NO_PROTOCOL
const char *cachecontrol;
#endif
harness_ctx *ctx = f->ctx;
ap_filter_rec_t *filter = f->frec;
if (f->r->status != 200
    && !apr_table_get(f->r->subprocess_env, "filter-errordocs")) {
    ap_remove_output_filter(f);
    return ap_pass_brigade(f->next, bb);
}

... I guess the filter-errordocs may be an option, too.
On Tue, Jun 16, 2015 at 7:06 AM, Jeff Kaufman<EMAIL_ADDRESS>wrote:

Arguably we should actually be replacing some 404 pages with
content-length: 0 404s, as this has produced high returns for Chrome mobile
bandwidth reduction.
I don't think this is something we could do by default. Many sites use
their 404 pages to offer people suggestions about how to find what they're
looking for.
I do think 0-length 404s would be good for favicon.ico,
apple-touch-icon.png, and apple-touch-icon-precomposed.png. On mobile
browsers that fetch these by default (not Chrome anymore) they were
accounting for 3%-4% of bytes downloaded:
https://bugs.webkit.org/show_bug.cgi?id=104585
—
Reply to this email directly or view it on GitHub
https://github.com/pagespeed/mod_pagespeed/issues/170#issuecomment-112386359
.


fix(deps): update terraform ibm to latest for the da solution

This PR contains the following updates:

Package
Type
Update
Change

ibm (source)
required_provider
patch
1.64.0 -> 1.64.1

Release Notes

IBM-Cloud/terraform-provider-ibm (ibm)
v1.64.1
Compare Source
BugFixes

The terraform-provider-ibm_v1.64.0 plugin crashed! (5258)
ibm_pi_instance: pi_network attribute :: Provider incorrectly seems to detect a change, applies it - next apply incorrectly detects the same change (5201)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

/run pipeline

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

GitHub release
v2.0.11

Your semantic-release bot :package::rocket:


Statistics component in Daphne

In GitLab by @corepointer on Mar 18, 2022, 11:32
Similar to the stats output of SystemDS we would benefit of stats collection in Daphne as well.
The challenge would be an approach that has low impact on actual runtime.

FYI: I'm working on this one, PR coming soon.
Is able to display the top K kernel functions by time spent, and even the top K source code lines from the DSL script. User will be able to choose with a flag. I think as a default setting, simply the source loc makes more sense, if someone wants to know which actual kernel spent how much time than we'll have the flag for that.

@philipportner I think this issue has been addressed by d4cf91031dcebe2467ca9b07c9e651c77f3917ff. Can we close this issue? Additional improvements could be subject to more specific follow-up issues.

Yes, thanks for catching that.


          ![Screenshot_20231004-020641](https://github.com/Vita3K/compatibility/assets/146876496/aa44cf6b-4746-46ee-8cad-e927c388975b)

Now playable without noticeable glitches
Originally posted by @jegesim in https://github.com/Vita3K/compatibility/issues/950#issuecomment-1745485340

issue is invalid

nice


docker build is failing

Describe the bug
This worked for me yesterday, but now I'm getting:
Step 4/19 : RUN yum update -y &&     yum clean all -y
 ---> Running in dd41b1ce3e2c
Updating Subscription Management repositories.
Unable to read consumer identity
This system is not registered to Red Hat Subscription Management. You can use subscription-manager to register.
Red Hat Universal Base Image 8 (RPMs) - BaseOS  1.3 MB/s | 762 kB     00:00
Red Hat Universal Base Image 8 (RPMs) - AppStre 3.9 MB/s | 3.7 MB     00:00
Red Hat Universal Base Image 8 (RPMs) - CodeRea  31 kB/s | 9.1 kB     00:00
(try to add '--allowerasing' to command line to replace conflicting packages or '--skip-broken' to skip uninstallable packages or '--nobest' to use not only best candidate packages)
Error:
 Problem 1: cannot install the best update candidate for package python3-rpm-4.14.2-26.el8_1.x86_64
  - nothing provides libzstd.so.1()(64bit) needed by python3-rpm-4.14.2-37.el8.x86_64
 Problem 2: cannot install the best update candidate for package rpm-4.14.2-26.el8_1.x86_64
...

To Reproduce
Steps to reproduce the behavior:

mvn clean install -f fhir-parent
docker build fhir-install

Expected behavior
The docker build succeeds
Additional context
It started very recently

I think Lee addressed in PR1000 (and Albert reviewed) moving to Review/QA


Human readable key stored

Hi,
When a key is stored (except one of the predefined for positions and size), the key is a sha-256 converted to String.
When you look in the registry (in Windows), you have this uggly look:

I see that you want to use a hash of the key given, but

you don't apply this rule to your keys, so why the other should do?
why not store the breadcrumb directly?

 – John F. Woods
which is also available for the results of the code.

Hi @DjayDev
Thanks for your comment 👍. Like I mentioned in #15, the reasoning behind this is explained in the JavaDoc of StorageHandler#hash(String).
I agree this is not a very pretty solution, but it's the best approach we found that would allow to save indefinitely long breadcrumbs to the preferences and thus circumvent the limits of the 80 character key length limitation as posed by the Java Preferences API... I respect Joshua Bloch (the author of the Java Preferences API) and I am pretty sure he had his reasons for imposing this limit, although to be fair he didn't document them anywhere as far as I could see.
If you have a suggestion for an implementation, which guarantees having keys with a length of less than 80 characters for indefinitely long breadcrumbs which remain human-readable, we would be glad hearing about it.
Thanks!

Not ideal, but the first idea that comes would be to have an hybrid solution: when the breadcrumb is less than 80char, we leave it clear and for breadcrumbs more than 80 chars, we would use the hash (and a pref to use always the hash like it is now so it won't break the current implementation).
Based on your experience, do you think the breadcrumbs is often higher than 80 chars?
If you're ok with such workaround, I could have a look. Let me know what you think.

I think this approach could work, I just don't see the benefit of having all of this additional code (potentially resulting in bugs and making it harder to maintain) to just have some of the keys in human-readable form. I don't think most people will have a look at the registry to change the values, or is there a certain use case that you have in mind, where it would be useful to have the keys stored in human-readable form?

The use case is to be used with the listener like I showed in the samples I submitted. I know I should be able to do it with the hash function (not tested but I guess) but it's more convenient to do it with readable preferences.
It's a way to read and/or change the preferences outside the dialog like it was asked in the question #8

I see what you mean @DjayDev. To have a listener for the changed settings I think it makes more sense to apply a listener to the property itself that you define in the appropriate spot where you need it in your application and to react to it there. If you just generally want an information for debugging to see the changed settings, I think the approach I proposed in #15 with logging would make sense.
As for #8, I think it makes more sense to fix this problem at its root than to go with a workaround like this. For example, we could save the preferences when they change externally with a listener in PreferencesFX itself, which would be a quite easy and clean solution in my opinion.
If there is any other use case you know of where this would be benefitial, feel free to let me know!

Since there was no response, I will close this issue. Feel free to open it up again or create another issue should you still run into issues.


updated robots.txt

to ignore any agent on root path

Thank you for the contribution @felipeschulz!


[FR] resizable selection area?

hi, I just wonder if there any possible to update the selection area, after initially pasting the quote to the markdown file
it's two separate problems actually

update the backlink
update the quoted text with newly selected

I've noticed that the link to the annotation contains the selection area, like below
#page=60&selection=100,8,102,65

simply envision this, implementing the resize may depend too much on obsidian internals, and may take too much effort.
since now you can manage backlinks, just updating the backlinks may be more approachable. if we could update the selection query params in links, we can somehow emulate such resize behavior

the second problem may not be so easy, and if we use embedded with margin, maybe it is not necessary to implement this. but I am somehow concerned about the performance if you embed too many PDF viewer instances in tabs, especially when you open dozens of tabs.
the way how logseq implements this feature may give an inspiration. it establishes an external identifier manager, in which the annotation and the quoted are both linked to that identifier, once the annotation changes, the corresponding quoted text will also get updated. but this will also need too much effort.

by the way, is there any other way I could sponsor you? Neither GitHub sponsor nor buymeacoffee support PayPal, while I don't have any bank card  which I can use directly on these websites

First of all, what makes you want to avoid just doing the copy-paste once again?
Also, could you elaborate on what the workflow in your mind looks like exactly?

As for the &selection=... syntax, I have already succeeded in deciphering it! So it will not be a big problem.

update the quoted text with newly selected

Auto-updating not only the link itself but also the surrounding texts will be very difficult given the variety of custom copy formats.
So as you suggest, the only feasible solution would be using embeds and updating the link text.
Updating link texts will not be difficult, but as you say, opening too many PDF embeds might degrade the performance.
Opening PDF files is such a heavy job.
Can you kindly provide a reference for the Logseq feature you mentioned?

Thank you very much for considering to support me. It really means a lot to me.
I will seek other options for how you can sponsor me!

First of all, what makes you want to avoid just doing the copy-paste once again?

perfectionism

Also, could you elaborate on what the workflow in your mind looks like exactly?

It depends on the difficulty of implementing this feature, the workflow is already good enough, it's just another quality-of-life kind of enhancement

Can you kindly provide a reference for the Logseq feature you mentioned?

I just realized a more approachable way to achieve this
we need to keep sync between the link and quoted text (by a unique identifier), let us separate this into two problems

save the identifier in the link
save the identifier in quoted text, while adaptable to different template formats

the first one is pretty easy, we can use query params like below
#page=60&selection=100,8,102,65&identifier=123123

the second one is the same as zotlit and flashcards-obsidian, we put an explicit identifier just around the quoted text
meanwhile, we may need an HTML5 comment to wrap quoted text
<!-- customizable quote start --> <-- this will be hidden in `live preview` or `reading` mode
any template format
...
<!--customizable quote end -->
^123123    <-- explicit identifier in markdown file

as we have backlinks, we already have a mapping relationship between the identifier and file path, we could directly search(regex?) the identifier in that file, then replace the corresponding region
an example regex pattern
{customizable quote start}
.* <-- any text between
{customizable quote end}
\^(\d*)\n <--match the identifer

@SSmJaE I've just updated funding methods and added ko-fi, which supports PayPal!
I appreciate your support and suggestions!

I see, so even after the text selection (&selection=...) changes, the identifier stays untouched, which allows us to locate the corresponding quote in the markdown file and update its content.
That's an interesting idea. But how should we handle the variety of copy template formats?
To make this happen, we will have to identify which template format was used for the quote.
To make matters even worse, the quote might have been added with some arbitrary comments after pasting. When updating the quote, we need to avoid overwriting such comments.
Also, we will have to take care of changes in copy template formats themselves.

we will have to take care of changes

It's really quite simple; we don't have to change the text with 100 percent certainty.
If it matches the pattern, we update it.
If it doesn't match, we notify the user that we didn't find the supposed pattern.

how should we handle the variety of copy template formats?

I found that the html5 comment tag will still be visible in live preview mode, while other html5 tag like div, will also be hidden in live preview mode, and it can be inserted into callout, like what the picture illustrates.
Then we can save data as tag's attribute, which could also be used by regex pattern to locate it.
No matter what template format the user is using, once they have corresponding tag, we can locate and update them.
by the way, happy lunar new year

hi, any ideas?🙃


[BUG]:

Is there an existing issue for this?

[X] I have searched the existing issues.

🐞 Describe the Bug
can i go live with this library, ive done my sandbox testing and everything is okay... ive got live api and credentials and the checkout url is still starting with sandbox.paypal . HOW DO I GO LIVE ??
😕 Current Behavior
WHEN I REPLACE THE SANDBOX CLIENT ID WITH THE LIVE ID IT DOESN'T PICK UP BUT STILL USES THE SANDBOX URL
🤔 Expected Behavior
I Expected switching to a live client id it should detect thats a live account and get rid of the sandbox url and instead use the production url
🔬 Minimal Reproduction
use same steps for sandbox account but this time use live credentials instead of testing credentilas

🌍 Environment
| Software         | Version(s) |
| ---------------- | ---------- |
| react-paypal-js  |    "^7.8.2"        |
| Browser          |   google chrome        |
| Operating System |     windows 10 pro 64bit       |

Relevant log output
No response
Code of Conduct

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

➕ Anything else?
No response

It finally picked up my key am good now


Add multiple files creation at same time

For creating videos of code.
If a person is working on a project and then there are so many files like app.js, Home.js and Component.js and while showing the codes in the video it will be good to create multiple files.
Simply adding a feature of creating a new file in the next tab and only one file can be active at a time.
I would like to work on it if you get the idea well.

Nice Idea, You can start working on this if you want.


Search not returning an expected result

Searching my feeds for bereal, this article isn't returned in the results:
https://lifehacker.com/what-is-the-new-social-media-app-bereal-and-do-you-r-1848780094
Searching for new social media app does return the article.
It comes from this feed:
https://lifehacker.com/tag/software/rss
And the full XML content of the feed item is here:
<item><title><![CDATA[What Is the New Social Media App ‘BeReal’ (and Do You Really Want to Be Real)?]]></title><link>https://lifehacker.com/what-is-the-new-social-media-app-bereal-and-do-you-r-1848780094</link><description><![CDATA[
There are always new social media platforms launching, but few have the staying power of Facebook, Instagram, Twitter, Snapchat, TikTok, Truth Social, WhatsApp, and Telegram. Each day seemingly promises us some new platform, most of which we ignore or maybe use for a brief time before ultimately ignoring; others have…

Read more...
]]></description><category>mobile software</category><category>app store</category><category>internet culture</category><category>fads and trends</category><category>google</category><category>mobile applications</category><category>computing</category><category>instagram</category><category>bereals</category><category>technology</category><category>software</category><category>selfie</category><category>pinterest</category><category>tiktok</category><category>snapchat</category><category>snapchat</category><category>video software</category><pubDate>Tue, 12 Apr 2022 16:00:00 GMT</pubDate><guid isPermaLink="false">1848780094</guid><dc:creator><![CDATA[Lindsey Ellefson]]></dc:creator></item>

I wonder if it's because of the quotes around the title; if so that should be fixed.

Agreed — it’s most likely the quotes

Looks like updating the search table from fts4 to fts5 might fix this.

Another option would be to strip codepoints in the punctuation class out before indexing.


You cannot call a method on a null-valued expression - Run-ExchangeSetupForTargetTenant - Line 732

This line throws a "You cannot call a method on a null-valued expression" error if there are no errors in the $Error variable or if $Error[0] doesn't contain an ErrorRecord, which is almost always the case. So the script almost always fails on this line.
$Error[0].Exception.ErrorRecord.FullyQualifiedErrorId.Contains("MaximumConcurrentMigrationLimitExceededException")

Thanks Trevor,
We'll get this fixed.
Roman
From: Trevor @.>
Sent: Monday, April 26, 2021 9:08 AM
To: microsoft/cross-tenant @.>
Cc: Subscribed @.***>
Subject: [microsoft/cross-tenant] You cannot call a method on a null-valued expression - Run-ExchangeSetupForTargetTenant - Line 732 (#53)
This line throws a "You cannot call a method on a null-valued expression" error if there are no errors in the $Error variable or if $Error[0] doesn't contain an ErrorRecord, which is almost always the case. So the script almost always fails on this line.
$Error[0].Exception.ErrorRecord.FullyQualifiedErrorId.Contains("MaximumConcurrentMigrationLimitExceededException")

You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2Fcross-tenant%2Fissues%2F53&data=04|01|Roman.Powell%40microsoft.com|4a1bc219ebfa47f8ca9208d908cd670b|72f988bf86f141af91ab2d7cd011db47|0|0|637550500558738634|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=7ur458zNx2PMxJT4DRuL4f9BI4HPiZ16iyCoyLCIGTM%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAPUHY6EI23UDUJXDI3JOUHDTKWFULANCNFSM43TGA2PA&data=04|01|Roman.Powell%40microsoft.com|4a1bc219ebfa47f8ca9208d908cd670b|72f988bf86f141af91ab2d7cd011db47|0|0|637550500558748594|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=bpGzTcZKKsTGKzwlGtCdCIeDX%2BiwhqNe%2FChklTkRxlM%3D&reserved=0.

Hello. I am experiencing the same issue reported by 1trevor. I'm assuming there is a work around until resolved. Can someone please elaborate on what I need to do to make the script work? Or does it work and the error is just "cosmetic"? Thanks in advance.

Ron,
This is a cosmetic error, you can verify this by running Get-OrganizationRelationship and Get-MigrationEndpoint and verifying both were created.
Thanks,
Roman
From: RonJaworski @.>
Sent: Wednesday, May 26, 2021 9:52 AM
To: microsoft/cross-tenant @.>
Cc: Roman Powell @.>; Comment @.>
Subject: Re: [microsoft/cross-tenant] You cannot call a method on a null-valued expression - Run-ExchangeSetupForTargetTenant - Line 732 (#53)
Hello. I am experiencing the same issue reported by 1trevor. I'm assuming there is a work around until resolved. Can someone please elaborate on what I need to do to make the script work? Or does it work and the error is just "cosmetic"? Thanks in advance.

You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2Fcross-tenant%2Fissues%2F53%23issuecomment-848942726&data=04|01|Roman.Powell%40microsoft.com|e190397afb6c46dc907a08d92066a619|72f988bf86f141af91ab2d7cd011db47|0|0|637576447528167841|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=dPFZmKf0ItmnhQvbil8sVgYw%2FF41ffrhhk1SS9zpknY%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAPUHY6EXEH7XSYLYTQUNXB3TPURMZANCNFSM43TGA2PA&data=04|01|Roman.Powell%40microsoft.com|e190397afb6c46dc907a08d92066a619|72f988bf86f141af91ab2d7cd011db47|0|0|637576447528177796|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=y6ZYk8hocnGcT0BhOmin%2FWtUg5E3CdR0jhRddFvdB6Q%3D&reserved=0.

Thanks for the reply, Roman. The way I handled it to have the script continue to run past that point was...

Commented out lines 732 through 744.
Then I had to change the verification function of the script so that it wouldnt overwrite my changes with the script from the repository. I commented out lines 820 through 829. I copied line 815 through 817 and pasted between 829 and 830. Then I customized the write-host on the first copied line to indicate to me that it was using my version of the script and not the standard from the repository.

This might not be the "correct" way to do it, but it worked. I hope this helps anyone else who might get stuck. I'm really not a powershell guru, I know barely enough to be extremely dangerous; so use this method at your own risk.


~/.blueprintignore not respected

~/.blueprintignore contents:
:file:/etc/hosts
!:source:/opt

Yet blueprint.sh still includes:
mkdir -p "/etc"
cat >"/etc/hosts" <<EOF
<IP_ADDRESS>       localhost

# The following lines are desirable for IPv6 capable hosts
::1     ip6-localhost ip6-loopback
fe00::0 ip6-localnet
ff00::0 ip6-mcastprefix
ff02::1 ip6-allnodes
ff02::2 ip6-allrouters
EOF

And doesn't include a second sources entry

I had the same issue. Moving it to /etc/blueprintignore as suggested worked.
I'm on Debian 6 and this is happening when I'm logged in as root directly (not sure if that matters or not but thought I'd mention it).


Minified source map is empty

From PR #241 that migrated build to webpack.

Not including source maps at all right now; closing