Duplicate dep with hash and without

Describe the bug
Actual
The @histoire/vendors dep files are loaded into the page twice, one time without hash and one time with an hash.

Expected
The files are loaded once.

Reproduction
https://github.com/Akryum/vite-issue-duplicate-dep
System Info
System:
    OS: Linux 5.18 Fedora Linux 36 (Workstation Edition)
    CPU: (24) x64 AMD Ryzen 9 3900XT 12-Core Processor
    Memory: 37.38 GB / 62.71 GB
    Container: Yes
    Shell: 5.1.16 - /bin/bash
  Binaries:
    Node: 16.16.0 - ~/.nvm/versions/node/v16.16.0/bin/node
    Yarn: 1.22.19 - ~/.nvm/versions/node/v16.16.0/bin/yarn
    npm: 8.11.0 - ~/.nvm/versions/node/v16.16.0/bin/npm
  Browsers:
    Chrome: 104.0.5112.79
    Firefox: 103.0.1
  npmPackages:
    vite: ^3.0.7 => 3.0.7

Used Package Manager
pnpm
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

This seems to be already fixed in at least 3.2.7.


send more than one paragraph to ChatGPT at a time

Let's send more than one paragraph to ChatGPT at a time, in order to speed up the process and increase the quality of translation.

@rultor release, tag is 0.0.36

@rultor release, tag is 0.0.36

@yegor256 OK, I will release it now. Please check the progress here

@rultor release, tag is 0.0.36

@yegor256 Done! FYI, the full log is here (took me 3min)


Add more test cases.

The following tests were added:

Issue tests.
Expression in aggregate functions.
Expression in group by functions.
Logical and or test.

Known potential issue fix:

Replace some count(1) with id_dt

/ok-to-test

/run-all-tests

/run-all-tests tikv=pr/2738

/run-all-tests

/run-all-tests

/run-all-tests

@birdstorm @ilovesoup PTAL

/run-all-tests


Added preparing DNS for wildcard to use sish

Added documentation for basic preparation to use sish

Thanks @markFieldman! Looks like there is a problem with the lint job. I'm gonna push an update to fix that. Would you be able to rebase on those changes and then modify the readme entry to something like:
## DNS Setup

To use sish, you need to add a wildcard DNS record that is used for multiplexed subdomains.
Adding an `A` record with `*` as the subdomain to the IP address of your server is the simplest way to achieve this configuration.

And move it below the Whitelisting ips section.


Display Click to Load placeholders

Task/Issue URL: https://app.asana.com/0/0/1204326356868965/f
Description
Steps to test this PR
Feature 1

[ ]
[ ]

UI changes

Before
After

!(Upload before screenshot)
(Upload after screenshot)

Current dependencies on/for this PR:

feature/josh/handle-click-to-load-state-message

PR #3141   👈

This comment was auto-generated by Graphite.


Comment module bug

Code of Conduct

[X] I have read and agree to the GitHub Docs project's Code of Conduct

What article on docs.github.com is affected?
If I click the comment button together, it keeps appending the same and the input sometimes fails
What part(s) of the article would you like to see updated?

Additional information

@syhk - Thanks for opening this! This seems like it would be a better fit for our wonderful support team. Please file a bug report with all the information you've provided.
I'm going to go ahead and close this issue now. Thank you 💛


sockeye gpu locking incompatible with $CUDA_VISIBLE_DEVICES

It appears that sockeye's default gpu locking behavior is incompatible with CUDA_VISIBLE_DEVICES being set. Here's an example of two independently scheduled sockeye jobs running on the same machine, with CUDA_VISIBLE_DEVICES set:
# Job 1, cluster assigns GPU 2
export CUDA_VISIBLE_DEVICES=2
python -m sockeye.train .... --device-ids 0
# Sockeye is now running on absolute gpu 2 but it thinks its training on (absolute) gpu 0
# so it puts a global lock on gpu 0 with file /tmp/sockeye.gpu0.lock
# -----
# Job 2 on same machine, separate process, cluster assigns GPU 3
export CUDA_VISIBLE_DEVICES=3
python -m sockeye.train .... --device-ids 0
# Without locking, this sockeye process would grab gpu 3 and work fine
# but it thinks it's grabbing absolute gpu 0 and sees that gpu 0 is locked
# (/tmp/sockeye.gpu0.lock) so it fails.

In other words, when CUDA_VISIBLE_DEVICES is set, sockeye still behaves as if it is receiving an absolute/global gpu id, when in fact the gpu ids have been renumbered locally with respect to that process. This results in a incorrect lock placement.
The current workaround I see is for users who set CUDA_VISIBLE_DEVICES to also run with the --disable-device-locking flag; a nicer fix would involve sockeye device locking checking CUDA_VISIBLE_DEVICES to back-compute the absolute gpu id from it's local/relative gpu id, in order to correctly number the global lock. I think it would look something like this:
# Example 1:
# local sockeye gpu id: 0
# CUDA_VISIBLE_DEVICES=3
# sockeye places global lock on gpu 3
# -----
# Example 2:
# local sockeye gpu ids: 1,2
# CUDA_VISIBLE_DEVICES=3,5,0,2
# sockeye places global lock on gpus 5 and 0
# -----
# Example 3:
# local sockeye gpu ids: 1,2
# CUDA_VISIBLE_DEVICES is not set
# sockeye places global lock on gpus 1 and 2

Thanks for this nice toolkit!

Thanks for brining this up! Sockeye is indeed unaware of CUDA_VISIBLE_DEVICES and assumes the device ids are the absolute ids. In case you know the exact device id you want to use I'd either disable locking via --device 0 --disable-device-locking (as you suggested) or directly specify the device id you want to use without using CUDA_VISIBLE_DEVICES or have CUDA_VISIBLE_DEVICES set to the all devices. To some extend CUDA_VISIBLE_DEVICES fulfills the same purpose as device locking in Sockeye, namely to make sure no two trainings run on the same device.
Maybe for the time being we could add a warning if locking is enabled and CUDA_VISIBLE_DEVICES is set. For the longer term we could think about adding the parsing/mapping logic based on CUDA_VISIBLE_DEVICES as you suggested.

Agreed that adding a warning would make this dependency explicit.
As a side note, if you don’t care about the actual (absolute) device id you can use negative integers in Sockeye to request any available device. This mixes well with CUDA_VISIBLE_DEVICE settings as you can start each sockeye process with a different ENV setting but all with —device-ids -1.

@rudinger, if you are inclined to contribute this as a PR, the changes you'd need to make are fairly simple:

determine_context() in train.py
_setup_context() in translate.py

It looks like these could also be merged and moved to utils.py (though some logic is needed for testing since only a single GPU is supported at test time). The existing expand_requested_device_ids() would be a natural place to add the CUDA environment variable logic.


Source: remove call to deprecated std.unicode.utf8Decode function

Re-ran CI with latest Zig; green now.


Config.pbtxt and max_batch_size setting for SavedModel

I have a TRTIS serving running with fairly simple config and letting TRTIS figure out the input and output shapes with "--strict-model-config=false".
The config.pbtxt file itself has max_batch_size:-1
For any other value greater than 0, I get unable to autofill for 'model_name', configuration specified max-batch 100 but model signature does not support batching
If max_batch_size is missing or if the value is set to 0, I get model output cannot have empty reshape for non-batching model
Q1. But the actual question is the following: By setting max_batch_size to -1, I am able to send a client request and get the correct response - but only for some outputs.
For an output with dim:
data_type: TYPE_INT32
        dims: 1
        reshape {
        }

I get the correct response.

But for the other outputs with dim (for example):
output {
        name: "classes"
        data_type: TYPE_FP32
        dims: -1
        dims: 1
      }

when I make the request, I get the following error:
[inference:0 5] output 'classes' for model 'model_name' has shape [1] but model configuration specifies shape [-1,1]

Is there anything else that I can do to get this to work? If I change the shape, the model does not load and if I don't, the inference request does not work.
Any suggestions or help would be great.

max_batch_size isn't allowed to have a value of -1. If TRTIS didn't flag an error on that then that is a mistake.
I'm not sure if you got autofill to work or if you just manually created the configuration. Also, you didn't mention what type of model you were using. If you start with no config.pbtxt and run with --log-verbose=1 --strict-model-config=false you can see in the console output what TRTIS saw as the model metadata and potentially more information on why it wasn't able to generate a config.

Thanks for the reply @deadeyegoodwin
Here are more details.

It is an object detection model

I do have a config.pbtxt which has only three lines and I rely on autofill to get the input and output dims

name: "model_name"
platform: "tensorflow_savedmodel"
max_batch_size:-1

If I have anything else, the server does not come up and the error is as follows:
a) If the max_batch_size is > 0 (eg: 100 or 1), I get configuration specified max-batch 100 but model signature does not support batching
b) Since it says model does not support batching, to disable batching if I set batch size to 0, I get model output cannot have empty reshape for non-batching model
Is there something I can do while creating the Savedmodel to get around these errors if -1 is in-correct (that is the only value with which the server starts up)
Or is there a diff config.pbtxt that I can use. If I try it with no config.pbtxt, I again get model output cannot have empty reshape for non-batching model for model_name
Also, if I give it a frozen model - without a config.pbtxt - the server comes up but the status API does not show input and output dims - is autofill supported for a frozen model without a config.pbtxt supported

Since you have a TF model you should save it as a saved-model (graphdef aka frozen model is an older format that doesn't embed as much metadata and so the TRTIS autofill can't do much with it).
Then:

remove config.pbtxt completely
run trtserver with --log-verbose=1 --strict-model-config=false
share the console output. That output will contain information showing the saved-model metadata that the TRTIS autofiller is working with

When TRTIS says the model does not support batching, that means that the first dimension on every input and output is not -1 (or None in TF terminology). When means that TRTIS doesn't know how the model handles batching and so TRTIS doesn't allow it. The empty reshape means that as far as TRTIS can determine your model has an output that is always  an empty vector, [ ]. TRTIS doesn't like that...

Thanks for the information @deadeyegoodwin.
Here is what I have:

The SavedModel that I have works with TF Serving
Did try --log-verbose=1  before  but it seems to be printing the tensor values and hence the size of the output  if very large. Have tried to capture most of the useful information below:

 trtserver --model-store /models  --strict-model-config=false --log-verbose=1
I0506 17:00:05.443149 15213 main.cc:267] Starting endpoints, 'inference:0' listening on
I0506 17:00:05.443236 15213 main.cc:271]  localhost:8001 for gRPC requests
I0506 17:00:05.443408 15213 grpc_server.cc:265] Building nvrpc server
...

2019-05-06 17:00:05.575978: I external/org_tensorflow/tensorflow/cc/saved_model/reader.cc:31] Reading SavedModel from: /models/model_name/1/saved_model
2019-05-06 17:00:05.596011: I external/org_tensorflow/tensorflow/cc/saved_model/reader.cc:54] Reading meta graph with tags { serve }
....
external/org_tensorflow/tensorflow/cc/saved_model/loader.cc:183] Restoring SavedModel bundle.
2019-05-06 17:00:09.457858: I external/org_tensorflow/tensorflow/cc/saved_model/loader.cc:298] SavedModel load for tags { serve }; Status: success. Took 3881876 microseconds.
I0506 17:00:09.679315 15213 loader.cc:50] Loaded saved-model: meta_info_def
......
....
...
signature_def {
  key: "serving_default"
  value {
    inputs {
      key: "in_image"
      value {
        name: "image:0"
        dtype: DT_UINT8
        tensor_shape {
          dim {
            size: 120
          }
          dim {
            size: 192
          }
          dim {
            size: 3
          }
        }
      }
    }
    outputs {
      key: "classes"
      value {
        name: "classes:0"
        dtype: DT_FLOAT
        tensor_shape {
          dim {
            size: -1
          }
          dim {
            size: 1
          }
        }
      }
    }
    outputs {
      key: "detections"
      value {
        name: "detections:0"
        dtype: DT_FLOAT
        tensor_shape {
          dim {
            size: -1
          }
          dim {
            size: 4
          }
        }
      }
    }
    outputs {
      key: "num_detections"
      value {
        name: "num_detections:0"
        dtype: DT_INT32
        tensor_shape {
        }
      }
    }
    method_name: "tensorflow/serving/predict"
  }
}

I0506 17:00:34.134484 15213 model_config_utils.cc:198] autofilled config: name: "model_name"
platform: "tensorflow_savedmodel"
input {
  name: "in_image"
  data_type: TYPE_UINT8
  dims: 120
  dims: 192
  dims: 3
}
output {
  name: "detections"
  data_type: TYPE_FP32
  dims: -1
  dims: 4
}
output {
  name: "classes"
  data_type: TYPE_FP32
  dims: -1
  dims: 1
}
output {
  name: "num_detections"
  data_type: TYPE_INT32
  dims: 1
  reshape {
  }
}
default_model_filename: "saved_model"

E0506 17:00:34.134557 15213 server.cc:137] model output cannot have empty reshape for non-batching  model for model_name

Based on your comments, I think I may be able to unblock myself by removing the num_detections output tensor. Thank you.
The issue seems to be the:

TF Serving output:

outputs {
      key: "num_detections"
      value {
        name: "num_detections:0"
        dtype: DT_INT32
        tensor_shape {
        }
      }
    }

TRTIS conversion - where the error shows up with batching is not-possible:

output {
  name: "num_detections"
  data_type: TYPE_INT32
  dims: 1
  reshape {
  }
}

TRTIS is able to respond with the right number for "num_detections" if I get around the check for empty []

What is confusing TRTIS is that the num_detections output always has zero size. Is that really true? If so why is it in the model? Anyway, if you remove the num_detections output from the model configuration you should be ok.


Fix SCALED_VERSION for v8

Hello,
This is a potential fix for #128
Please check and confirm and of course feel free to close if you have another way to fix it :)
Thanks !

Closing as obsolete since the change was intended but packages repository was not updated yet (see https://github.com/microsoft/linux-package-repositories/issues/113 for info)


Add getIdProp

Give true or false, then it will return only idProp or not.

+1 for this feature. Wondering why it wasn't added to master yet.


Issue while trying to use Google MailAPI client using Gradle

BUILD SUCCESSFUL in 2s
2 actionable tasks: 2 executed
rajesh-2755:Orgcreation rajesh-2755$ mkdir -p src/main/java src/main/resources
rajesh-2755:Orgcreation rajesh-2755$ gradle -q run
warning: [options] bootstrap class path not set in conjunction with -source 1.7
1 warning
Error: Could not find or load main class GmailQuickstart
FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':run'.

Process 'command '/Library/Java/JavaVirtualMachines/jdk1.8.0_131.jdk/Contents/Home/bin/java'' finished with non-zero exit value 1

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

Was this just running the quickstart? Or were you trying to do something in your own project?

It looks like this is running the Gmail Quickstart.

@sqrrrl Can you triage this or unassign yourself?
This issue is probably from running the sample in the wrong directory like https://github.com/gsuitedevs/java-samples/issues/28.

This is a different error than #28. Please follow #57 instead.

running the quickstart
Thanks & Regards,
Rajeshwaran Angamuthu
900 345 8596
On Thu, Aug 9, 2018 at 10:59 PM Steve Bazyl<EMAIL_ADDRESS>wrote:

Was this just running the quickstart? Or were you trying to do something
in your own project?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/gsuitedevs/java-samples/issues/15#issuecomment-411835675,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AoNpYnGNLZq7Cij3x2eENMjRpRJjJcCIks5uPHFmgaJpZM4VxkSb
.


feat: add blue mountain PA

Adds parsing for Blue Mountain in Pennsylvania. This was no easy feat, they sure make it difficult to scrape their conditions page

Thanks!
Could you have a quick look at my changes and let me know if everything still works for you.
BTW: I feel for you. One could probably write a not so short book on the subject "How not to write HTML" based entirely on this page content. PNG image with a word LIFTS  rendered - why??? At least pretty much everything is called EASIEST including another image file with a phrase Most Difficult.

Just tested and everything looks to still be working for me!

Thanks! Should be up and running now 🥳


SWO-MOS1PA Swann Motion and temperature sensor (occupancy)

Device description
SWANN Motion Detector
zigbee2mqtt:debug 2020-07-20 19:29:54: Received Zigbee message from 'SwannONe Motion Sensor', type 'commandStatusChangeNotification', cluster 'ssIasZone', data '{"zonestatus":49,"extendedstatus":0}' from endpoint 1 with groupID 0
zigbee2mqtt:info  2020-07-20 19:29:54: MQTT publish: topic 'zigbee2mqtt/SwannONe Motion Sensor', payload '{"occupancy":true,"tamper":false,"battery_low":false,"linkquality":70}'

'0x00124b00063b5aa2':
    friendly_name: SwannONe Motion Sensor

Please add Support for this device.

Hi, for temporary you can try my solution as example. Just find in sensors, sensor with options like your, make new folder, copy there sensors py and init, and add it to other init.. Just check my link. Hope will help.

Just try to
Stop domoticz,
ADD lines to

/adapters/init.py

#Swann
'MOS1PA'  MotionAndTemperatureSensorAdapter,    # SWO-MOS1PA Swann Motion and temperature sensor

Then start dz, allow to join new devices, and triger this sensor. May be it will work, if not, then will go next. ))

Support was added, could you please update plugin and check

Thank you very much. It's working good
It's the best zigbee motion sensor I've ever had


Jody/refactor

switch to the new healthchecks api endpoint
add info tag for better readibility

Oh also, minor nitpick: currently the entire pane is clickable. Do we want to only make the information button clickable?

Oh also, minor nitpick: currently the entire pane is clickable. Do we want to only make the information button clickable?

Yes we want to make the entire pane clickable because adding the information button is to let people know that there would be pop up modal.

The preview is showing nothing for some reason:

The preview is showing nothing for some reason:

Can we link the PRs in the infra-config repo that’s required by this PR?

The preview is showing nothing for some reason:

I think its because the public tag is not added, or need to rerun terraform

I provisioned healthchecks again, and this what I get now

Great! There are a few more things that I think we should polish:

The access instruction for bastion should be different than the instructions for other VMs the use bastion as a jump host:

Can we change the descriptions of the checks to match the wording used on the current page? SSH, GPU, Filesystem Capacity (note the slightly different capitalization in filesystem capacity).
The ordering of the checks are inconsistent across VMs. Perhaps we can simply sort them per VM?

@JodyZ0203 what's the status of this PR? We just hit our Healthchecks.io limit and we should get this in so we can delete the old checks

Cleaning up legacy PRs


Adding GPG gitignore

GPG private keys should not be checked into source control for the same reasons passwords should not be. This gitignore excludes them from being accidentally checked in.
Project: https://gnupg.org/
Documentation: https://gnupg.org/documentation/index.html

:thumbsup:


How can i set the initial state of agent in viewer

❓ Questions and Help
Hi, I am wondering how to set the initial state for agent when I use ./build/viewer.
I tried to change code in initialize_agent(self, agent_id, initial_state=None)  in simulator.py:
        agent = self.get_agent(agent_id=agent_id)         if initial_state is None:             initial_state = AgentState()         initial_state.position=np.ndarray=np.array([12.563317, 5.60849 ,-0.16552])         initial_state.rotation=np.quaternion(0,0.0980171,0,-0.995185)         initial_state.sensor_states = {}         agent.set_state(initial_state)         self._last_state = agent.state         return agent
and rebuilt it, but it did not work. The initial position of agent did not change at all.

./build/viewer runs the code in viewer.cpp and does not depend on any python functionality. You'll want to look into the C++ code to modify that application.

Ahh I see. Thanks a lot！


"Show more" doesn't work for CodeSnippets anymore

With the current version (0.33) the "Show more" button for the CodeSnippet component only adds a small padding to the bottom. The visible code is not truncated in the first place. See: https://carbon-svelte.vercel.app/components/CodeSnippet
An example, where this is still working, is here: https://carbon-components-svelte-git-fix-code-snippet-carbon-svelte.vercel.app/components/CodeSnippet
If I add the following to the style it works as in the working example:
  :global(.bx--snippet--multi .bx--snippet-container) {
    min-height: 3.5rem;
    max-height: 14.875rem;
    overflow: hidden;
  }
  :global(.bx--snippet--multi.bx--snippet--expand .bx--snippet-container) {
    max-height: 100%;
  }

Fixed in v0.36.0


Stop propagation of events

I can't find a way to Stop Propagation on click event properly.
I've used a Hack by adding :shown.sync="isShown" :triggers="[]" props on v-dropdown wrapped inside a div with @click.event="isShown = true attribute.
  <div @click.stop="isShown = true">
    <v-dropdown
      v-on="$listeners"
      v-bind="[$props, $attrs]"
      :shown.sync="isShown"
      :triggers="[]"
    >
      <slot/>
      <template #popper>
        <slot name="dropdown"></slot>
      </template>
    </v-dropdown>
  </div>

Is there another way to do it? otherwise, it would be nice to implement it. :smile:

Same question for two superposed elements having both tooltips shown on mouseover, how to stop propagation of mouseover event to not show both tooltips ?

Probably because the events are setup as passive :thinking:

Probably, yes. That prevents us to manually control the propagation of the event. You setup the events as passive for performance improvements I guess ?


Add accessibility support to SwitchButton on Windows

Fixes #693
Issue summary
There is no reliable way for screen reader users to find out the fuzzy (needs work) status. After some research, the most likely cause is as follows:

SwitchButton is owner drawn and therefore sets the BS_OWNERDRAW window style
When BS_OWNERDRAW is set, all other window styles should be ignored according to the docs
The default MSAA implementation in Windows ignores other window styles. As it relies on window style to find out whether the button is either a push button or a check box, this information is lost, and MSAA treats the toggle as a plain button without checkable state.

Solution
Implement a wxAccessible subclass as outlined in #696
Testing performed
Executed from source, ensured that the needs work button is reported as check box and that toggling the button is properly reported by the NVDA screen reader.

@vslavik Regarding code style, is there a style guide somewhere? I started with visual studio, then realized overall code style is very different from the current style of poedit.

Thanks!

@vslavik Regarding code style, is there a style guide somewhere?

Just mimic the surrounding code... I'm guessing you ask because of class naming and the somewhat schizophrenic CamelCase vs cxx_standard_like? I started switching to the latter in mostly non-GUI or abstracted code, but not sure it was such a bright idea, as it seems weird around wxWidgets' GUI code...
This looks good to me, I just pushed a removal of unneeded dynamic_cast in 4d334d127e71f4847a9fa7213395fb604832c7ee.

There is no reliable way for screen reader users to find out the fuzzy (needs work) status. After some research, the most likely cause is as follows:

I forgot to address this: that does sound plausible, thanks for making sense of the situation.
Merged now, thanks again!


How to add items to drop down list

Hi all,  (using winforms VB.NET)
I have a list of List of strings which I use to populate my menus DropDownList's, as per below (which works)
I've added the Ribbon Control to form1. But I can't add a drop down items using this code ...
Dim TempListofTablesNames As New List(Of String)()
TempListofTablesNames.AddRange(Mod_DBStructure.ListofTablesNames)
    If TempListofTablesNames.Count > 0 Then
        For i = TempListofTablesNames.Count - 1 To 0 Step -1
            Form1.ToolStrip2_SelectTableComboBox.Items.Add(TempListofTablesNames(i).ToString()) 'works'

'new ribbon code'              Form1.EditTable_RibbonButton.DropDownItems.Add(TempListofTablesNames(i).ToString(), My.Resources.table_edit.ToBitmap) 'errors'
        Next

How do I build a ribbon item? Thanks for looking.

Hi just in case someone finds it useful;
    If TempListofTablesNames.Count > 0 Then
        For i = TempListofTablesNames.Count - 1 To 0 Step -1

            Dim EditTableRibbonLink As New RibbonButton
            With EditTableRibbonLink
                .Text = TempListofTablesNames(i).ToString()
                .LargeImage = My.Resources.table_edit.ToBitmap
                .SmallImage = My.Resources._16x16_table_edit.ToBitmap
            End With
            Form1.EditTable_RibbonButton.DropDownItems.Add(EditTableRibbonLink)

		Next
	End If

Thanks Andy


English default language not working

Hello,
Thank you for this amazing project. Please note that if you set the default language to en the /admin will not load anymore.
Furthermore, even if you use fr and en as available languages, only the fr loads.
Thanks

Hello Mecanik, thanks for your issue and your interest for Hugolify,
I have some questions to better understand your issue (after some tests I wasn't able to reproduce) :

In which file do you set default language ? Is it in admin/app/config/locale.js or maybe other files related to Hugolify theme ?
Do you have more details for "the /admin will not load anymore" ? Is there an error displayed, or maybe an error in the console ? A screenshot could help
"only the fr loads." What do you expect to load and where ?

To clarify, we have two things related to languages in the cms :

config/locale.js : to select in what language the cms interface will be translated AND to select what is the default language to write content
parameters/available-languages.js : to define in what languages you will be able to write your content with decap cms.

I hope my comment will help to solve the problem or to better understand the issue, let me know if I missed something.
Thanks

Hello,
The documentation does not talk about admin/app/config/locale.js only available-languages.js (from what I can see), so I haven't touched locale.js yet.
The issue is that the CMS is only available in French and no English. After trying to edit available-languages.js, if you leave only 'en' the admin area won't load anymore. Errors are present in the console. If you leave 'fr', 'en' then the admin area loads and you can edit content in both FR and EN.
Now for the life of me I cannot understand why the admin area works now and it didn't yesterday, maybe you updated something?
Thanks

Hello,
Thanks for your informations.
To change the language of the cms ui, you need to change the locale in admin/app/config/locale.js.
You can set it to 'en' to get CMS in English.
Currently, this is not in the documentation so I'll add it. Thanks for having noticed it.
I think the errors was caused by the cache, because we didn't update the module yesterday and I cannot reproduce the error with informations you given.
I tested (and cleared cache after relaunched servers) :

locale='fr', availableLanguages = ['fr'] : display french ui as expected, edit french content
locale='fr', availableLanguages = ['fr', 'en'] : display french ui as expected, edit french and english content
locale='en', availableLanguages = ['en'] : display english ui as expected, edit english content
locale='en', availableLanguages = ['en', 'fr'] : display english ui as expected, edit english and french content
Please note that the order in availableLanguages define the order of languages in content edition

If errors remains, please let me know by sharing your availableLanguages and locale.

@arthaud-proust shouldn't we put this locale variable in availableLanguages?

@arthaud-proust shouldn't we put this locale variable in availableLanguages?

I would argue to keep different files because they aren't of same concern :

locale is for cms ui
availableLanguages is for content

But locale is used for i18n config to set default for available languages. I think this is an issue because we are merging two variable of separated concerns in this file.
I see two ways :

we keep the two files and we continue separating those two variables by adding a defaultLanguage variable in available-langues.js to set defaultLanguage for i18n. So that locale is only used to set cms ui language.
or, we put the locale inside availableLanguages, in this case availableLanguages will have two concerns : cms ui language and content language.

To respect separation of concerns I propose to go on the 1.

The current configuration system is OK, I just did not look at /admin/app/config/locale.js because it wasn't mentioned in the docs. Frankly, most people don't even read the docs... but when someone does read it, it would be nice to mention it.
Problem solved now and the CMS works in English and the main site as well.
Thanks


Fixes for language specific documentation links

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.2 out of 3 committers have signed the CLA.:white_check_mark: solancer:white_check_mark: shashi-pp:x: Shreyansh PandeyShreyansh Pandey seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


[cxx-interop] [nfc] llvm::Optional<AnyFunctionRef> -> std::unique_ptr<AnyFunctionRef>.

AnyFunctionRef is a forward declared class in SILDeclRef.h this is requred for dependency /layering reasons. However, we cannot get the alignement, etc. of a forward declared class, so we can't use it with llvm::Optional. It must only be a pointer.
(This is needed to be able to import SILDeclRef.h with C++ interop.)
This also helps us follows the general rule that C++ headers should be self-contained.

@hamishknight yeah, you're right. We should accept this API. I was just looking into it more and I found that we have an issue with eagerly loading result types when applying attributes. So, I'll close this PR and submit another one that fixes the issue with eager loading.


spec: clarify scope of imports in package with colliding declaration

Under Declarations and scope, the scope of imports is defined as follows:

The scope of the package name of an imported package is the file block of the file containing the import declaration.

This seems to suggest that the scope of an import only exists for the current file. And while this is true from the perspective of using the import, it's not necessarily true from the perspective of shadowing of an outer scope. Specifically, your import can't collide with any package variable, even if it's defined in another file. See the following example:
package main

import "play.ground/foo"

func main() {
    foo.Bar()
}
-- anotherfile.go --
package main

var foo int
-- go.mod --
module play.ground
-- foo/foo.go --
package foo

func Bar() {}

Although the import only appears in prog.go, the declaration of foo in anotherfile.go still collides with it. This results in the following error message:
./anotherfile.go:3:5: foo already declared through import of package foo ("play.ground/foo")
    ./prog.go:3:8: other declaration of foo

This might lead one to think "oh okay, imports can collide with other names introduced at package scope". However, this isn't entirely true either. For example:
package main

import "play.ground/foo"

func main() {
    foo.Bar()
}
-- anotherfile.go --
package main

import "play.ground/foo/foo"

var _ = foo.Tar()
-- go.mod --
module play.ground
-- foo/foo.go --
package foo

func Bar() {}
-- foo/foo/foo.go --
package foo

func Tar() {}

Each file has its own distinct import for foo ("play.ground/foo" and "play.ground/foo/foo"). They correspond to different identifiers brought into scope, but they do not collide. The compiler is perfectly happy to accept this.
Can this behavior be clarified in the spec? Specifically, that the package name of an import can collide with other package variables.

see https://github.com/golang/go/issues/7429

Ah yeah, this is a dupe of #7429. I didn't think it was being considered to change the current behavior.
I'll close this issue.


fix(29648): Error message related to JSDoc for non-JSDoc syntax error

Fixes #29648
@DanielRosenwasser @RyanCavanaugh At what stage is it better to handle such errors? At the parser or checker stage?

I think that as long as you don't have a complex check, it's generally better to put the error in the parser itself. The caveat is that the parser error should gracefully recover.


How should the test interface be configured?

Currently we don't have any options, no configuration, but we will need such thing as we go further.
As @sindresorhus said in this comment we can read CLI arguments using process.argv, but there should be a real API for configuring test interface, like this:
var test = require('ava');
test.config({ fullTrace: true });

If we have this, we can extend the config with CLI arguments. Individual tests' configurations should override CLI arguments.
Blocks #29 #31
@sindresorhus Please label as question.

I don't think I made myself clear enough in that thread. The debug flag is something we will handle internally, not every user. It will be passed to the Runner which will handle the changes, like making everything serial. The public interface should be as simple and config-less a possible.

@sindresorhus Then how should we check for flags inside test interface? Probably a helper function like this?
function hasFlag(key) {
  return process.argv.indexOf(`--${key}`) > -1 || process.argv.indexOf(`-${key}`) > -1;
}

Yes, we can just use something like has-flag.


SP-753: Make the SPHEREx logo on Lander pages be a backlink to the document i…

…ndex
Make the link sensitive to the document category of the document currently being displayed, and link to the index page for that category.

@jonathansick
The goal of this PR is to make the SPHEREx logo in the upper left on the lander pages (for all document types) be a link back to the main index page, with the appropriate document type selected in the sidebar.
I am somewhat unsure whether href="/ssdc-ms" (for example) -- i.e., an absolute pathname but with the scheme and hostname omitted -- is really the correct solution, or if there's a Jinja variable that I should be using for the base URL.  Also, I'm not sure whether I need something like "{%-" in order to avoid inserting extra whitespace into the rendering.

Thanks, @jonathansick .


chore(main): release 1.0.2

:robot: I have created a release beep boop
1.0.2 (2023-01-15)
Miscellaneous 🧹

allow chores to show up in changelog (41c3197)
Change action to Actions-R-Us/actions-tagger (b8c7df2)
deps-dev: bump @types/node from 18.11.17 to 18.11.18 (a3c2e4b)
deps-dev: bump @typescript-eslint/eslint-plugin (5c853a1)
deps-dev: bump @typescript-eslint/eslint-plugin (8e663d9)
deps-dev: bump @typescript-eslint/eslint-plugin (db7b5b4)
deps-dev: bump @typescript-eslint/parser from 5.47.0 to 5.47.1 (88f2f60)
deps-dev: bump @typescript-eslint/parser from 5.47.1 to 5.48.1 (302ea3a)
deps-dev: bump eslint from 8.30.0 to 8.31.0 (9b584f6)
deps-dev: bump eslint-plugin-jest from 27.1.7 to 27.2.0 (83b31e5)
deps-dev: bump eslint-plugin-jest from 27.2.0 to 27.2.1 (cca17eb)
deps-dev: bump prettier from 2.8.1 to 2.8.2 (8ee0545)
deps: bump actions/setup-node from 3.5.1 to 3.6.0 (d44b735)
deps: bump json5 from 1.0.1 to 1.0.2 (56d3bc4)
deps: bump paulschuberth/regex-extract-action from 1.0.0 to 1.0.1 (a8995b3)
deps: bump paulschuberth/regex-extract-action from 1.0.0 to 1.0.1 (61dc349)
main: release 1.0.1 (bda24d5)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/paulschuberth/regex-extract-action/releases/tag/v1.0.2 :sunflower:


Evironment variables not expanding in tasks

Environment variables (of the form ${env:name}) are not expanding, or are expanding to the empty string, when occurring in tasks.json.  This was working as of about a week ago, but didn't work today.

Version: 1.33.0 (user setup)
Commit: 0dd516dd412d42323fc3464531b1c715d51c4c1a
Date: 2019-04-04T15:14:28.026Z
OS: Windows_NT x64 10.0.17763

Also tested in the Insiders build:

Version: 1.34.0-insider (user setup)
Commit: 56f1b4795fed05677eb82a21dac4fc1ab62d747b
Date: 2019-04-09T05:19:45.294Z

Steps to Reproduce:

Create a new empty folder; add .\.vscode\tasks.json with the following contents:{
    "version": "2.0.0",
    "tasks": [
        {
            "label": "echo path",
            "type": "shell",
            "command": "Write-Host ${env:PATH}",
        }
    ]
}

Run this task (Ctrl+Shift+P, Tasks: Run Task, echo path); the output is:> Executing task: Write-Host  <

Terminal will be reused by tasks, press any key to close it.

i.e. the environment variable expands to the empty string.
Run the same command in the terminal (Ctrl+Shift+`, run Write-Host ${env:PATH}); the system path is printed.

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

I can repro, thank you for providing clear repro steps/examples.

On Windows the path env variable is "Path" and not "PATH", right?

Neither Path or PATH work now in tasks. Both used to work.

Both "Path" or "PATH" work fine in a launch config on Windows:
{
    "configurations": [
        {
            "type": "node",
            "request": "launch",
            "name": "Launch Program",
            "args": ["-e", "console.log(process.argv[1]);", "${env:PATH}"]
        }
    ]
}

Caused by https://github.com/Microsoft/vscode/commit/900d58bfe522e812acedf8720076590d853787b3

I'm also seeing this in user settings:
{
    "terminal.integrated.shell.windows": "C:\\Program Files\\Git\\bin\\bash.exe",
    "terminal.integrated.env.windows": {
        "TEST": "${env:PROCESSOR_REVISION}",
    },
}

open a terminal, and run:
$ env | grep -e TEST -e PROCESSOR_REVISION
TEST=
PROCESSOR_REVISION=3f02

@alexr00 please close if it's fixed.


Issue459 - TopAppBar doesn't match design spec

Added corrections for the TopAppBar in order to look like the most recent design
https://github.com/android/compose-samples/issues/459
Home
Before

After

Article
Before

After

Interests
Before

After

@googlebot I signed it!

Hey Angeles! thanks for the contributions.


unsafeSetLength, pushDownFromTo segfaults on small arrays

a = [1,2]
unsafeSetLength(a, 2L)
Segmentation fault (core dumped)

a = [1,2]
pushDownFromTo(a, 0L, 1L)
Segmentation fault (core dumped)

The same operations of larger arrays (length >= 4) do the expected things
Running on (Ubuntu 18.04.2 LTS, LLVM-6.0.0)

Actually this is undefined behavior, since 'a' gets interned as a constant, so we shouldn't let this get to the back end at all.  The best way to block that would be to distinguish these constant/interned value types from mutable types.
Mutation in general is dangerous here and there are a few easy ways to hit undefined behavior.  As this PL/tool developed, mutation semantics were generally not used and so safely ignored.  Probably we need to formalize those rules in a monad structure or something similar, if there's interest in it.
Anyway, we do have a couple of issues in the backlog pointing out these issues with mutation, so I will close this to consolidate.  But thank you for commenting!  :)


Is Firechat being maintained ?

Hello, I'm considering Firebug for a Chat functionality in an App and would like to know if it makes sense to build it based on Firechat.
Was Firechat not updated lately because it was not necessary or is not being maintained?
Thanks

See here: https://github.com/firebase/firechat/pull/60
A Firebase employee just had a pull request 5 days ago to move it to firebase 3.0.0. I think they're still waiting on someone to check/review it.
Also isn't firebug a development tool? not sure how that pertains to chat. Do you mean firebase?

Thanks!
:)
Let's see if someone from FireBASE reacts to this ; )

See here: #60
A Firebase member just had a pull request 5 days ago to move it to firebase 3.0.0. I think they're still waiting on someone to check/review it.


[NPM] Handle telemetry connection failures

Is this a request for help?:

Is this an ISSUE or FEATURE REQUEST? (choose one):
Handle telemetry connection failures. If there is orphaned sock file, npm should clear that up and recreate it
Which release version?:

Which component (CNI/IPAM/CNM/CNS):

Which Operating System (Linux/Windows):

For Linux: Include Distro and kernel version using "uname -a"

For windows: provide output of "$(Get-ItemProperty -Path "C:\windows\system32\hal.dll").VersionInfo.FileVersion"

Which Orchestrator and version (e.g. Kubernetes, Docker)

What happened:

What you expected to happen:

How to reproduce it (as minimally and precisely as possible):

Anything else we need to know:

fixed in v1.0.29


wiki: include instructions for contribution

The "contributing" section in the main page needs update. Currently it says it's open to any Go community member with a GH account, but that is not true any more.
It would be nice if each page has

a link to the source md file (ideally I'd like to get directed to a web-based editor like other wiki services, but that won't be a small size work)
last update timestamp (many pages and contents in the wiki are very outdated. timestamps may help users to guess freshness)

Independent of this, I think we may want to spend some time to scrub the main pages to delete/fix outdated contents and broken links. For example outdated information in the "Working with Go" section of the main page may mislead new users, search engine results, and AI models.

I agree that these would be worthy changes to make.   Do we have an owner for the wiki repo?

@rsc


warn instead of forcing ESM interoperability

To allow the required override of module ts config value, the esModuleInterop is set to true. But that has been done in all the cases.
For more flexibility, only allowSyntheticDefaultExports is set to true in the case where module is something else than commonjs AND esModuleInterop is not set to true. Also in that case a diagnostic is reported which by default would fail the compilation (but can be bypassed using the diagnostic filtering system).
This also means that user can stick to deprecated/not standardised version of import if they want.

Pull Request Test Coverage Report for Build 1863

25 of 25 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.9%) to 90.765%

Totals

Change from base Build 1862:
0.9%

Covered Lines:
974

Relevant Lines:
1023

💛  - Coveralls


fix a glitch about string formatting in sample-reads-randomly.py

I met this error when the condition of that block of codes is satisfied, as below. I think this is a simple problem and the fix should work.
[qingpeng@dev-intel14 Subset]$ sample-reads-randomly.py -N 100 -M 12000 -R 1 -o test20000.fa.100 test20000.fa
|| This is the script 'sample-reads-randomly.py' in khmer.
|| You are running khmer version 1.3
|| You are also using screed version 0.7
||
|| If you use this script in a publication, please cite EACH of the following:
||
||   * MR Crusoe et al., 2014. http://dx.doi.org/10.6084/m9.figshare.979190
||
|| Please see http://khmer.readthedocs.org/en/latest/citations.html for details.
Subsampling 100 reads using reservoir sampling.
Subsampled reads will be placed in test20000.fa.100
opening test20000.fa for reading
... 10000 reads scanned
... 20000 reads scanned
reached upper limit of %d readsTraceback (most recent call last):
File "/opt/software/khmer/1.3--GCC-4.4.5/bin/sample-reads-randomly.py", line 5, in
pkg_resources.run_script('khmer==1.3', 'sample-reads-randomly.py')
File "build/bdist.linux-x86_64/egg/pkg_resources.py", line 528, in run_script
"""List of resource names in the directory (like os.listdir())"""
File "build/bdist.linux-x86_64/egg/pkg_resources.py", line 1394, in run_script
if cop == 'not':
File "/opt/software/khmer/1.3--GCC-4.4.5/lib/python2.7/site-packages/khmer-1.3-py2.7-linux-x86_64.egg/EGG-INFO/scripts/sample-reads-randomly.py", line 179, in
main()
File "/opt/software/khmer/1.3--GCC-4.4.5/lib/python2.7/site-packages/khmer-1.3-py2.7-linux-x86_64.egg/EGG-INFO/scripts/sample-reads-randomly.py", line 144, in main
% args.max_reads
TypeError: not all arguments converted during string formatting

[x] Is it mergable?
[x] Did it pass the tests?
[x] If it introduces new functionality in scripts/ is it tested?
Check for code coverage.
[x] Is it well formatted? Look at make pep8, make diff_pylint_report,
make cppcheck, and make doc output. Use make format and manual
fixing as needed.
[x] Did it change the command-line interface? Only additions are allowed
without a major version increment. Changing file formats also requires a
major version number increment.
[x] Is it documented in the ChangeLog?
[x] Was a spellchecker run on the source code and documentation after
changes were made?

ready for review please

Nice work; LGTM.


Filtering out empty patch requests generated from the 3-way merge to avoid unnecessary load on kube apiserver

Description of the change:
This change is in the helm operator when generating patch requests during each reconcile cycle.  The change filters out empty patches generated from the 3-way merge, which could be in the form of the "{}" byte array that represents an empty map.  This byte array is neither nil or zero-length, so the release reconciler sees it as non-empty and sends out a patch request to the kube apiserver.
This addresses https://github.com/operator-framework/operator-sdk/issues/4956.
Motivation for the change:
We have a deployment scenario with hundreds of helm operators, each generating ~60 empty patch requests even there are no changes.  That caused heavy traffic onto the kube apiserver.
Testing done:

Built the operator with this change and confirmed no more such empty requests reported in #4956.
Also tested a normal reconciliation scenario, by manually modifying a resource and observing that the operator still picks up the modification and reverts it upon the reconcile event.
Updated the unit tests as well.

/lgtm

looks reasonable to me, allowing the tests to run

/cherry-pick v1.8.x
/cherry-pick v1.7.x

@estroz I had to force push the last commit to fulfill the DCO sign-off requirement.  Do you mind re-authorizing the workflows?  Thanks in advance!


Added hello world india

What the program do ?
Prints hello world in java
In what programming language it is written?
Java

This pull request exeeds the max additions limit. Please fix it and reopen


2022 MPWG Program Plan

Oct 2nd thread with call to action lands in the Community forum via John.
Program plan: https://docs.google.com/presentation/d/1jX1YfeOEN_k60voVj9CVGwT7yXvlF8NA/edit#slide=id.p1
Deadline: Candidate Program Plan Approved by Steering Committee (Due November 5, 2021)
Budget plan: https://docs.google.com/spreadsheets/d/100jj5LL0VLZIChrDmptFSRuCau5MKyCkW90QSlDEfW0/edit#gid=950497108
Deadline: Final Budget reviewed and approved by Steering Committee (Due November 19, 2021)
2021 MPWG Plan and Budget listed under MPWG page
https://microprofile.io/workinggroup/

Program plan went for a vote today with a 7 day ballot https://www.eclipse.org/lists/microprofile-wg/msg01171.html

Approved https://www.eclipse.org/lists/microprofile-wg/msg01171.html


De-hardcoded the URL with RAGMAN_BACKEND_HOST as env variable

Added RAGMAN_BACKEND_HOST as an environment variable to be able to set the server URL without hardcoding the server URL in different files.

port number should also be configurable

Where you have this:
let url = process.env.RAGMAN_BACKEND_HOST + ":5000/chat"
It should just be
let url = process.env.RAGMAN_BACKEND_HOST + "/chat"
In the .env, RAGMAN_BACKEND_HOST should be,  e.g., http://localhost:5000


AttributeError: 'pydantic_private'

Initial Checks

[X] I confirm that I'm using Pydantic V2

Description
I have a quart app where I'm trying to use quart_auth (https://quart-auth.readthedocs.io/en/latest/index.html) and Beanie as my ODM (https://beanie-odm.dev/). Beanie relies on pydantic's BaseModel for it's Document class. I'm trying to create a user class that subclasses both AuthUser from quart_auth and the Document class from Beanie.
My code
from beanie import Document
from quart_auth import AuthUser, QuartAuth

class User(AuthUser, Document):
    discord_id: str

auth = QuartAuth()
auth.user_class = User

I'm seeing this error when trying to render a template from my base route.
File ".../lib/python3.11/site-packages/quart_auth/extension.py", line 263, in load_user
    user = self.resolve_user()
           ^^^^^^^^^^^^^^^^^^^
  File ".../lib/python3.11/site-packages/quart_auth/extension.py", line 161, in resolve_user
    return self.user_class(auth_id)
           ^^^^^^^^^^^^^^^^^^^^^^^^
  File ".../lib/python3.11/site-packages/quart_auth/extension.py", line 61, in __init
    self.authid = authid
    ^^^^^^^^^^^^^
  File ".../lib/python3.11/site-packages/pydantic/main.py", line 776, in setattr
    if self.pydantic_private is None or name not in self.private_attributes:
       ^^^^^^^^^^^^^^^^^^^^^^^^^
  File ".../lib/python3.11/site-packages/pydantic/main.py", line 764, in _getattr
    return super().getattribute(item)  # Raises AttributeError if appropriate
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
AttributeError: 'User' object has no attribute '__pydantic_private'

Any fixes or workarounds here? I need the functionality from both libraries for my user class. I've looked into moving the Document class into the User class as an attribute, but that does not work because I need to be able to get the Document class from the AuthUser when calling quart_auth.current_user() and I need to get the AuthUser class from the Document class when loading the user document from the database.
It looks like this closed issue is similar if that is at all helpful https://github.com/pydantic/pydantic/issues/7855
Example Code
from beanie import Document
from quart_auth import AuthUser, QuartAuth

class User(AuthUser, Document):
    discord_id: str

user = User(1)

Python, Pydantic & OS Version
pydantic version: 2.6.0
pydantic-core version: 2.16.1
pydantic-core build: profile=release pgo=false
install path: /Users/jonathanbrenner/.pyenv/versions/3.11.2/envs/gamenight/lib/python3.11/site-packages/pydantic
python version: 3.11.2 (main, Feb 23 2023, 16:48:29) [Clang 14.0.0 (clang-14<IP_ADDRESS>)]
platform: macOS-14.2.1-x86_64-i386-64bit
related packages: typing_extensions-4.9.0 mypy-1.8.0
commit: unknown

It looks like your traceback got modified when you sent it
    if self.pydantic_private is None or name not in self.private_attributes:

That isn't in the Pydantic source code but this is
                if self.__pydantic_private__ is None or name not in self.__private_attributes__:

If your terminal has removed underscores (perhaps using them as bold formatting!) that would be problematic for this bug report, because it would be hiding which were the private attributes.

@lmmx Thank you for noticing that, I updated the traceback without the missing underscores! I think I copy and pasted it a couple times and somewhere along the way the underscores were removed somehow.

The details of those libraries are unclear to me but my impression is:
* You are subclassing a Pydantic model

* You are doing multiple inheritance with a non-Pydantic model class (`QuartAuth`)

* The non-Pydantic model base class `QuartAuth` has a private attribute `_auth_id`

* Private attributes must be declared with `PrivateAttr()` - [reference](https://docs.pydantic.dev/latest/api/fields/#pydantic.fields.PrivateAttr) + [usage docs](https://docs.pydantic.dev/latest/concepts/models/#private-model-attributes)

Therefore you're seeing this error because _auth_id is being accessed but Pydantic won't let you do that (fields cannot begin with an underscore unless explicitly declared as private attributes upfront in this way).

@lmmx Thank you for help! These assumptions are all correct. I tried declaring _auth_id as a private attribute on my User class but that results in the same error. Do you have any suggestions?
from typing import Optional

from beanie import Document
from pydantic import PrivateAttr
from quart_auth import AuthUser, QuartAuth

class User(AuthUser, Document):
    _auth_id: Optional[str] = PrivateAttr()
    discord_id: str

user = User(1)

I think your problem now is that you have 2 base classes but are presumably not initialising both... You definitely want to initialise the Pydantic model or you're not going to get that PrivateAttr set up correctly.
Er, you might solve this by manually __init__ing like the docs do. The fact you didn't get an error even trying to do this suggest to me it's possible but I'm not certain.
I find it pretty unorthodox to initialise a non-Pydantic model via multiple inheritance like this.
I would just let the other class be a private attribute too (or else I'd go on grep.app and look for examples which do what you're doing).
I'd guess that you do this by
from typing import Optional

from beanie import Document
from pydantic import PrivateAttr
from quart_auth import AuthUser, QuartAuth

class User(Document):
    _auth_id: Optional[str] = PrivateAttr()
    _user: AuthUser = PrivateAttr()
    discord_id: str

    def __init__(self, *args, **kwargs):
        auth_id = kwargs.get("auth_id")
        super().__init__(*args, **kwargs)
        self._auth_id = auth_id

user = User(discord_id="foo")

This doesn't work though, it gives a Beanie error (I'm not familiar with the library)
Traceback (most recent call last):
  File "/home/louis/lab/beanie/repro.py", line 19, in <module>
    user = User(discord_id="foo")
  File "/home/louis/lab/beanie/repro.py", line 15, in __init__
    super().__init__(*args, **kwargs)
  File "/home/louis/miniconda3/envs/jbrenner_repro/lib/python3.10/site-packages/beanie/odm/documents.py", line 189, in __init__
    self.get_motor_collection()
  File "/home/louis/miniconda3/envs/jbrenner_repro/lib/python3.10/site-packages/beanie/odm/interfaces/getters.py", line 16, in get_motor_collection
    return cls.get_settings().motor_collection
  File "/home/louis/miniconda3/envs/jbrenner_repro/lib/python3.10/site-packages/beanie/odm/documents.py", line 1084, in get_settings
    raise CollectionWasNotInitialized
beanie.exceptions.CollectionWasNotInitialized

This is downstream of beanie though so refer to I think you should refer to Beanie's docs now.

I found a solution where instead of multiple inheritance I added the Document class as an attribute on my AuthUser subclass. I can see now that my original code is an odd pattern and am happy with my solution.


Add initial UDS stream implementation

This PR adds experimental support for UDS streams, which can be used by using DD_DOGSTATSD_URL=unixstream:///path/to/dsd.sock or the following code:
    Callable<SocketAddress> addressLookup = new Callable<SocketAddress>() {
            @Override
            public SocketAddress call() throws Exception {
                return new UnixSocketAddressWithTransport(new UnixSocketAddress("/path/to/dsd.sock"), UnixSocketAddressWithTransport.TransportType.UDS_STREAM));
            }
        };

        client = new NonBlockingStatsDClientBuilder().prefix("my.prefix")
            .addressLookup(addressLookup)
            .port(0)
            .build();

Before this transport becomes GA we should revisit the builder to make it easier to use.
The telemetry will behave exactly like the go client with https://github.com/DataDog/datadog-go/pull/290
If unix:// is used or TransportType.UDS we still default to SOCK_DGRAM and do not try to guess the socket type (yet) as this isn't trivial with the current code structure (in particular when we create the client before the socket exists)

I find it a little strange that none of the gitlab CI appears to have fired. Worthwhile to push an empty commit, or rebase from main, in case gitlab was broken at the branch point of this PR.

I find it a little strange that none of the gitlab CI appears to have fired. Worthwhile to push an empty commit, or rebase from main, in case gitlab was broken at the branch point of this PR.

We don't have gitlab CI for this project

I think I addressed everything but give me some time to validate that the tests are not flakky


How to make dbeaver export of db?

Description
How to make dbeaver (Version <IP_ADDRESS>310081633/under kubuntu 22.04) export from Postgres 14 db ?
I found in submenu  of current database Tiools->Backup and
filled 1st and second tabs : https://prnt.sc/cXvQDDC2bs6e and https://prnt.sc/Qgy58MH6JdYu
but in both cases button “Next” is disabled and I can not proceed next...
What did  I miss ?
DBeaver Version
Comminity Edition Version <IP_ADDRESS>310081633
Operating System
No response
Database and driver
No response
Steps to reproduce
No response
Additional context
No response

Hello, @sergeynilov.
Unfortunately, we do not have support for native tools for Mac and Linux.
Workaround: First you have to install your local postgresql server. When this is done, just make a rightclick on your PostgreSQL connection and then click on "Edit Connection". Here you can choose your local client.

After that when you make Backup  you need to select in this menu the local client

Hope it helps.

I had the same issue with mariadb/mysql. For those who are confused like me:
Choose /usr/bin folder for that Database Client Homes.
Maybe developers should consider adding some description for this interface.

Then you can continue with the dump by the start button

Related ticket: https://github.com/dbeaver/dbeaver/issues/14620

It's been a while since no update here. If the issue is still actual please let me know and provide additional information.
For now I'm closing it as a duplicate of #14620 .


Expo Support

Hello there,
I am getting the following warning using expo for a react native app I am creating
Some dependencies are incompatible with the installed expo package version:

react-native-webview - expected version: 11.13.0 - actual version installed: 11.17.1
Your project may not work correctly until you install the correct versions of the packages.
To install the correct versions of these packages, please run: expo install [package-name ...]

The project is working fine but I am wondering if I will have issues going forward

Use expo install instead of yarn install


fix: the network will crash if no edge provided

Brief Information
This pull request is in the type of (more info about types):

[ ] build
[ ] ci
[ ] docs
[ ] feat
[x] fix
[ ] perf
[ ] refactor
[ ] test

Related issues (all available keywords):

fix #593

Details

Added the judgment of whether ths edges is empty.
Now network chart  works  well  when edges are empty.

Checklist

[x] I have read the CONTRIBUTING doc
[x] I have signed the CLA

Others

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.2 out of 3 committers have signed the CLA.:white_check_mark: wj23027:white_check_mark: andyhuang18:x: github-actions[bot]You have signed the CLA already but the status is still pending? Let us recheck it.

I met this problem before. Need to rebase your commit information~

I met this problem before. Need to rebase your commit information~

Thanks for the tips!


initialize the map when creating a jobParams

Codecov Report

Merging #66 into master will increase coverage by 0.39%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #66      +/-   ##
==========================================
+ Coverage   56.32%   56.71%   +0.39%
==========================================
  Files           7        7
  Lines         332      335       +3
==========================================
+ Hits          187      190       +3
  Misses        137      137
  Partials        8        8

Impacted Files
Coverage Δ

service/job.go
83.09% <100%> (+0.74%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8254d13...c89944a. Read the comment docs.


Collection API for VueJS and v-for

First.. v-for is very powerful but I feel the string based api is limiting, not runtime malleable, and not extensible. There are many additional data features ( paging, stop/start, lazy loading of data, events ) that would be ideally exposed in a list/collection api.
Today:
Example: http://codepen.io/ted/pen/LGvrMN
<listitem v-for="location in locations | filterBy filter in 'state' 'city' | limitBy 10" track-by="city" :location="location" :filter="filter"></listitem>

Rather than refining collections in a template, it would be better to create a collection api within the viewmodel.
var app = new Vue({
  el: '#app',
  data: {
    filter: "",
    locationsRaw: []
  },
  collections:{
      locations: Vue.collection( this.locationsRaw ).track( 'city' ).filter( this.filter , 'state' , 'city' ).limit( 10 )
  }
})

template:
<listitem v-for="location in locations" :location="location" :filter="filter"></listitem>

Creates a list and returns itself ( chain-able to make viewmodel creation simpler )
'locations': Vue.collection( datasource )

track by features
.track( id )

filter by features
.filter( filter:string, value1 , value2, ...)

limit by features
.limit( count )

order by features
.order( 'name' , -1 )

As these are not in the template, they should be malleable at runtime...
remove filters
app.locations.clearFilters()

add filter to results where population > 10000
app.locations.filter( 'population' , 10000 , 1 )

clear limit
app.locations.clearLimits()

set limit to 100
app.locations.limit(100)

These can also be done in 1 go given chain-able api:
app.locations.clearLimits().limit(100)

As with all APIS in vue, it should be extensible so you can add new filters to the collection API.
Vue.collectionFilter( 'name' , function );

Initially it should be limited to fully expose the existing list api and the ability to add extensions to it.
If the existing internal data API were exposed, it would be possible to add features like:
stop and start DOM updating
app.locations.stop()
// expensive data manipulation task
app.locations.start()

If the above were possible, you could build lazy loading, and many other data manipulation layers into Vue without exhausting the existing string v-for api. I feel this gives Vue the foundation for a larger data manipulation api but does not cross the line into framework territory.
Respectfully,
Ted :)

Isn't this achievable with a computed property?

You could generate the v-for string in a computed property but you would still be limited to the string based API in features. I would like to see the existing v-for internals exposed as an extensible collections API.
You could provide an extension mechanism for the string v-for API such that "| command arg arg" maps to:
Vue.collection( command, function( arg , arg ){} )
That feels like there is value in making the v-for string format extensible at a min.

:+1:

What I mean is this:
var filterBy = Vue.filter('filterBy')

// ... in component definition
computed: {
  filteredLocations () {
    return filterBy(this.locations, this.filter, 'state', 'city').slice(0, 10)
  }
}

You do have the full power of JavaScript - you are also not limited to the built in filters: you can write whatever filtering function you want.

Ok... well actually
Maybe wow is a better response.
My paradigm error was limiting "computed" use to primitives, I really never thought about using them with arrays / objects.
My mental model of the reactive parts of vue is a spreadsheet.
data --> raw data values entered into a cell
computed --> calculated values that depend on other cells
The problem with the spreadsheet paradigm is that 'cells' rarely contain complex data types.
So...

v-for string is just an expression with "|" delimited calls to builtin filters. So it is extensible by adding filters via Vue.filter( name , function )

computed properties can return complex types and results are cached until any dependent values change. You can create new filters to manipulate arrays/objects and use full js as needed. See http://vuejs.org/guide/reactivity.html#Inside_Computed_Properties

Closed!
Thanks Evan. Elegant


Overflow panic! in precise_time_ns()

One of my Rust experiments, which had been working well for ages, surprised me by not starting this morning. Producing a panic that I can reproduce as follows:
The code:
$ cat src/main.rs
use time::precise_time_ns;

fn main() {
    let start_time = precise_time_ns();

    println!("{}", start_time);
}

The result:
$ rustup default stable
info: using existing install for 'stable-x86_64-unknown-linux-gnu'
info: default toolchain set to 'stable-x86_64-unknown-linux-gnu'

  stable-x86_64-unknown-linux-gnu unchanged - rustc 1.39.0 (4560ea788 2019-11-04)

$ cargo clean
$ cargo run
   Compiling libc v0.2.66
   Compiling time v0.1.42
   Compiling rust_time_bug v0.1.0 (/mnt/c/Users/michael/rust_time_bug)
    Finished dev [unoptimized + debuginfo] target(s) in 5.62s
     Running `target/debug/rust_time_bug`
thread 'main' panicked at 'attempt to multiply with overflow', /home/michael/.cargo/registry/src/github.com-1ecc6299db9ec823/time-0.1.42/src/sys.rs:581:13
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace.

Running on Debian Buster.

Forgot to mention. This is Debian Buster running inside the Windows Subsystem For Linux on Win 10.
I rebooted the machine an sure enough every thing is working fine again.

Looks like you're using v0.1? A full rewrite is in progress, so running the same code on master should work (it's deprecated, though).

After some googling around it appears that this is a problem with the Windows Subsystem for Linux.
https://github.com/microsoft/WSL/issues/3840
When I patched a println! into the time crate I found both tv_sec and tv_nsec were negative!
        pub fn get_precise_ns() -> u64 {
            let mut ts = libc::timespec { tv_sec: 0, tv_nsec: 0 };
            unsafe {
                libc::clock_gettime(libc::CLOCK_MONOTONIC, &mut ts);
            }
            println!("ts.tv_sec = {}", ts.tv_sec);
            println!("ts.tv_nsec = {}", ts.tv_nsec);
            (ts.tv_sec as u64) *<PHONE_NUMBER> + (ts.tv_nsec as u64)
        }

I don't know if time-rs can be made to tolerate that.

Given that (1) it's a bug in the subsystem and (2) a full rewrite is in progress and should be released in just over two weeks, I'm going to close this as a wontfix.
I see that you mentioned on the users forum that you were timing something. I recommend using Duration::span in v0.1 or Duration::time_fn in v0.2.

OK. Sounds good.
Yes, using span or such would be better. What I have now only came about because it started out as more or less direct translation of some C code that used clock_gettime().
Sorry for the bother. Thanks.

Not an issue! If 0.1 were fully supported and maintained, I'd likely have at least attempted to hack something up as a temporary fix. As long as everything is in good faith, I'll respond in the same :slightly_smiling_face:


Azure Budget reports creation failure when no such failure occurred

When deploying an Azure Budget, the cli shows a status of **creating failed** during the update. When checking the Azure Portal, I can see the budget has been created. I believe the issue is due to the way the status is polled from Azure, based on the diagnostic message.
Expected Behavior

pulumi up should show no error
Current Behavior

pulumi up shows the following error:

Steps to Reproduce
The following snippet
    public class BillingStack : Stack
    {
        private const string SubscriptionId = "{REPLACEWITHSUBSCRIPTIONID}";

        public BillingStack()
        {
            var today = DateTime.Today;
            var beginningOfMonth = new DateTime(today.Year, today.Month, 1);

            var budget = new Budget("subBudget", new BudgetArgs
            {
                Amount = 10.00,
                BudgetName = "Subscription Budget",
                Category = "Cost",
                Scope = $"subscriptions/{SubscriptionId}/",
                TimeGrain = "Monthly",
                TimePeriod = new BudgetTimePeriodArgs
                    {StartDate = beginningOfMonth.ToString("s")}
            });
        }
    }

Context (Environment)
This was using a dotnet 3.1 application:
<Project Sdk="Microsoft.NET.Sdk">

  <PropertyGroup>
    <OutputType>Exe</OutputType>
    <TargetFramework>netcoreapp3.1</TargetFramework>
    <Nullable>enable</Nullable>
  </PropertyGroup>

  <ItemGroup>
    <PackageReference Include="Pulumi.Azure" Version="3.*" />
    <PackageReference Include="Pulumi.AzureNextGen" Version="0.2.6" />
  </ItemGroup>

</Project>

Dupe of https://github.com/pulumi/pulumi-azure-nextgen/issues/122 and has been fixed in 0.2.7


added states to simple examples notebook

I've updated section 4 of the simple examples notebook with states information.

Thanks @Ivy8127, looks good so far.


Update install instructions

go get currently modifies the go.mod file and that's probably not what you want installing a linter. Use go install instead.
https://go.dev/ref/mod

Thank you!


Can't find a find a DVTiPhoneSimulatorRemoteClient.framework on my machine

I'm trying to use ios-sim, but it seems to have a dependency on DVTiPhoneSimulatorRemoteClient.framework
A query like find /Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platform -iname "*RemoteClient*" returns nothing. Where is this supposed to be or how do I download it?

Try version 4.x


Modify dimensions value parser, and Use urfave/cli version 1.

PR Summary

Modify dimensions value parser
Explicitly use urfave/cli version 1.

Modify dimensions value parser
Enabled parsing correctly when using Amazon Resource Name (ARN) for dimension value.
e.g.  AWS Shield Advanced metrics DDoSDetected
ref: https://docs.aws.amazon.com/waf/latest/developerguide/monitoring-cloudwatch.html
Explicitly use urfave/cli version 1.
urfave/cli is now incompatible with version upgrade, so v1 is explicitly imported.

@y13i
Sorry, the build on Travis CI failed, so I will consider fixing it.


CMR-7020 - adding a clarification on when tests are needed

for ticket #10, clarify the rules on when tests are needed
The Contribution page has been updated.

Included in this change is a suggestion from Joe who also reviewed the code change. This was the last commit made, his recommendation.

Included in this change is a suggestion from Joe who also reviewed the code change. This was the last commit made, his recommendation.


dump config

Motivation
Dump config into work_dir.
Modification
# dump config
cfg.dump(osp.join(cfg.work_dir, osp.basename(args.config)))

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

Thanks for your contribution. @zeakey


uncaught exception when using the synchronous "decode" function

I'm using nodejs.

Create pretty much any little lz4 compressed file a.lz4
Using vim or whatever, delete the last few characters of the file.
Try to decompress it with decode, and it throws an Uncaught Error, as you can see below.

Unfortunately, this makes decode unusable in certain robust applications.  The underlying code I
think forgets to listen to an error event.
> try { require('lz4').decode(require('fs').readFileSync('a.lz4')) } catch(err) { console.log("oops")}
Uncaught Error: Unexpected end of LZ4 stream @0
    at Decoder.emit_Error (/projects/6659c2e3-ff5e-4bb4-9a43-8830aa951282/websocketfs/node_modules/.pnpm/lz4@0.6.5/node_modules/lz4/lib/decoder_stream.js:64:22)
    at Decoder.check_Size (/projects/6659c2e3-ff5e-4bb4-9a43-8830aa951282/websocketfs/node_modules/.pnpm/lz4@0.6.5/node_modules/lz4/lib/decoder_stream.js:70:32)
    at Decoder.read_DataBlockData (/projects/6659c2e3-ff5e-4bb4-9a43-8830aa951282/websocketfs/node_modules/.pnpm/lz4@0.6.5/node_modules/lz4/lib/decoder_stream.js:213:12)
    at Decoder._main (/projects/6659c2e3-ff5e-4bb4-9a43-8830aa951282/websocketfs/node_modules/.pnpm/lz4@0.6.5/node_modules/lz4/lib/decoder_stream.js:310:25)
    at Decoder._flush (/projects/6659c2e3-ff5e-4bb4-9a43-8830aa951282/websocketfs/node_modules/.pnpm/lz4@0.6.5/node_modules/lz4/lib/decoder_stream.js:280:7)
    at Decoder.final [as _final] (node:internal/streams/transform:132:10)
    at callFinal (node:internal/streams/writable:698:12)
    at prefinish (node:internal/streams/writable:710:7)
    at finishMaybe (node:internal/streams/writable:720:5)
    at Writable.end (node:internal/streams/writable:634:5) {
  domainEmitter: Decoder {
    _readableState: ReadableState {
      objectMode: false,
      highWaterMark: 16384,
      buffer: BufferList { head: null, tail: null, length: 0 },
      length: 0,
      pipes: [],
      flowing: true,
      ended: false,
      endEmitted: false,
      reading: false,
      constructed: true,
      sync: false,
      needReadable: false,
      emittedReadable: false,
      readableListening: false,
      resumeScheduled: true,
      errorEmitted: false,
      emitClose: true,
      autoDestroy: true,
      destroyed: false,
      errored: null,
      closed: false,
      closeEmitted: false,
      defaultEncoding: 'utf8',
      awaitDrainWriters: null,
      multiAwaitDrain: false,
      readingMore: false,
      dataEmitted: false,
      decoder: null,
      encoding: null,
      [Symbol(kPaused)]: false
    },
    _events: [Object: null prototype] {
      prefinish: [Function: prefinish],
      data: [Function (anonymous)]
    },
    _eventsCount: 2,
    _maxListeners: undefined,
    _writableState: WritableState {
      objectMode: false,
      highWaterMark: 16384,
      finalCalled: true,
      needDrain: false,
      ending: true,
      ended: false,
      finished: false,
      destroyed: false,
      decodeStrings: true,
      defaultEncoding: 'utf8',
      length: 0,
      writing: false,
      corked: 0,
      sync: true,
      bufferProcessing: false,
      onwrite: [Function: bound onwrite],
      writecb: null,
      writelen: 0,
      afterWriteTickInfo: [Object],
      buffered: [],
      bufferedIndex: 0,
      allBuffers: true,
      allNoop: true,
      pendingcb: 2,
      constructed: true,
      prefinished: false,
      errorEmitted: false,
      emitClose: true,
      autoDestroy: true,
      errored: null,
      closed: false,
      closeEmitted: false,
      [Symbol(kOnFinished)]: []
    },
    allowHalfOpen: true,
    options: {},
    binding: {
      compressBound: [Function (anonymous)],
      compress: [Function (anonymous)],
      compressLimited: [Function (anonymous)],
      compressHC: [Function (anonymous)],
      compressHCLimited: [Function (anonymous)],
      uncompress: [Function (anonymous)],
      uncompress_fast: [Function (anonymous)]
    },
    buffer: <Buffer f7 02 23 20 77 65 62 73 6f 63 6b 65 74 66 73 0a 0a 2a 2a 0f 00 f1 10 3a 2a 2a 20 6c 69 6b 65 20 73 73 68 66 73 2c 20 62 75 74 20 6f 76 65 72 20 61 20 ... 2685 more bytes>,
    pos: 0,
    descriptor: {
      blockIndependence: true,
      blockChecksum: false,
      blockMaxSize: 65536,
      streamSize: false,
      streamChecksum: true,
      dict: false,
      dictId: 0
    },
    state: 6,
    notEnoughData: true,
    descriptorStart: 4,
    streamSize: null,
    dictId: null,
    currentStreamChecksum: null,
    dataBlockSize: 3058,
    skippableSize: 0,
    [Symbol(kCapture)]: false,
    [Symbol(kCallback)]: null
  },
  domainThrown: false
}
> <Buffer >

I used the following async code as workaround, for anybody who sees this (I don't really need sync, but like the api):
/*
Various utilities

*/
import { createDecoderStream, createEncoderStream } from "lz4";
import { createReadStream, createWriteStream } from "fs";
import { PassThrough } from "stream";

export async function readFileLz4(path: string): Promise<Buffer> {
  const decoder = createDecoderStream();
  const input = createReadStream(path);
  const output = new PassThrough();
  input.pipe(decoder).pipe(output);

  const chunks: Buffer[] = [];
  const waitForFinish = new Promise((resolve, reject) => {
    output.on("finish", resolve);
    output.on("error", reject);
    output.on("data", (chunk) => {
      chunks.push(chunk);
    });
  });
  await waitForFinish;
  return Buffer.concat(chunks);
}

export async function writeFileLz4(path: string, contents: string) {
  // We use a stream instead of blocking in process for compression
  // because this code is likely to run in the project's daemon,
  // and blocking here would block interactive functionality such
  // as terminals.

  // Create readable stream from the input.
  const input = new Readable({
    read() {
      this.push(contents);
      this.push(null);
    },
  });
  // lz4 compression encoder
  const encoder = createEncoderStream();
  const output = createWriteStream(path);
  // start writing
  input.pipe(encoder).pipe(output);
  // wait until done
  const waitForFinish = new Promise((resolve, reject) => {
    output.on("finish", resolve);
    output.on("error", reject);
  });
  await waitForFinish;
}


Implement Patron Class

Patron: A class that represents a patron. The Patron class should have properties such
as name, address, phone number, and list of books they have borrowed.

Created and tested


953 change changed_since scope to use last_published_at

Context
Why are we doing this?
To ensure providers are exported whenever they, or any entities that indicate the the provider info has changed.
Background
To support the incremental results in the API providers endpoint we want to update the last_published_at column on the provider when a related entity, site or provider enrichments for example. This will be needed in the future once the Rails backend takes on the work of updating the providers and their related entities.
The C# has already been updated to populate this data. https://github.com/DFE-Digital/manage-courses-api/pull/267
Changes proposed in this pull request

Switch to provider.last_published_at instead of updated_at
Schema has been manually modified to match changes from .NET app.

do not merge
Until the c# schema change has shipped to prod https://github.com/DFE-Digital/manage-courses-api/pull/267

https://github.com/DFE-Digital/manage-courses-api/pull/267 has shipped so good to go

rebased


Current files for our game

Fixes still need to be applied like error checking and other edge cases I've missed.

Please review and highlight edge cases/error checking i've missed.
These files aren't fully completed yet, but they work for now.


Fix nested buffers

Binary data must be passed to widgets in separate buffers instead of the main json structure.
The serialization code is incorrect for nested buffer paths like ["vertices", "data"]. This never happens in any existing widgets, they all use a non-nested path like ["values"]. We run into this problem now that we can add new widgets.
The code is supposed to strip the buffers from the json structure. The recursive case is wrong, it doesn't rebuild the original object (upd must have access to the original structure, v, not just the nested recursive case v'). Note that I only tested this in the aeson 2 case, but I did update version 1 case as well.

I'm not 100% following the reasoning but happy to merge this since it doesn't affect any existing widgets. Thanks!


ANSI-352: Do not use DH cipher suite

Description
Fixes # ANSI-352
Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Test Configuration:
Ran ansible for 1 zk, 1 kafka and 1 SR node on AWS rhel 8.2 ami without the change (got dh key too small error on SR healthcheck) and with the change (no error).
I'll be testing this further on a bigger environment.
Checklist:

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

This is awesome! Thanks for making the PR, the question is should those properties be conditionally added only for rhel8 or should they be added for all os's.
Also we could take the stance that supporting rhel8 will come out in 6.0, in which case you should make your PR into the master branch. Alternatively we call this a bug in 55 and the PR is where it should be
And the final thing that we should add as part of this PR is a molecule scenario that tests rhel8 to validate it all. I can help with that.

I think we should consult with other folks (security, release and product probably) on which versions should support it. My gut feeling is we want to avoid known vulnerabilities for any OS and version, not just rhel8. And I think rhel8 support is planned starting with 5.5 (probably 5.5.1?). OTOH one can argue that this should probably be solved in the code of the product itself, if this setting becomes semi-mandatory.
cc @sdanduConf @andrewegel

@stan-confluent @domenicbove  Really great work here.  We don't officially support rhel 8 at this time, I think we should confirm if we will with 6.0 or not, in which case we can merge this into master for that release.

@stan-confluent Thank you for this fix! I have the same problem but with connect. Can you confirm your fix works on the health-check there? For me it sadly does not.

@Fobhep Nothing that can be made public at this time unfortunately.

@Fobhep Nothing that can be made public at this time unfortunately.

I see - thanks anyways :)

Closing because added in this pr: https://github.com/confluentinc/cp-ansible/pull/307
Will be released with 6.0
@Fobhep for the time being, you can either run with the update-crypto-policies --set LEGACY && reboot command, or try using the master branch, which has the ciphers

@domenicbove & @Fobhep I'm probably stating the obvious, but LEGACY policies are probably less secure than current ones, so while it probably is a good way to get unblocked, I don't think it should be a long-term solution

Yes, agreed @stan-confluent the legacy policies is not a real solution, thats why the 60 branch will have this fix in it. Also I found an even better solution here: https://github.com/confluentinc/cp-ansible/pull/310


chore(main): release 0.5.0

:robot: I have created a release beep boop
0.5.0 (2024-09-28)
Documentation

add Vitepress site with Github Pages (#18) (245efbe)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v0.5.0
:sunflower:


Bitcoin Cash + Bitpay, "Could not load payment request"

Hi everyone,
Haven't dug into this much but found that Bitcoin Cash on Bitpay payment requests does not seem to be working. Says "Could not load payment request" after scanning the QR code.
Thanks!

Seems to be working with 2.2 now. Thanks!


ADALTokenCache.AfterAccessNotification always creates new database entry

The code within Models/AdalTokenCache.cs creates a new instance of UserTokenCache without checking for the existence of a cached value.  The resulting EntityState is therefore always EntityState.Added, causing records to be inserted rather than updating the existing entry.
if (this.HasStateChanged)
{
  Cache = new UserTokenCache
  {
    webUserUniqueId = User,
    cacheBits = this.Serialize(),
    LastWrite = DateTime.Now
  };
  //// update the DB and the lastwrite
  db.Entry(Cache).State = Cache.UserTokenCacheId == 0 ? EntityState.Added : EntityState.Modified;
  db.SaveChanges();
  this.HasStateChanged = false;
}

Similar issues issues have been resolved in other repositories.  Will submit a pull request if desired.

Closing all issues since this repo is being archived and no longer maintained.


Unable to reach app. Please try again.

Hi, followed all the instructions however unable to use the open badges application.
Reviewed application insights and can see a few execptions and dependencies which seem to be failing.
In Teams i can sign in and then get the error 'Unable to reach app. Please try again."
These are the errors that i can see in azure:

Also receive the following when trying the launch URL manually as a test:
{
"error": {
"code": "ServiceError",
"message": "Missing required query string parameter: state. Url = https://token.botframework.com/.auth/web/redirect?code="
}
}

I'm receiving the same error, though I'm not seeing any errors in Application Insights. I did notice in the URL you have above there is an extra slash HTTP: GET api.badgr.io/**/**v2/users/self

I'm receiving the same error, though I'm not seeing any errors in Application Insights. I did notice in the URL you have above there is an extra slash HTTP: GET api.badgr.io/**/**v2/users/self


pin dependencies in requirements.txt

@talhajunaidd Thanks for the PR. For now, we are considering no fixed version yet for zero, the test requirements can be pinned, in that case please remove pinned requirements for zero.

better to close this MR :)


Spotify has deprecated web Apis audio features

Hello.
Here it says that they've deprecated the audio-features apis  so if you're having this issue :
 {'data': {'event_type': '__AD_LOG_EVENT', 'data': {'level': 'WARNING', 'app_name': 'spotify_mood_lights_sync', 'message': 'Traceback (most recent call last):\n  File "/usr/lib/python3.11/site-packages/spotipy/client.py", line 275, in _internal_call\n    response.raise_for_status()\n  File "/usr/lib/python3.11/site-packages/requests/models.py", line 1021, in raise_for_status\n    raise HTTPError(http_error_msg, response=self)\nrequests.exceptions.HTTPError: 403 Client Error: Forbidden for url: https://api.spotify.com/v1/audio-features/?ids=1LC2Hcgf2RgEbxPHNTnzaC\n\nDuring handling of the above exception, another exception occurred
You know who to thank for..
Thanks a lot to the developers that made this awesome integration and put a lot of love and effort on this.

Thanks for the heads-up. Looks like this project is dead then. Maybe some alternative service will emerge that I could use, but I doubt that there's anything close to Spotify's dataset.


[SPARK-30914][CORE][DOC] Add version information to the configuration of UI

What changes were proposed in this pull request?
1.Add version information to the configuration of UI.
2.Update the docs of UI.
I sorted out some information show below.

Item name
Since version
JIRA ID
Commit ID
Note

spark.ui.showConsoleProgress
1.2.1
SPARK-4017
04b1bdbae31c3039125100e703121daf7d9dabf5#diff-364713d7776956cb8b0a771e9b62f82d

spark.ui.consoleProgress.update.interval
2.1.0
SPARK-16919
e076fb05ac83a3ed6995e29bb03ea07ea05e39db#diff-fbf4e388a66b6a37e984b91cd71a3e2c

spark.ui.enabled
1.1.1
SPARK-3490
937de93e80e6d299c4d08be426da2d5bc2d66f98#diff-364713d7776956cb8b0a771e9b62f82d

spark.ui.port
0.7.0
None
f03d9760fd8ac67fd0865cb355ba75d2eff507fe#diff-ed8dbcebe16fda5ecd6df1a981dc6fee

spark.ui.filters
1.0.0
SPARK-1189
7edbea41b43e0dc11a2de156be220db8b7952d01#diff-f79a5ead735b3d0b34b6b94486918e1c

spark.ui.allowFramingFrom
1.6.0
SPARK-10589
5dbaf3d3911bbfa003bc75459aaad66b4f6e0c67#diff-f79a5ead735b3d0b34b6b94486918e1c

spark.ui.reverseProxy
2.1.0
SPARK-15487
92ce8d4849a0341c4636e70821b7be57ad3055b1#diff-364713d7776956cb8b0a771e9b62f82d

spark.ui.reverseProxyUrl
2.1.0
SPARK-15487
92ce8d4849a0341c4636e70821b7be57ad3055b1#diff-364713d7776956cb8b0a771e9b62f82d

spark.ui.killEnabled
1.0.0
SPARK-1202
211f97447b5f078afcb1619a08d2e2349325f61a#diff-a40023c80383451b6e29ee7a6e0593e9

spark.ui.threadDumpsEnabled
1.2.0
SPARK-611
866c7bbe56f9c7fd96d3f4afe8a76405dc877a6e#diff-5d18fb70c572369a0fff0b97de94f265

spark.ui.prometheus.enabled
3.0.0
SPARK-29064
bbfaadb280a80b511a98d18881641c6d9851dd51#diff-f70174ad0759db1fb4cb36a7ff9324a7

spark.ui.xXssProtection
2.3.0
SPARK-22188
5a07aca4d464e96d75ea17bf6768e24b829872ec#diff-6bdad48cfc34314e89599655442ff210

spark.ui.xContentTypeOptions.enabled
2.3.0
SPARK-22188
5a07aca4d464e96d75ea17bf6768e24b829872ec#diff-6bdad48cfc34314e89599655442ff210

spark.ui.strictTransportSecurity
2.3.0
SPARK-22188
5a07aca4d464e96d75ea17bf6768e24b829872ec#diff-6bdad48cfc34314e89599655442ff210

spark.ui.requestHeaderSize
2.2.3
SPARK-26118
9ceee6f188e6c3794d31ce15cc61d29f907bebf7#diff-6bdad48cfc34314e89599655442ff210

spark.ui.timeline.tasks.maximum
1.4.0
SPARK-7296
a5f7b3b9c7f05598a1cc8e582e5facee1029cd5e#diff-fa4cfb2cce1b925f55f41f2dfa8c8501

spark.acls.enable
1.1.0
SPARK-1890 and SPARK-1891
e3fe6571decfdc406ec6d505fd92f9f2b85a618c#diff-afd88f677ec5ff8b5e96a5cbbe00cd98

spark.ui.view.acls
1.0.0
SPARK-1189
7edbea41b43e0dc11a2de156be220db8b7952d01#diff-afd88f677ec5ff8b5e96a5cbbe00cd98

spark.ui.view.acls.groups
2.0.0
SPARK-4224
ae79032dcf160796851ca29116cca146c4d86ada#diff-afd88f677ec5ff8b5e96a5cbbe00cd98

spark.admin.acls
1.1.0
SPARK-1890 and SPARK-1891
e3fe6571decfdc406ec6d505fd92f9f2b85a618c#diff-afd88f677ec5ff8b5e96a5cbbe00cd98

spark.admin.acls.groups
2.0.0
SPARK-4224
ae79032dcf160796851ca29116cca146c4d86ada#diff-afd88f677ec5ff8b5e96a5cbbe00cd98

spark.modify.acls
1.1.0
SPARK-1890 and SPARK-1891
e3fe6571decfdc406ec6d505fd92f9f2b85a618c#diff-afd88f677ec5ff8b5e96a5cbbe00cd98

spark.modify.acls.groups
2.0.0
SPARK-4224
ae79032dcf160796851ca29116cca146c4d86ada#diff-afd88f677ec5ff8b5e96a5cbbe00cd98

spark.user.groups.mapping
2.0.0
SPARK-4224
ae79032dcf160796851ca29116cca146c4d86ada#diff-afd88f677ec5ff8b5e96a5cbbe00cd98

spark.ui.proxyRedirectUri
3.0.0
SPARK-30240
a9fbd310300e57ed58818d7347f3c3172701c491#diff-f70174ad0759db1fb4cb36a7ff9324a7

spark.ui.custom.executor.log.url
3.0.0
SPARK-26792
d5bda2c9e8dde6afc075cc7f65b15fa9aa82231c#diff-f70174ad0759db1fb4cb36a7ff9324a7

Why are the changes needed?
Supplemental configuration version information.
Does this PR introduce any user-facing change?
No
How was this patch tested?
Exists UT

Test build #119383 has started for PR 27806 at commit a6c2ae5.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/24120/
Test PASSed.

Test build #119384 has started for PR 27806 at commit 28adc5a.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/24121/
Test PASSed.

Test build #119383 has finished for PR 27806 at commit a6c2ae5.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/119383/
Test FAILed.

retest this please

Test build #119396 has started for PR 27806 at commit 28adc5a.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/24133/
Test PASSed.

Test build #119384 has finished for PR 27806 at commit 28adc5a.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/119384/
Test PASSed.

Test build #119396 has finished for PR 27806 at commit 28adc5a.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/119396/
Test FAILed.

retest this please

Test build #119430 has started for PR 27806 at commit 28adc5a.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/24167/
Test PASSed.

Merged to master.

Test build #119430 has finished for PR 27806 at commit 28adc5a.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/119430/
Test FAILed.

Merged to branch-3.0 too


:sparkles: exclude certain datasets from metadata changes

Fix for https://github.com/owid/etl/pull/2995
This PR reduces false positives from data & metadata changes (all are described in code in detail)

Exclude certain datasets like COVID and AI Epoch from METADATA CHANGE in chart-diff (fixing the root cause turned out to be too challenging)
Use catalogPath instead of variableId when comparing data & metadata checksums
Remove data & metadata checksums change when variable is updated by indicator upgrader

If you want to test it, you can check that running it against cset-ai shows only relevant config changes
STAGING=cset-ai streamlit run apps/wizard/app_pages/chart_diff/app.py

Sweet, I see there are three charts appearing in chart-diff if I enable 'show all' option ー is that expected?

Those charts use different descriptionKey. I have no idea why, but I'll look into it in a different PR.


chore(main): release 2.0.0

:robot: I have created a release beep boop
2.0.0 (2024-02-09)
⚠ BREAKING CHANGES

change the user command name

Features

animate down square movements (4578e82)
animate left square movements (0a1156a)
animate right square movements (123e8bf)
animate up square movements (a71e61c)
change the user command name (83845da)

Bug Fixes

calculations (a67d125)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/NStefan002/2048.nvim/releases/tag/v2.0.0 :sunflower: