Python 3.6 no longer available via deadsnakes on latest Ubuntu LTS (22.04 / Jammy Jellyfish)

The current install script uses Python 3.6 -- however, Python 3.6 isn't available in the deadsnakes repository when using the latest Ubuntu LTS (see https://wiki.ubuntu.com/Releases -- 22.04 / Jammy Jellyfish).
It seems that the earliest version if Python available is Python 3.7 which does seems to have torch==1.9.0+cu111 requirement.
Changing the install script to use Python 3.7 (see diff below) seems to work;
diff --git a/install.sh b/install.sh
old mode 100644
new mode 100755
index 1abb1bc..5a35f53
--- a/install.sh
+++ b/install.sh
@@ -7,14 +7,14 @@ then
 fi

 # Install python
-if ! python3 --version 2>&1 | grep '3.6';
+if ! python3 --version 2>&1 | grep '3.7';
 then
-    echo "Installing Python 3.6"
+    echo "Installing Python 3.7"
     sudo apt-get install -y software-properties-common
     sudo add-apt-repository -y ppa:deadsnakes/ppa
     sudo apt-get update --fix-missing
-    sudo apt-get install -y python3.6 python3.6-distutils python3.6-dev
-    sudo python3.6 -m easy_install pip
+    sudo apt-get install -y python3.7 python3.7-distutils python3.7-dev
+    sudo python3.7 -m easy_install pip
 else
     echo "Python installed"
 fi
@@ -23,6 +23,6 @@ fi
 sudo apt-get install -y ffmpeg libsndfile1

 # Install Dependencies
-python3.6 -m pip install -r requirements.txt
+python3.7 -m pip install -r requirements.txt
 echo "Successfully installed python packages"
 echo "Install complete"

Output at the end;
Installing collected packages: webrtcvad, Unidecode, pydub, llvmlite, inflect, audioread, Werkzeug, tqdm, torch, threadpoolctl, regex, python-engineio, pysrt, pycparser, omegaconf, numba, matplotlib, joblib, Jinja2, itsdangerous, deepspeech, bidict, torchvision, torchaudio, scikit-learn, resampy, python-socketio, nltk, Flask, cffi, soundfile, librosa, Flask-SocketIO
Successfully installed Flask-1.1.2 Flask-SocketIO-5.0.1 Jinja2-3.1.2 Unidecode-1.0.22 Werkzeug-2.1.2 audioread-2.1.9 bidict-0.22.0 cffi-1.15.1 deepspeech-0.9.3 inflect-0.2.5 itsdangerous-2.1.2 joblib-1.1.0 librosa-0.6.0 llvmlite-0.32.1 matplotlib-3.2.2 nltk-3.6.2 numba-0.47.0 omegaconf-2.0.6 pycparser-2.21 pydub-0.24.1 pysrt-1.1.2 python-engineio-4.3.3 python-socketio-5.7.0 regex-2022.6.2 resampy-0.3.0 scikit-learn-1.0.2 soundfile-0.10.3.post1 threadpoolctl-3.1.0 torch-1.9.0+cu111 torchaudio-0.9.0 torchvision-0.10.0+cu111 tqdm-4.64.0 webrtcvad-2.0.10
Successfully installed python packages
Install complete

Hi @mithro,
Thanks for letting me know. The reason the build is currently python 3.6 is because deepspeech currently only compiles correctly on windows for this version (due to some dll problems).
I'd prefer to use the same python version for both windows/linux but if this is going to be an ongoing issue for newer versions of linux we can implement this change.

It seems to work ok with 3.7 if you lock these versions in to their 3.6 version:
itsdangerous==2.0.1
werkzeug==2.0.3
joblib==1.1.0


Issues with Next button of date picker

I am currently using the latest version of ink and I have encountered a problem with the date picker. Basically, if the day 31st of any month is selected, and if also the next month does not have up to 31 days, the Next button (» by default) will not work anymore.
This also happens with the prev button as well. My guess is that it tries to find the day 31 on September and simply fails.

Can´t reproduce. I suppose you are using ink v3.1.10 which is the latest, right? If you can share a code example for debuging would be great. Also, what browser are you using?


partitionKey can't reference field annotated with @Expose({ name: })

I was trying to use the class-transformer decorator @Expose to map the attribute name in the Db to a different name in the entity. It seems to work for the most part, but doesn't seem to work when it's the partitionKey.
@Entity({
  name: "order",
  primaryKey: {
    partitionKey: "{{OrderId}}",
  },
})
export class OrderEntity {
  @Attribute()
  @Expose({ name: "OrderId" })
  orderId: string;
}

will result in :
Error: key "{{OrderId}}" references variable "OrderId" but it could not be resolved

Making partitionKey "orderId" results in the same error.

Any update on this?


rkt: authentication when fetching docker images

Following on from #194 / #364 - rkt should also support authentication when retrieving docker:// images.
This'll be blocked by appc/docker2aci#7

/cc @krnowak

Fixed by #767


Upstream v5.46.0

Upgrade upstream to v5.46.0
Fixes https://github.com/pulumi/pulumi-aws/issues/3847

/release minor


[examples] Changed the endpoint ID from 1 to 0 for the pump example apps.

Problem

Change the endpoint ID from 1 to 0 for the utility clusters.

Change overview

The endpoint ID was changed from 1 to 0 in the .zap files for the pump-app and pump-controller-app examples.
The following files changed:

examples/pump-app/pump-common/gen/endpoint_config.h
examples/pump-app/pump-common/pump-app.zap
examples/pump-controller-app/pump-controller-common/gen/endpoint_config.h
examples/pump-controller-app/pump-controller-common/pump-controller-app.zap

Testing
How was this tested? (at least one bullet point required)

Ran the ./scripts/tools/zap/generate.py examples/pump-app/pump-common/pump-common.zap
Ran the ./scripts/tools/zap/generate.py examples/pump-controller-app/pump-controller-common/pump-controller-common.zap
Then compiled and linked both example projects (pump-app and pump-controller-app) with success.

@bzbarsky-apple you are right. This might be a better solution. I was in a hurry to get the EP id changed before friday, to get this PR into the TE#3 branch. The ZAP UI tool was acting up, so I decided to just "hand-modify" the EP id in the .zap file. I will try to get the Application part moved into an other EP (Endpoint 1).

Would it be the same "Device Type" (Pump (0x0303) in this case) in both Endpoints, even though the Utility Clusters are not device specific?

For now, sure, but longer-term we should have a "Matter Utility Device" device type or something and use it for the various endpoint 0s.


HV-1313 Support group conversion for container element types

https://hibernate.atlassian.net/browse/HV-1313

I added some tests in the TCK for the BV part (only the annotations support for now though, I wanted to exercise the logic, will see if I have the time to add XML tests).
In the end, I think it was a mistake to make the annotated object the root of the container elements CascadingMetaData. It would have worked if we didn't have to support the legacy behavior of @Valid on the root element for Maps and Lists, but considering this legacy behavior, it was not a good idea.
So I think I will change it to having:

a separate CascadingMetaData for the annotated object: the container elements metadata will be empty for this one
a root CascadingMetaData (with cascading to false) for the container elements. It's really nice to have a root.

Sorry for the big patch but it was the minimum to get it working without compilation issues and existing test failures.

Ahaha :)

Closing; superseded by #736.


Create a JavaScript version of the API documentation

What problem is this solving
The official API documentation is written for TypeScript users, and does not cater to JavaScript users.
For example, I was looking for a list of instance properties and methods, but instead I end up being bombarded with "Interfaces", "Type aliases" and other TypeScript-specific terminology. Without the intimate knowledge of TypeScript, I am unable to understand what methods and properties are available to me.
This is frustrating, because I need to migrate several Vuex modules and learning TypeScript right now, is not really an option.
The beauty of Vuex was it's simplicity - making TypeScript knowledge mandatory for Pinia users takes that away completely.
Please, comment respectfully - not everybody has switched to TypeScript yet and not everybody wants to.
Proposed solution
Provide a simplified version of the API documentation for the JavaScript users, which would include a list of available properties and methods, without type annotations.
A great idea would be a toggle like on the official Vue website, where users can switch between Options and Composition API version of documentation.

The API documentation is generated from the types but it's written for JS and TS users. While I understand the big list of interfaces can be frightening, one doesn't need to learn TS to travel through the docs. Moreover, it's simply impossible for me to write and maintain yet another version of the docs. My recommendation here is to spend some time in the docs to get used to navigating through the API documentation or to use the Guide which is for everybody. It will definitely take you less time to get comfortable with it than for someone to rewrite a new API documentation for JS users.
I definitely recommend you to start from the Guide rather than the API documentation to grasp what you need to migrate from Vuex. Then, taking a look at https://pinia.vuejs.org/cookbook/migration-vuex.html#migrating-from-vuex-≤4

The beauty of Vuex was it's simplicity

If there is anything you could say about Pinia compared to Vuex is that it is simpler 😄

making TypeScript knowledge mandatory for Pinia users takes that away completely.

As shown in docs, TS is not a mandatory knowledge for Pinia. I'm confident that if you start from the getting started section docs, you will not only find the familiarity of state, actions, and getters from Vuex, but also realize how much less of a hassle pinia is compared to vuex.


*: move "PushMetrics" to pkg/metricutil

Putting PushMetrics in pkg/metricutil seems more reasonable.
PTAL @huachaohuang @siddontang

Rest LGTM

I suggest use metricutil directly and consistently.
Rest LGTM


MYCROFT_LOOSE_REQUIREMENTS

Can we add support for loose requirements?

@NeonDaniel Saw your PR but could you please add support for loose requirements?


Frontend index update script

Some sort of script that can be run periodically to check in the image storage location to see if there are any new images that are currently not in the elastic index. If any are found, they will be indexed and then thumbnails will be generated for these images.

demo test.zip
I've wrapped up a version of frontend and backend working functionally, except for authentication which we decided to postpone after the quick demo on Monday. I'm trying to have some test of deepweeds, voc, cwfid data for the demo, but currently cwfid is working but deepweeds raise something like below:

Let me know if you can recognise this error. I attached the weedcoco file of the cwfid(working) and deepweeds(not working)
Also, the uploader turns out not that scalable to handle huge amount of images being uploaded, not because of the concurrency, but the rendering cost at frontend. I'll have a deep dive to see if it's manageable later. For now, it can handle uploads of several hundred images.

The error means an annotation category name is not formatted correctly, but I'm not sure why this would be the case.

Admittedly, that regex there is a bit of a hack; the transformation should be done elsewhere, in a way that is defensive enough to not crash the system if there's a single corrupted field. However, all it requires is that each category name has a colon (:) in it, and I can't see an example of that in weedcoco/deepweeds_to_json/deepweeds_imageinfo.json, nor reason for it in the importer for it...?

Fixed with deposit logic.

Fixed with deposit logic.


#589 Implement firebase app check

What's included?
Here we are adding the recaptcha element to the vue app in order to enable the Firebase App Check:
https://firebase.google.com/docs/app-check
Who should test?
✅ Developers
How to test?
The App Check requires a key which I have stored in an env variable.
Someone will need to create one for the live platform, but I will share the one I created in slack.
Step 1 (this PR)
Add the app check to the front end application and ensure it is being correctly passed in when firebase is requested (realtime db and storage). To check this, you can go to: https://console.firebase.google.com/u/4/project/digital-platform-develop/settings/appcheck and click Storage or Realtime Database to see the % of authorised requests.
Step 2 (requires a new ticket)
Enforce it across the application. This will mean:

Clicking the Enforce buttons across the various platforms
Adding a check to all callable functions to check for context.app and reject the request if this not present (https://firebase.google.com/docs/app-check/cloud-functions)

Note:
This will prevent tools like postman from working when making requests to firebase.
Risk - how likely is this to impact other areas?
🔴 High risk - this includes a lot of changes to shared code

PREVIEW:DEVELOP
can be OFF, DEVELOP or STAGING

Thanks @warrensearle I've updated the env files, workflow files and added the secret in github for develop only. Please take a look and let me know if it's okay!


API

Thanks for stopping by to let us know something could be better!
Please run down the following list and make sure you've tried the usual "quick fixes":

Search the issues already opened: https://github.com/GoogleCloudPlatform/google-cloud-node/issues
Search StackOverflow: http://stackoverflow.com/questions/tagged/google-cloud-platform+node.js
Check our Troubleshooting guide: https://googlecloudplatform.github.io/google-cloud-node/#/docs/guides/troubleshooting
Check our FAQ: https://googlecloudplatform.github.io/google-cloud-node/#/docs/guides/faq

If you are still having issues, please be sure to include as much information as possible:
Environment details

OS:
Node.js version:
npm version:
google-cloud-node version:

Steps to reproduce

require google-cloud
?

Making sure to follow these steps will guarantee the quickest resolution possible.
Thanks!

What's up, @mbanyar033?

I am closing on the assumption that this was opened by mistake.
@mbanyar033, feel free to open a new issue if there is something you want to report. :-)


[AutoPR datafactory/resource-manager] [DataFactory]Add CompressionCodec property for ORC Dataset

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/7452

This PR has been merged into https://github.com/Azure/azure-sdk-for-go/pull/6068


feat: Added option to disable GPU acceleration

Closes #ISSUE_NUMBER
closes #184

Adds an option in the menu to disable GPU acceleration. When the menu item is clicked it relauches the application with GPU acceleration disabled.
Menu Item to disable GPU acceleration active

Menu Item to disable GPU acceleration inactive after GPU acceleration has been disabled

 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.

@Somtozech Could you please revert the last commit?

@tassoevan I have reverted the last commit.


doc: update ai-proxy doc

https://github.com/alibaba/higress/pull/1068#issuecomment-2212476221

 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.

 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.


Ban Scala 3.6.0 from cs update

From the Scala Discord:

Hi all,
some of you might have seen that Scala 3.6.0 is available on Maven Central, however it was accidentally released (instead of 3.6.0-RC1) and should not be used by anyone and especially you should avoid releasing libraries with that version.
We will fast-track 3.6.1 to replace it, and we'll also fast-track followup 3.6.x releases if needed to promptly address regressions.
Scala Steward updates for both those versions has been disabled, so you should not get automatic updates for them.
We will follow up with more information and a post mortem next week.

cs update currently updates to 3.6.0, which probably should be prevented.

Thanks for reporting! We don't have a way to blacklist releases, but once 3.6.1 is released (which is being worked on right now) the update will skip 3.6.0 which is broken.
Also, cs actually install Scala CLI underneath, which uses by default the current version of the Scala compiler, but it is not broken itself.


Infinite restart after saving a file

After saving file and compile, nodemon go wild, restarting app all over again

Check the task you are executing just there is watching not watching + run
2016-09-14 20:26 GMT+01:00 zonkil<EMAIL_ADDRESS>
After saving file and compile, nodemon go wild, restarting app all over
again
[image: image]
https://cloud.githubusercontent.com/assets/14218072/18526616/8afb2576-7ac1-11e6-88db-599eea2dab46.png
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/moizKachwala/Angular2-express-mongoose-gulp-node-typescript/issues/12,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKU3imEqqmGuY1jvrEraix3EY7ns464Eks5qqEpsgaJpZM4J9JaB
.

I am executing:
npm run deploy
which execute
"deploy": "concurrent --kill-others \"gulp watch\" \"gulp start\""

so remove start in that task
2016-09-14 20:37 GMT+01:00 zonkil<EMAIL_ADDRESS>
I am executing:
npm run deploy
which execute
"deploy": "concurrent --kill-others "gulp watch" "gulp start""
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/moizKachwala/Angular2-express-mongoose-gulp-node-typescript/issues/12#issuecomment-247129138,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKU3iu30kr_U9MAsR4haqz30x0mtAFZ0ks5qqEzhgaJpZM4J9JaB
.

If it should be removed why is it in package.json?
I haven't change anything, just downloded project and run as said in instruction.

you have to remove or start or watch, do that if you want to fix it. I
think you need to get the concept of open source, this is a project not
released to work in a company.. of course many things could be improved and
fixable
2016-09-14 21:06 GMT+01:00 zonkil<EMAIL_ADDRESS>
If it should be removed why is it in package.json?
I haven't change anything, just downloded project and run as said in
instruction.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/moizKachwala/Angular2-express-mongoose-gulp-node-typescript/issues/12#issuecomment-247136913,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKU3ioIAG4wVUdvH4-bXdb-E7p7zLlnkks5qqFOxgaJpZM4J9JaB
.

Check how it is build for example the task "default"

I thought that it was make that way to automatic compile and restart. But it dosent work. If I start "default" or "start" it always is in infinite loop. If I only start "watch" I have to manualy restart.
And what concept of open source has to do with it? I found some issue and I am letting you know about it. You can just ignore it or do what ever you want.

I have also faced the same issue and I am looking into this.

I'm pretty new in all JS stuff, but removing tasks: ['tslint'] from nodemon params helped (in some way, nodemon still restarting few times). watch execute tslint so start don't have to, I think.

Yes, something wrong in watch and tslint. I notify when its fixed

fixed


missing runtime data

I just used redis cluster as the storage, found that the runtime data was lost after restart conductor server. But the workflow and task defs of metadata not lost and still there.
My server.properties as below:
db=redis
workflow.dynomite.cluster.hosts=<IP_ADDRESS>:6379:us-east-1e
queues.dynomite.nonQuorum.port=6379
EC2_AVAILABILITY_ZONE=us-east-1e
workflow.elasticsearch.instanceType=memory
workflow.elasticsearch.url=localhost:9300

workflow.elasticsearch.instanceType - This determines the type of ES instance we are using with conductor. The two values are either MEMORY or EXTERNAL. If MEMORY, then an embedded server will be run.  Hence when the Conductor was restarted the ES data which was stored in memory was lost. The Embedded ES creates temp dir in your local, when the instance is started,  EmbeddedElasticSearch::setupDataDir method performs cleanup of the existing temp dir.

workflow.elasticsearch.instanceType - This determines the type of ES instance we are using with conductor. The two values are either MEMORY or EXTERNAL. If MEMORY, then an embedded server will be run. Hence when the Conductor was restarted the ES data which was stored in memory was lost. The Embedded ES creates temp dir in your local, when the instance is started, EmbeddedElasticSearch::setupDataDir method performs cleanup of the existing temp dir.

Ok, if i set the value to EXTERNAL and use the external ES instance, the running data can not be missing?

workflow.elasticsearch.instanceType - This determines the type of ES instance we are using with conductor. The two values are either MEMORY or EXTERNAL. If MEMORY, then an embedded server will be run. Hence when the Conductor was restarted the ES data which was stored in memory was lost. The Embedded ES creates temp dir in your local, when the instance is started, EmbeddedElasticSearch::setupDataDir method performs cleanup of the existing temp dir.

Ok, if i set the value to EXTERNAL and use the external ES instance, the running data can not be missing?

Yes, as mentioned earlier when the Conductor is restarted the Embedded ES instance is destroyed and restarted and hence ES data will be lost.
Mostly using in memory type is good for demo purpose and data will be lost once server dies.
For demo purpose you can also run ES instance in local with instanceType External. you can download the related version jar from here..  https://www.elastic.co/downloads/elasticsearch

workflow.elasticsearch.instanceType - This determines the type of ES instance we are using with conductor. The two values are either MEMORY or EXTERNAL. If MEMORY, then an embedded server will be run. Hence when the Conductor was restarted the ES data which was stored in memory was lost. The Embedded ES creates temp dir in your local, when the instance is started, EmbeddedElasticSearch::setupDataDir method performs cleanup of the existing temp dir.

Ok, if i set the value to EXTERNAL and use the external ES instance, the running data can not be missing?

Yes, as mentioned earlier when the Conductor is restarted the Embedded ES instance is destroyed and restarted and hence ES data will be lost.
Mostly using in memory type is good for demo purpose and data will be lost once server dies.
For demo purpose you can also run ES instance in local with instanceType External. you can download the related version jar from here.. https://www.elastic.co/downloads/elasticsearch

OK，get it,  Many thanks！

Thank you @mahalaxmiganesh for the responses.


CoreTest.testCamelContextAwareRegistryBeansInitialized failing after …

…RouteBuilder started implementing CamelContextAware
Fix #3002
@jamesnetherton Sending this against main

It's not clear to me whether we're just masking an actual issue here. If I search for CamelContextAware beans, then I'd expect the CamelContext to not be null.

Yeah, when thinking of it again, I am getting the same impression. On one hand user beans are their responsibility but OTOH the same holds for our RouteBuilders. We should better set @ApplicationScoped on them so that ppl. do not copy it from us. And once we do it, we can actually get back to querying the whole registry. Let me do it like that.

620f4ad:

set @ApplicationScoped on our RouteBuilders that have bean producer method
get back to querying the whole registry.

The CI failure seems to be unrelated. Let's merge


merlin 2.3.1 does not build with ocaml 4.02.0

I tried to install the merlin package on my 4.02.0 opam switch, but this fails with the following
#=== ERROR while installing merlin.2.3.1 ======================================#
# opam-version 1.2.2
# os           linux
# command      make -j 4
# path         /home/yann/.opam/4.02.0/build/merlin.2.3.1
# compiler     4.02.0
# exit-code    2
# env-file     /home/yann/.opam/4.02.0/build/merlin.2.3.1/merlin-32510-4d7566.env
# stdout-file  /home/yann/.opam/4.02.0/build/merlin.2.3.1/merlin-32510-4d7566.out
# stderr-file  /home/yann/.opam/4.02.0/build/merlin.2.3.1/merlin-32510-4d7566.err
### stdout ###
# make -f Makefile.ocamlmakefile WITH_BIN_ANNOT="" WITH_DEBUG="" REALLY_QUIET=1 ocamlmerlin
# make[1] : on entre dans le répertoire « /home/yann/.opam/4.02.0/build/merlin.2.3.1 »
# make[2] : on entre dans le répertoire « /home/yann/.opam/4.02.0/build/merlin.2.3.1 »
# OCamlMakefile:1076 : la recette pour la cible « src/ocaml/typing/typecore.cmx » a échouée
# make[2] : on quitte le répertoire « /home/yann/.opam/4.02.0/build/merlin.2.3.1 »
# OCamlMakefile:777 : la recette pour la cible « native-code » a échouée
# make[1] : on quitte le répertoire « /home/yann/.opam/4.02.0/build/merlin.2.3.1 »
# Makefile:57 : la recette pour la cible « ocamlmerlin » a échouée
### stderr ###
# File "src/ocaml/typing/typecore.ml", line 3075, characters 10-19:
# Error: This function has type
#          string -> Parsetree.expression list -> Parsetree.expression
#        It is applied to too many arguments; maybe you forgot a `;'.
# make[2]: *** [src/ocaml/typing/typecore.cmx] Erreur 2
# make[2]: *** Attente des tâches non terminées....
# make[1]: *** [native-code] Erreur 2
# make: *** [ocamlmerlin] Erreur 2

I did not manually change the compiler source in my opam installation. What is going on ?

Merlin doesn't support 4.02.0. Do you have any reason not to use 4.02.{1,2,3}?

Yes, there does not seem to be an opam switch for 4.02.{1,2,3} with the patch by Arthur Charguéraud to get improved error messages, like 4.02.0+improved-errors.
I am currently looking for a good setup for my students next years ; improved-errors is a must and merlin might be interesting too.

@ysalmon IIRC 4.02.0 had horrendous compilation times so it was not widely used by anyone. Maybe the  patch can be ported to 4.02.1 as is ?

I'm sure the folks maintaining the OPAM repo will create a switch for this if that is the case.

I can provide a patch for 4.02, but I would prefer @dbuenzli suggestion. It is not good to use 4.02.0 in general.

Thanks for this information ; I was not aware of this performance problem in 4.02.0 as we do not compile big programs. I will get in touch with Arthur Charguéraud to see if he can provide a patch for a better version of the compiler.

Be careful when setting up development environment: Merlin will display normal type error messages.
You may want to use Merlin only for type information and completion, and the normal edit-compile-fix cycle to get error messages.

Thanks for your help. You are right, it does not make sense to support 4.02.0 if it is flawed.
So we'll be using merlin in the future, but not just yet (mind you, we're still officially supposed to use Caml Light, so we are coming a long way !).


Create a new Page | CODE of CONDUCT

The following changes are needed to be done on the footer section of the website
Todo

Create a new HTML page with a matching color scheme with the content from here

Important Points

All changes should be responsive
Before creating a PR, send a screenshot in this issue thread

I am working on this

Wait until you are assigned to the Issue, or the PR will be marked as invalid.

You can start working on it @Exter-dg

Screenshots for Reference:


Model with multiple layers output

When I try to use a segmentation model trained on Ultralytics (tutorial from git repo), I get this error in QGIS :

I understand that this is currently an unsupported feature. Nevertheless, is there an alternative or a workaround?
Regards,
tb-vhbc.

Hi @tb-vhbc ,
You didn't attached configuration from the deepness window, so I can only suppose what could be wrong. If you train ULTRALYTICS yolov8 model for instance segmentation, you should use model type: Detector instead od Segmentor. Let me know if that helps you. Otherwise provide a screenshot of the deepness window, please

Hi,
Basing on your error message, maybe it is something we saw earlier. Before exporting to ONNX, create a wrapper which selects only one output:
import torch
import torch.nn
class NewModel(nn.Module):
  def __init__(self, m):
    super().__init__()
    self.m = m
  def forward(self, x):
    x = self.m(x)
    return x['out']
model_new = NewModel(model)
x = torch.rand([1, 3, 256, 256])
_ = model_new(x)

Hi @bartoszptak
Below are the captures of deepness window :

And this the error output from QGIS :
NotImplementedError: Model with multiple output layers is not supported! Use only one output layer.
Traceback (most recent call last):
  File "C:\Users/tb-vhbc/AppData/Roaming/QGIS/QGIS3\profiles\test-segmentation/python/plugins\deepness\deepness_dockwidget.py", line 264, in _browse_model_path
    self._load_model_and_display_info()
  File "C:\Users/tb-vhbc/AppData/Roaming/QGIS/QGIS3\profiles\test-segmentation/python/plugins\deepness\deepness_dockwidget.py", line 385, in _load_model_and_display_info
    self._update_model_output_format_mapping()
  File "C:\Users/tb-vhbc/AppData/Roaming/QGIS/QGIS3\profiles\test-segmentation/python/plugins\deepness\deepness_dockwidget.py", line 393, in _update_model_output_format_mapping
    for output_number in range(self._model.get_number_of_output_channels()):
  File "C:\Users/tb-vhbc/AppData/Roaming/QGIS/QGIS3\profiles\test-segmentation/python/plugins\deepness\processing\models\segmentor.py", line 77, in get_number_of_output_channels
    raise NotImplementedError("Model with multiple output layers is not supported! Use only one output layer.")
NotImplementedError: Model with multiple output layers is not supported! Use only one output layer.

Version de Python : 3.9.5 (tags/v3.9.5:0a7dcbd, May  3 2021, 17:27:52) [MSC v.1928 64 bit (AMD64)]
Version de QGIS : 3.32.1-Lima Lima, 4a078ecf63

Hi, as Bartosz wrote, please use Detector instead od Segmentor model type


CLN/TST: use assertIsInstance

Coverage remained the same at 94.737% when pulling 5e98d7a85ae8f072cfa4c97fcef2a65b632c27d3 on sinhrks:test_cln into 0cf0f412f5c14e447bb47249fdb0dc4d2ae136ee on pandas-ml:master.


Restore compatibility with UglifyJS

Fixes #15

Since multiple people seems affected by this, I'll merge and release this now

Publishing blocked on #17


Added sight checks to Mirror Image

Mirror Image doesn't function if the attacker can't see the target, relies on senses other than sight, or can perceive illusions, so I added returns if the attacker is blinded or the target is within the range of the attacker's blindsight or truesight.

Why is there an early return for what looks like a blinded check?

Because creatures that can't see are immune to Mirror Image.

This pull request appears to be messed up. The files changed is adding a few empty lines.

Yeah, I made some mistakes with the forking, still getting used to GitHub. I'm gonna close this PR and open a new, fixed one. Thank you for your patience as I find my footing with the process.


Update invite link for Codemotion Madrid

Invitation link to Slack for Codemotion Madrid is no longer valid, and also, we are transitioning to Discord, so if you can update and add the new link to Discord (invitation link shouldn't expire now): https://discord.gg/GSaX3Uq9p5
Thanks

Hello, any update on this?
Link invitation changed to this actually, to this: https://discord.gg/kG7rQj6VXv


getBoundingClientRect() incorrectly reports width for hidden containers

It is not possible to use minigrid on hidden container e.g. Bootstrap navbar menu items. Firstly you have to set display to initial, calculate width and height and then reset back

There is another problem in Internet Explorer 11.  Just change display = 'initial' to display = 'block'

Thanks @insekticid! Going to review it and update tonight 👍


Dev

Related Issue

Creating a new section in Relativity for Mass Variation or Relativistic Mass

Fixes: #497
Describe the changes you've made
Things Done by me :

Added the full definition in Relativisty.js of Mass Variation From the notes i created in 1st year on my own with the  formula and process keeping the process similar to the others format
Added the  Function name RelativeMASSCalculator in Calculator.js for whole calculation and finding result .
The calculation function was similar to the  time dilation so i take the reffrence from there as both has the same formula with slight change of mass and time

Checklist:

[x ] My code follows the style guidelines of this project.
[x ] I have performed a self-review of my own code.
[x ] 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.

Screenshots

Original
Updated

original screenshot

Sorry i did'nt make the branch name understandable like for example Mass Variation  instead i make it Dev

@shivvamm have u added the steps for the calculator?

@shivvamm add the steps for the calculator and then send the PR

@shivvamm add the steps for the calculator and then send the PR

Sir I already added the steps

@shivvamm reset button is not clearing the result field

@shivvamm reset button is not clearing the result field

Let me check it I did  created the reset as same as the others
😅

sir its working perfect on my local server and the function is also proper
https://user-images.githubusercontent.com/58168590/157642296-951d25f3-b82e-40a1-872b-cf50c6b3453b.mp4


Text and HTML versions of drafts have different boilerplates

Describe the issue
https://www.ietf.org/archive/id/draft-ietf-oauth-jwk-thumbprint-uri-03.txt (linked from https://datatracker.ietf.org/doc/draft-ietf-oauth-jwk-thumbprint-uri/) refers to the "Simplified BSD License" in its boilerplate.
The HTML version at https://www.ietf.org/archive/id/draft-ietf-oauth-jwk-thumbprint-uri-03.html (linked from the same datatracker page) correctly refers to the "Revised BSD License".
How did this happen?
Code of Conduct

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

Archived-At: https://mailarchive.ietf.org/arch/msg/oauth/UGtcPbhrxntez0CcBiKodiWKX1s
"we submitted both the .xml and .txt files"
The usual outdated xml2rfc at the submitter's end?
The warning against submitting superfluous .txt with .xml needs to be much stronger.

(And the archive link in the message I cited does not work, which is another longstanding annoyance.)

https://mailarchive.ietf.org/arch/msg/oauth/cIeFXccJuk4px3-Iv_wCrQ-s49s/

The usual outdated xml2rfc at the submitter's end?

I thought the datatracker was re-generating the text version if XML was submitted - @rjsparks?

We would have to look, but what I remember is that we don't throw away anything that was submitted - if someone submits both xml and txt, we keep both (think about it from an IPR perspective). I thought we had a path where we would generate txt and warn/error on differences, but maybe that was aspirational and not ever implemented.

Until the submission process lets submitters preview the rendered versions, people will insist that we take their text because of potential issues/differences in external reference resolution. It's getting better, but the community trust in the datatracker to not do the right thing with those references has been low.
We really need to solve the "submit a self-contained document" problem.

Until the submission process lets submitters preview the rendered versions,

You say it can’t?
See below.
OK, weirdly, it doesn’t allow previewing the whole plaintext (unless you guess that clicking on HTML, changing .html to .txt, gives you a plaintext view with incorrect text encoding metadata (*)), but in principle the feature is there.
Grüße, Carsten
(*) which you can then fix under View/Text encoding in Safari and probably everywhere else.

(See the broken text encoding at
https://www.ietf.org/staging/draft-bormann-asdf-sdf-compact-03.txt
$ curl -I https://www.ietf.org/staging/draft-bormann-asdf-sdf-compact-03.txt
...
content-type: text/plain
...
$ curl -I https://www.ietf.org/archive/id/draft-bormann-asdf-sdf-compact-02.txt
...
content-type: text/plain; charset=utf-8
...

I'm going to split this into several issues.
First, idnits needs to stop accepting the older incorrect license name: https://github.com/ietf-tools/idnits/issues/12
The staging area is a mess that needs concentrated attention, but I've sent a ticket to the secretariat to address the  webserver configuration issue leading to the encoding troubles.
The "View first two pages" functionality really needs rethought. Keeping the first two pages in the database has sped diagnostics in the past when things broke because the author extraction heuristics went into the weeds, for example, but text isn't the only format anymore, and sometimes the content of those first two pages is not benign.
And much of this will change when we take the step in #3577


Move Product Backlog information from Google Docs to Kanban

Our product backlog is looking much cleaner now! This looks great!


[bugfix] moment.locale() returns undefined after updating a non-existing local…

…e #5043

Coverage decreased (-0.01%) to 88.396% when pulling 6e063558d2e492ca5f28f4e2f37fe681ab624c77 on nishantgupta204:develop into d0f0dd8f40be987593767a3dc5e27c45c6a3fa49 on moment:develop.

Coverage decreased (-0.01%) to 88.396% when pulling 6e063558d2e492ca5f28f4e2f37fe681ab624c77 on nishantgupta204:develop into d0f0dd8f40be987593767a3dc5e27c45c6a3fa49 on moment:develop.

Coverage decreased (-0.01%) to 88.396% when pulling 6e063558d2e492ca5f28f4e2f37fe681ab624c77 on nishantgupta204:develop into d0f0dd8f40be987593767a3dc5e27c45c6a3fa49 on moment:develop.

Coverage decreased (-0.01%) to 88.396% when pulling 6e063558d2e492ca5f28f4e2f37fe681ab624c77 on nishantgupta204:develop into d0f0dd8f40be987593767a3dc5e27c45c6a3fa49 on moment:develop.

Coverage decreased (-0.01%) to 88.396% when pulling 6e063558d2e492ca5f28f4e2f37fe681ab624c77 on nishantgupta204:develop into d0f0dd8f40be987593767a3dc5e27c45c6a3fa49 on moment:develop.

@marwahaha - Can you please have a look at the PR. Thanks

@nishantgupta204 OK, it looks good overall, but I don't like the fact that it changes the existing behavior.
defineLocale prints a deprecation warning and then proceeds with updating the locale.

Close in favor of #5470
As explained there, the code handles properly the fact that the locale is not existent, it's just the name which is wrong (undefined).


Merge remote-tracking branch 'upstream/stable' into merge_stable

Conflicts:
std/internal/cstring.d
win64.mak

Now updated, had to fix an incompatibility with tempCString.
https://github.com/MartinNowak/phobos/commit/12119cf360d34883e081a34ca841ac4720a2ec55

Auto-merge toggled on


Fix #311, removed unnecessary check for CMAKE_BUILD_TYPE

The check for Release or Debug in CMAKE_BUILD_TYPE breaks some assumption in Debian packaging.
Since the check is useless, it has been removed.

Hi @studiofuga thanks for your contribution, could you resolve the merge conflicts? Also, I've seen that you changed the indentation / style of most of the CMake code. Which formatter are you using? We might want to add it as a contributor note.

Sure I opened a new PR. Also removed all changes due to spaces. I'm using clion's default formatter. I suggest to provide a .clang-format file to unify the code styles. Thanks.


Transpose of symbolic vector

The result of transposing a symbolic vector is a transpose object with element type Any instead of Sym:
julia> T = SymPy.Sym{PyCall.PyObject};

julia> x = T.([1, 2])
2-element Vector{Sym{PyCall.PyObject}}:
 1
 2

julia> transpose(x)
1×2 transpose(::Vector{Sym{PyCall.PyObject}}) with eltype Any:
 1  2

For a matrix, on the other hand, the correct result is obtained:
julia> A = T.([1  2; 3  4])
2×2 Matrix{Sym{PyCall.PyObject}}:
 1  2
 3  4

julia> transpose(A)
2×2 Matrix{Sym{PyCall.PyObject}}:
 1  3
 2  4

Thanks for the report!!

Thanks a lot for fixing this so quickly. In Julia >1.8 everything works, but there seems to be an issue still with v1.6:
  Element type mismatch. Tried to create a `Transpose{SymPyCore.Sym}` from an object with eltype `SymPyCore.Sym`, but the element type of the transpose of an object with eltype `SymPyCore.Sym` must be `Any`.
  Stacktrace:
    [1] error(s::String)
      @ Base ./error.jl:33
    [2] checkeltype_transpose(#unused#::Type{SymPyCore.Sym}, #unused#::Type{SymPyCore.Sym})
      @ LinearAlgebra /buildworker/worker/package_linux64/build/usr/share/julia/stdlib/v1.6/LinearAlgebra/src/adjtrans.jl:86
    [3] LinearAlgebra.Transpose{SymPyCore.Sym, Vector{SymPyCore.Sym}}(A::Vector{SymPyCore.Sym})
      @ LinearAlgebra /buildworker/worker/package_linux64/build/usr/share/julia/stdlib/v1.6/LinearAlgebra/src/adjtrans.jl:69
    [4] transpose(A::Vector{SymPyCore.Sym})
      @ SymPyCore ~/.julia/packages/SymPyCore/PayoX/src/matrix.jl:24

This comes from a call like transpose(x) in the example above.
I am not married to Julia v1.6. I just saw this popping up in the CI workflows and thought you might want to know.

Thanks! I had missed this


Fix some issues found with hardening the agent:

Make sure the group created in the Dockerfile is 5000 as intended
Give the logdna binary with the effective bit since it's not capability aware
Remove securityContext since we don't want to run as non-root in the default behavior
Add documentation about running as non-root

@logdnabot test this

@logdnabot test this


Using mapnik.Datasource with parameterized queries

For example I create a new layer with Datasource:
new mapnik.Datasource({
    type: 'postgis',
    ...
    table: '(SELECT geom, name FROM some_table WHERE category = ' .parseInt(user_input). ') AS layer'
});

I need to filter input values for preventing sql-injections (parseInt, ...);
Postgres have escaping functionality, so can I use some of this:
new mapnik.Datasource({
    type: 'postgis',
    ...
    table: '(SELECT geom, name FROM some_table WHERE category = $1) AS layer',
    params: [ user_input ]
});

Anybody?

This feels like something solvable via mapnik variables (experimental): https://github.com/mapnik/mapnik/issues/2553

Is this functionality supports sql injection filters?

@im4LF I don't understand your question. Please followup over at https://github.com/mapnik/mapnik/issues/2553


The prediction became poor after upgrading from 1.0 to 2.* for R

Environment info
C++/Python/R version: R
examples
The R version 1.0 works:
bst = lgb.train(params, dTrain, 150, list(train = dTrain,test = dTest))
the last row of the output is
[150]: train's rmse:0.0134871  test's rmse:0.015928
when calculating the test set error by hand:
pred = predict(bst, data_sparse[test_ind,]) # it takes sometime for large data, but the distribution looks good
sqrt(sum(dataRaw$weight[test_ind]*(dataRaw$y[test_ind] - pred)^2)/sum(dataRaw$y[test_ind]))
[1] 0.015928   # which matches the last row of model training output
summary(pred)
Min.        1st Qu.    Median      Mean   3rd Qu.      Max.
-0.001950  0.009145  0.015400  0.019230  0.024810  0.301900
The R version 2.* doesn't work
bst = lgb.train(params, dTrain, 150, list(train = dTrain,test = dTest))
the last row of the output is
[150]:    train's rmse:0.0133429  test's rmse:0.0156751
when calculating the test set error by hand:
pred = predict(bst, data_sparse[test_ind,])  # it is super fast even for large data, however, seems it tend to predict almost everything to be the same （Q1 = Q2 = Q3）
sqrt(sum(dataRaw$weight[test_ind]*(dataRaw$y[test_ind] - pred)^2)/sum(dataRaw$y[test_ind]))
[1] 0.0244318   # which DOSEN'T matches the last row of model training output
summary(pred)
Min.        1st Qu.    Median      Mean   3rd Qu.      Max.
-0.0006544  0.0060730  0.0060730  0.0076750  0.0060730  0.2611000

Hi I'm so sorry for the late reply. As I installed the latest version (lightgbm_2.1.0.tar.gz), the problem seems to have been gone... The versions I installed on around 7/15/2017 and 12/01/2017 had the problem I mentioned. Both times I just followed your installation guidance so the version should be the latest at those moments. Now my VM with these old versions of LightGBM is gone, so I'm not able to provide more info and I can't reproduce the case. Sorry about that... but thankfully it's good now!

@zhangsssss very happy to know the good news.


Interoperability issue with postcss-custom-properties

This is actually a cssnext issue but it's the interoperability between postcss-custom-properties and postcss-custom-media that fails. Not sure where the code change should be but posting this in all 3 repositories.
Using variables inside a media query doesn't get processed.
Test case: postcss -u postcss-cssnext var.css

MOBILE

DESKTOP

:root {
  --with-var: 640px;
}
.desktop { display: none; }
@media screen and (min-width: var(--with-var)) {
  .mobile {
    display: none;
  }
  .desktop { display: block; }
}

Should output:
.desktop { display: none; }
@media screen and (min-width: 640px) {
  .mobile {
    display: none;
  }
  .desktop { display: block; }
}

Instead it outputs
Warning {
  type: 'warning',
  text: 'Missing @custom-media definition for \'--with-var\'. The entire rule has been removed from the output.',
  line: 5,
  column: 1,
  node:
   AtRule {
     raws:
      { before: '\n',
        between: ' ',
        afterName: ' ',
        semicolon: false,
        after: '\n' },
     type: 'atrule',
     name: 'media',
     parent: undefined,
     source: { start: [Object], input: [Object], end: [Object] },
     params: 'screen and (min-width: varundefined)',
     nodes: [ [Object], [Object] ] },
  plugin: 'postcss-custom-media' } 0 [ Warning {
    type: 'warning',
    text: 'Missing @custom-media definition for \'--with-var\'. The entire rule has been removed from the output.',
    line: 5,
    column: 1,
    node:
     AtRule {
       raws: [Object],
       type: 'atrule',
       name: 'media',
       parent: undefined,
       source: [Object],
       params: 'screen and (min-width: varundefined)',
       nodes: [Object] },
    plugin: 'postcss-custom-media' } ]

Per spec it's not a bug.

point taken.  Do you know why the editors haven't chosen to merge custom-properties with custom custom-media-queries?

properties depend on the DOM tree, media don't (device).


Made bugfix regarding "initial lonely heading interpreted as title" c…

Fix from #124

Travis doesn't like this :boom:, and there are merge conflicts now. It looks like the extra parameter didn't get added to one of the renderers. There also seems to be a bunch of incomplete questions in the code in the form of a big comment block.

Unfortunately I missed those actions here. I'll try to catch up and fix my patch.

Pull request #152 should do it now.


Hello boy, why does the Darknet model have no jump layer addition?

the jump layer addition you refer to is the residual connect?? The forward function implement it.

yeal. i found it in the DarknetBlock ， thanks


add issue forms

add issue forms link

see this link for Converting a Markdown issue template to a YAML issue form template

see this link for Common validation errors when creating issue forms

 Issues from GitHub project.

 Issues from GitHub project.


Prepare release 2.2.5

Incorporates fix for #237 and function enhancements from #238
Note: can not upgrade spring-context above 5.3.24 to 6.0.2 due to Java version needing to stay at version 8 for several product dependencies.

Released version 2.2.5. Should appear in Maven Central in a couple of hours.
I also updated jackson-databind to 2.14.1 in the pom.xml


Invalid Redirect URI

When MonzoAPI() is called with keyword arguments (i.e. not using env variables) I get the following response:
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/local/lib/python2.7/dist-packages/pymonzo/monzo_api.py", line 76, in __init__
    self._token = self._get_oauth_token()
  File "/usr/local/lib/python2.7/dist-packages/pymonzo/monzo_api.py", line 155, in _get_oauth_token
    client_secret=self._client_secret,
  File "/usr/local/lib/python2.7/dist-packages/requests_oauthlib/oauth2_session.py", line 244, in fetch_token
    self._client.parse_request_body_response(r.text, scope=self.scope)
  File "/usr/local/lib/python2.7/dist-packages/oauthlib/oauth2/rfc6749/clients/base.py", line 411, in parse_request_body_response
    self.token = parse_token_response(body, scope=scope)
  File "/usr/local/lib/python2.7/dist-packages/oauthlib/oauth2/rfc6749/parameters.py", line 379, in parse_token_response
    validate_token_parameters(params)
  File "/usr/local/lib/python2.7/dist-packages/oauthlib/oauth2/rfc6749/parameters.py", line 386, in validate_token_parameters
    raise_from_error(params.get('error'), params)
  File "/usr/local/lib/python2.7/dist-packages/oauthlib/oauth2/rfc6749/errors.py", line 415, in raise_from_error
    raise cls(**kwargs)
oauthlib.oauth2.rfc6749.errors.InvalidClientIdError: (invalid_request) Invalid redirect URI

I'm assuming this is to do with how I have configured the client on the Monzo dev portal, but I can't see what's wrong with it?

Sorry, my bad, fixed this by making absolutely sure the redirect links matched in every part of the app and all the app logic worked... If anyone sees this and is having the same issue, make sure the links are the same in every place they are used. Copy-paste is your friend (duh, I'm an idiot...)


No found Default Schema box

No found Default Schema box in Steps 6 under Create a Windows Authentication login section in SSMS 15.0.18142.0.
Steps 6 In the Default Schema box, type dbo to map the login to the database owner schema.

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

ID: 3c35290b-b676-c52d-4204-ada11a3bb2af
Version Independent ID: 87264101-997a-d17a-c96a-7a2cf15689e1
Content: Lesson 1: Connecting to the Database Engine - SQL Server
Content Source: docs/relational-databases/lesson-1-connecting-to-the-database-engine.md
Product: sql
GitHub Login: @MashaMSFT
Microsoft Alias: mathoma

Hi @shaoyangcoder , Thank you for contacting Microsoft and providing your feedback to our documentation! We'll need some time to look into this further. Depending on the complexity, the turnaround time for this may vary anywhere from a few business days to a few business weeks. We'll reply via comment should we need anything further in the meantime, and please feel free to do the same.
If you're suggesting a simple change to the documentation, it may be faster for you to submit the edit yourself via a Pull Request (PR). You can do so by following these steps:

Navigate to the page of interest
Select the "Edit" button at the top right of the page
Select the "Pencil" icon on the right
Modify the text within the text box in markdown format
Name your file change and copy the link to your Git Issue into the "extended description" box
Select "Propose file change"
Select "Create pull request" on the 'Comparing changes' page
Select "Create pull request" on the 'Open a pull request' page
Wait for an author to review and approve the change request - this can take a bit depending on the complexity

Doing so will allow authors to directly add your text to the documentation. This also displays your github image on the page of authors, and list you as a contributor to the documentation.
Otherwise, we'll continue to research your suggestion, and get back to you as soon as we have more information for you.
Thanks again for your submission!
Masha from SQL Docs team

Hi @shaoyangcoder , thank you again for contacting Microsoft! My apologies, would you mind clarifying what you don't see?
When I follow the instructions and navigate to the User Mapping page, I see the default schema box on the right hand side, next to the database names, and there I am able to type dbo. Are you seeing something different?

Thanks!
Masha from the SQL Docs team


Add external collaborator

Which MoJ repository/s are you requesting access to (e.g. money-to-prisoners)?
'lcjb-explorer'
https://github.com/ministryofjustice/lcjb-explorer
What is the GitHub username of the person who needs access?
mihaialdoiu
What is their full name?
Mihail-Valentin Aldoiu
What is their email address?
<EMAIL_ADDRESS>What company/organisation do they belong to (e.g. "Department for Education")?
Faculty Science Ltd
What level of access should they have?

[x] Read
[ ] Triage
[x] Write
[ ] Maintain
[ ] Admin

Why is this access being requested?
Joining the C529.1 project between MOJ, HO and Faculty
Which MoJ team/person is responsible for this request? This must be a ".justice.gov.uk" or ".digital.justice.gov.uk" email address.
<EMAIL_ADDRESS>Team email addresses are preferred, rather than individual email addresses, to make it easier to follow-up at a later date.
<EMAIL_ADDRESS>Review date
Date after which this collaborator's access should be reviewed. This should be a date in the format: 2021-01-15, no more than one year into the future from today.
2024-07-26

Closing, I need a different GH account for this.


make error

Issue summary
virtual memory exhausted:  Cannot allocate memory
gninasrc/CMakeFiles/gnina.dir/build.make:202: recipe for target 'gninasrc/CMakeFiles/gnina.dir/lib/cnn_data.cpp.o' failed
make[2]: *** [gninasrc/CMakeFiles/gnina.dir/lib/cnn_data.cpp.o] Error 1
CMakeFiles/Makefile2:1888: recipe for target 'gninasrc/CMakeFiles/gnina.dir/all' failed
make[1]: *** [gninasrc/CMakeFiles/gnina.dir/all] Error 2
Makefile:159: recipe for target 'all' failed
make: *** [all] Error 2
Steps to reproduce
I tried these commands, but it didn't work.
mkdir /opt/images/
rm -rf /opt/images/swap
dd if=/dev/zero of=/opt/images/swap bs=1024 count=2048000
mkswap /opt/images/swap
swapon /opt/images/swap
Your system configuration
Operating system:
Compiler:
CUDA version (if applicable):
CUDNN version (if applicable):
Python version:

You don't have enough memory to compile.  This is not an issue with the code base, but with your environment.

You don't have enough memory to compile. This is not an issue with the code base, but with your environment.

thanks a lot.

I'm sorry, but you need 8GB of RAM to compile the CNN models.  If you don't have enough memory to compile, you aren't going to have enough RAM to use the CNN models either.


Add a deploy test for azure saml ses

Description
Adds a github action that will execute a deploy to staging which just 'plans' the terraform. this should do everthing outside of actually deploying. this should be reasonably fast and cost less than deploying. we can ideally run this on changes to the cloud directory to confirm if we are doing something to the deploy system.
Checklist

[ ] Add support for this to do setup
[ ] Setup this as part of the tests framework
[ ] confirm this fails when it fails
[x] add the correct arm info

Issue(s)
Fixes #<issue_number>; Fixes #<issue_number>; Fixes #<issue_number>...

note we might want to switch the secrets to do a new service principal here


bypass request timeout when using BulkRequests

in benchmarks

ack

I think we should still have a timeout, even if it is very large in this case.


Add support against linkage with C++ programs

Thanks. I have instead added the 'extern "C"' to include/fvad.h only, the only header file that should be included from C++ programs that use the library.
The other header files are internal headers that shouldn't require 'extern "C"' because they are intended to be included only from the library's own C source files, which should be compiled with the compiler in C mode.

Makes sense. Thank you


Multi-Agent (Players)

Hi,
Thank you so much for publishing this implementation of MCTS. Using it has helped me greatly in learning how MCTS works and experimenting 👍
Do you have any examples or suggestions for how to handle a multi-player environment? The only example in the repo is a simple counting game, which is a single player. I see that there are player methods, but the example does not show how to use it.
Also, are there any considerations for whether the player moves are simultaneous vs serial per turn? In the example I'm working on, there may be 2 to 8 players, one of whom is the player I'm trying to maximize, and the others are opponents I'm trying to minimize, and all players always make one of their available moves simultaneously once per turn. In other words, my player does not know which of the moves each other player will make, until the next child node has been simulated (assuming I choose randomly what the other player is likely to do).
Thanks in advance!

Hi,
You should implement current_player in GameState and interpret_evaluation_for_player in Evaluator. If you have specific questions about how to do this, feel free to ask.
The code can't easily accommodate imperfect information. I'm not too familiar with the literature so I'm not sure whether MCTS is typically used for imperfect information games.

Thank you! So if I understand correctly, I would need current_player to return whose turn it is given a state, and interpret_evaluation_for_player would (for example) negate the evaluation score if it was the opponent being interpreted?

Yep. It's written in this generic way to support >= 3 players.

For a game like Chess or Go, each player makes a move in turn, so this feature makes sense in that case. For the use case I'm working on, all players report their chosen move each tick (they make a turn simultaneously. The way I'm handling that as imperfect information right now is to just randomly pick a move for each opponent during make_move after applying my own move. This has some drawbacks, but in general it kind of works, because MCTS evens out once it has enough samples.
The problem is that random moves are never what the opponent will actually do, so the evaluation misses a lot of more likely outcomes. What I'd like to do is incorporate available opponent moves into the search tree instead of just picking randomly for them, so that interpret_evaluation_for_player can then minimize scores for good moves for an opponent and maximize good moves for me. But I'm not sure if the current_player workflow works for this, because the moves happen simultaneously. The state does not have a "current player", the move being made in a turn of a playout is always my move, plus >= 1 unknown opponent move.
If I use current_player anyway, and just use my state to advance turns as if they are one by one instead of simultaneous, would that work, and what should I do in make_move? Will make_move run once per current player? And if it does, then what callback should I use to finish updating the state after all players have made their move?
Thanks again for your help :)

The sole purpose of current_player is to pass to interpret_evaluation_for_player.
It sounds like you want to model it as a single-player game. This could work, but it's not very principled. A more principled way to handle simultaneous moves is to use an algorithm that converges to a Nash equilibrium.
In general, this library assumes that at each node of the tree there is a best move, which isn't true when there's imperfect information.


[apps-service] Update service-groups table

:nut_and_bolt: Description
Update the service-groups table inside the Applications Tab. Match the mockups.
Table Column in Mockup

Current state

:+1: Definition of Done

Change column header "Services" to "Service Group"
Merge the value of "Name" column to "Service Group" column, putting it on top of services counts
Remove the "Name" column, apply the same sorting logic for "Name" to "Service Group" column.
Make the services counts "xx of xx" font style bold
Split the value of "Release" column to "Package" and "Release": "Package" value includes the "origin/package_name", and "Release" value includes the "version/build_stamp".
Change the alignment of the row content (including the donut chart and the health status icon) to the top, instead of central vertically.

:athletic_shoe: Demo Script / Repro Steps
Build and start the UI. Populate the application's database and navigate to the Applications Tab.
New Table Style
Normal View

Large/Expanded View

Small/Contracted View

:chains: Related Resources
Jira: https://chefio.atlassian.net/browse/A2-851
Closes: https://github.com/chef/automate/issues/469
:white_check_mark: Checklist

[ ] Necessary tests added/updated?
[ ] Necessary docs added/updated?
[x] Code actually executed?
[x] Vetting performed (unit tests, lint, etc.)?

So I noticed in the mockup, the wrapping that we have when the package overflows the box is like:
the-origin/
long-package-name

But the behavior shown in the pictures is like:
the-origin/long-
package-name

I did a little research and I think there's no way to do what we want in CSS. There are some options to customize hyphenation, but the character at which the break occurs is determined by a language setting (i.e., English or whatever) on the content. I suppose it could be fixed in javascript but I don't really know. Do we care enough to look into this?

Summarizing some discussion from Chef internal slack, we think breaking on / makes it more readable but we are okay with how it is now. There is a zero-width space unicode character, and if you put it after the /, then the package name does break where we want it to. BUT, that isn't an acceptable solution because the zero-width space gets included in copy/paste operations, which would be annoying if users want to copy information from the table for use in hab pkg promote or anything like that.


Drop image list from input pickle files, and not only from args.json files

We hit the same issue as #1237, but for input pickle files.
We cleared the quadratic scaling of args-json-file sizes with the number of components, but this same scaling happens for input pickle files (which are actually used to extract and write the args.json file, in the case of parallel tasks). These are the files with names like {task_order}_par_{component})_in_{gibberish}.pickle, and in one relevant case they take 5M each.
In a normal use they are always removed after the task is over, but when something fails then it's possible that they remain in the job folder. In principle we could improve this clean-up phase, but the first important update is that we should move the fix in #1238 upwards one level in the stack trace, and strip the image list from the dataset metadata before building the pickle files.

For the record, the place where this image list appears is in
task_pars = TaskParameters(
    input_paths=[PosixPath('...')],
    output_path=PosixPath('...'),
    metadata={...}  # <---- here
...


503 qualitygradestatus refactor

Not sure why tests didn't run, but they should once @angielt resolves the conflict here.


feat(cloudid): notify clouduser created

这个 PR 实现什么功能/修复什么问题:

[x] 功能、bugfix描述
[x] 冒烟测试
[ ] 单元测试编写

是否需要 backport 到之前的 release 分支:

release/3.5

/area cloudid
/cc @zexi

Codecov Report

Merging #9135 (62c83f6) into master (0eb08f1) will decrease coverage by 1.62%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #9135       +/-   ##
==========================================
- Coverage    5.84%   4.22%    -1.63%
==========================================
  Files         836     169      -667
  Lines      129960   25005   -104955
==========================================
- Hits         7600    1056     -6544
+ Misses     121678   23853    -97825
+ Partials      682      96      -586

Flag
Coverage Δ

aFlag
4.22% <ø> (-1.63%)
:arrow_down:

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

Impacted Files
Coverage Δ

pkg/hostman/diskutils/nbd/nbdman.go

pkg/baremetal/tasks/baseprepare.go

pkg/mcclient/options/k8s/fed_namespace.go

pkg/mcclient/options/k8s/deployment.go

pkg/keystone/models/federated_user.go

pkg/keystone/tokens/policy.go

pkg/compute/models/elasticcache_instances.go

pkg/compute/models/storagedrivers.go

pkg/multicloud/ucloud/network.go

pkg/compute/models/networks_id_change_handlers.go

... and 656 more

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 0eb08f1...62c83f6. Read the comment docs.

/lgtm
/approve


How to make a curve to a rigid body.

I draw a curve using Bodies.fromVertices but the curve not close, how to make it to a rigid body?

You need to load polydecomp to use concave vertices, maybe that is the problem. Otherwise you will need to explain more and give code examples.

// terrain.svg path
......
<path
       d="M34,89 C223,363 226,2 339,199"
       id="path3780" style="stroke-width: 5px;" />
......

 $.get('./terrain.svg').done(function(data) {
                var vertexSets = [];

                $(data).find('path').each(function(i, path) {
                    vertexSets.push(Svg.pathToVertices(path, 10));
                });

               // insert some points ###
                vertexSets[0].slice(0).reverse().forEach(function(item) {
                    vertexSets[0].push({
                        x: item.x,
                        y: item.y - 2
                    });
                });

                terrain = Bodies.fromVertices(400, 350, vertexSets, {
                    isStatic: true,
                    render: {
                        fillStyle: '#2e2b44',
                        strokeStyle: '#2e2b44',
                        lineWidth: 10
                    }
                }, false);

......

I insert some points to svg path to make it become a rigid body. I want to know it is right way?

The result pictured looks fine to me, is there a problem with it?

I wondered if there was a better way to draw a curve rigid body.

Nope this is the recommended way, you could also try using chamfer for certain types of curved bodies.


Filtering in Search Tree

From https://github.com/microsoft/vscode/pull/161022#issuecomment-1255876319

The tree view also miss the in-tree filtering (CTRL+F) like the one for files:

https://github.com/microsoft/vscode/issues/67397

/duplicate https://github.com/microsoft/vscode/issues/67397


pull only config when we don't need whole image

when looking to obtain the volume info for a docker image, we only need the config, but we were pulling the whole image. For large images, that is a big waste. In any case, there was no need to involve docker anyways.
This just pulls the config itself and parses it.
Thanks to @giggsoff for explaining the requirements.

Thanks for the edits @deitch, great opportunity to save space and time!


support get total cache size

if possible, can you help to support get cache size?

@YouHan26 Hey, this does not come out of the box with react-native-fetch-blob so I will have to figure out an other way to do it.
For now I'm thinking about traversing all files under the base cache dir and collecting their file sizes, however this seems a bit too wasteful to do over the RN bridge.
Any suggestions are welcome here :)

@kfiroo yeah, I have used traversing all files to get total size. It looks not so slow. I can accept it. Need I commit a pr?
Thanks a lot for your work of cached-image component, It help me a lot. :)

@YouHan26 I would prefer to have a single call to the native code to retrieve this info, I would look into react-native-fetch-blob fs code to see if it's possible.
Glad you find it helpful! :)

@YouHan26 Hey, master has a version with a working getCacheInfo that returns the following format
{
  files: [// an array of all files in the cache represented by their `fs.stat()` result],
  size: 0 // size of cache in bytes (see example)
}

Hope that helps

@kfiroo it works. Thanks

@YouHan26 How did you then delete those images from the cache seeing as you don't have the pre-hash filenames?

Ah. I think I see it.. there are other function in the ImageCacheProvider that I can probably use. I'll try those directly.


[FEATURE REQUEST] Allow setting packing=True for training but =False for evaluation in the same trainer

I would like to train on a very large dataset, but eval on a very small dataset
I need to set packing=True with the SFT trainer because of the amazing efficiencies it brings.
However, this means that I get an ValueError: Error occurred while packing the dataset. Make sure that your dataset has enough samples to at least yield one packed sequence. error.
This is due to the fact that the eval dataset is very small.
I would like to be able to set packing=True for the training dataset and eval_packing=False for the evaluation set.
This feature is implemented by Axolotl here, so it should be technically possible.

Would TRL consider implementing such a feature?
Thanks!

Hi @Peter-Devine
Absolutely yes! I'll work on it!

@younesbelkada Not all heroes wear capes - thanks!

Hi @Peter-Devine
#1369 should enable this!

@younesbelkada
Hey, sorry, just got round to checking this. First, thanks for the quick fix!
Running it, it seems like it doesn't log the loss of each (or any) eval dataset I have, but only logs the runtime, samples per second, and steps per second.

Here is the code I used to train:
from dataclasses import dataclass, field

import torch
from datasets import load_dataset, load_from_disk
from tqdm import tqdm
from transformers import TrainingArguments
from trl import SFTTrainer
import os

from unsloth import FastLanguageModel

tqdm.pandas()

# ###############
# Model & Tokenizer
# ###############

model, tokenizer = FastLanguageModel.from_pretrained(
    model_name = model_name_or_path,
    max_seq_length = max_seq_length,
    dtype = dtype,
    load_in_4bit = load_in_4bit,
)
tokenizer.pad_token = tokenizer.eos_token

model = FastLanguageModel.get_peft_model(
    model,
    r = lora_r, # Choose any number > 0 ! Suggested 8, 16, 32, 64, 128
    target_modules = ["q_proj", "k_proj", "v_proj", "o_proj",
                      "gate_proj", "up_proj", "down_proj",],
    lora_alpha = lora_alpha,
    lora_dropout = lora_dropout, # Supports any, but = 0 is optimized
    bias = lora_bias,    # Supports any, but = "none" is optimized
    use_gradient_checkpointing = use_gradient_checkpointing,
    random_state = 123,
    use_rslora = use_rslora,  # We support rank stabilized LoRA
    loftq_config = loftq_config, # And LoftQ
)

################
# Dataset
################
train_dataset = load_from_disk(f"train_dataset/{train_dataset_name}")
eval_dataset_folder = "eval_dataset"
eval_dataset = {eval_name: load_from_disk(os.path.join(eval_dataset_folder, eval_name)) for eval_name in os.listdir(eval_dataset_folder) if eval_name != "megagon"}

################
# Training
################
# del model_kwargs["attn_implementation"]

training_args = TrainingArguments(
        per_device_train_batch_size = per_device_train_batch_size,
        per_device_eval_batch_size = per_device_eval_batch_size,
        gradient_accumulation_steps = gradient_accumulation_steps,
        warmup_steps = warmup_steps,
        eval_steps=eval_steps,
        evaluation_strategy=evaluation_strategy,
        save_steps=save_steps,
        max_steps = max_steps,
        learning_rate = learning_rate,
        fp16 = not torch.cuda.is_bf16_supported(),
        bf16 = torch.cuda.is_bf16_supported(),
        neftune_noise_alpha=neftune_noise_alpha,
        logging_steps = logging_steps,
        optim = optim,
        weight_decay = weight_decay,
        lr_scheduler_type = lr_scheduler_type,
        seed = seed,
        output_dir = output_dir,
        logging_dir=logging_dir,
        run_name=run_name,
)

trainer = SFTTrainer(
    model = model,
    train_dataset = train_dataset,
    eval_dataset=eval_dataset,
    dataset_text_field = dataset_text_field,
    max_seq_length = max_seq_length,
    dataset_num_proc = 4,
    packing = True,
    eval_packing = False,
    args = training_args
)

trainer.evaluate()
trainer.train()
trainer.save_model(training_args.output_dir)

with the following variables
run_name = train_dataset_name
dataset_text_field = "text"
model_name_or_path=model_name
report_to="all"
learning_rate=2e-4
max_seq_length = 4096
per_device_train_batch_size=1
gradient_accumulation_steps=1
per_device_eval_batch_size=1
output_dir = training_output_folder + "/" + train_dataset_name + "_train_output"
logging_dir = tensorboard_output_folder + "/" + train_dataset_name + "_logging_output"
logging_steps=1
max_steps=66
eval_steps=22
evaluation_strategy = "steps"
save_steps=22
lora_r=64
lora_alpha=16
neftune_noise_alpha=5
attn_implementation=None
dtype = None
load_in_4bit = True
lora_dropout = 0
lora_bias="none"
use_gradient_checkpointing=True
use_rslora = False
loftq_config = None
warmup_steps = 0
weight_decay = 0.01
optim = "adamw_8bit"
lr_scheduler_type = "linear"
seed = 123

Hi @Peter-Devine
Hmm that's strange, do you face the same issue without eval_packing=False?

Hey @younesbelkada , I don't get this particular bug when i remove eval_packing=False  and just repeat the eval dataset n times until it fits into one packing batch. It nicely reports the loss when I do that.


Enable constructor parameters in message classes

This enables support for message classes with constructor parameters which match class properties.  This is in response to issue #100 (which admittedly I am the one who filed).
In order to preserve backward compatibility, this option is only enabled if the message class is annotated with the @ApiMessage annotation, and by setting its new "withConstructorParameters" to true.
This is a feature I would like very much.  If it should be implemented in some other way to keep with the style and conventions of the library, I am happy to make changes.

CC @mkustermann


[Feat]: Sign in with validation

What feature?
I have an idea we should need to add sign in form with validation please give me that opportunity because i am perfectly fit for that role as i know react and we add firebase for authentication. Gssoc extd
Add screenshots
Add screenshots
Code of Conduct

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

📢 Invitation to Join the DevDisplay WhatsApp Community!
We are excited to invite you to join the WhatsApp community of DevDisplay, where developers and contributors come together to collaborate and share their ideas.
Collaborative Support: If you are interested in working together with extensive support, we encourage you to join our community.
Open Source Contribution: If you would like to contribute to the DevDisplay open-source project, your participation is welcomed in our WhatsApp group.
By joining, you will have the opportunity to connect with like-minded individuals, discuss project developments, and contribute to our shared goals.
For more information and to join the community, please visit: Join DevDisplay WhatsApp Community.
We look forward to your participation!


Domain activation is Failed - [CNAME already in use]

App Id: d39uf4dnz7rsy0
Region: eu-west-1
Step: Domain activation
Status: Failed
Issue/question
Hi, we already were in touch with your customer support about the CNAME issue. amplify thinks that there is another app running under the CNAME we want to declare but according to route53 there is not. Also there is no app in amplify with the app-id you gave us. When we add a custom domain to amplify, it creates the CNAME link, which gets visible in route53 but then fails with the error message that it is already in use.
Error message
Error message: One or more of the CNAMEs you provided are already associated with a different resource.
Additional information
We also use Cloudfront if that is of importance. Also the customer support issue case is<PHONE_NUMBER>.

Refer to our documentation to troubleshoot domain setup.
https://docs.aws.amazon.com/amplify/latest/userguide/custom-domain-troubleshoot-guide.html
https://docs.aws.amazon.com/amplify/latest/userguide/custom-domain-troubleshoot-guide.html#i-get-a-cnamealreadyexistsexception-error
Let us know if you're still facing issues.

Refer to our documentation to troubleshoot domain setup.
https://docs.aws.amazon.com/amplify/latest/userguide/custom-domain-troubleshoot-guide.html
https://docs.aws.amazon.com/amplify/latest/userguide/custom-domain-troubleshoot-guide.html#i-get-a-cnamealreadyexistsexception-error
Let us know if you're still facing issues.


Feature: composite build / includBuild() : allow passing project properties

TL;DR a functionality like this:
includeBuild("some-other-project") {
  projectProperty("my-property", "some-value")
}

so that some-other-project's project properties can be configured.
Analogous to ./gradlew --project-prop my-property=some-value

I would like something like this or atleast the ability for the composite build to know that it is being included in another project.

I would like something like this or atleast the ability for the composite build to know that it is being included in another project.

It is easy to achieve your goal by checking if gradle.parent is null. See here, https://github.com/Omico/androidx-compose-material3-pullrefresh/blob/cd03216a001745aa85e7334a2f5a534b2e5d5375/settings.gradle.kts#L36

Sorry for the late reply.
This feature request is in the backlog of the relevant team and is prioritized by them.

Relatest to:

https://github.com/gradle/gradle/issues/29700
https://github.com/gradle/gradle/issues/25123


playerstate: introduce weaponCharge, fix #336

playerstate: introduce weaponCharge, fix #336
Required by https://github.com/Unvanquished/Unvanquished/pull/1188
Unfortunately includes https://github.com/DaemonEngine/Daemon/pull/330 because of the Unvanquished PR including commits also requiring it (it's hard to split the commits on the other side).

@Amanieu do you know how much this is “breaking change”?

Yes this is a breaking change to the network protocol.

But no more breaking than changing the syscall interface between the engine and a gamelogic module.

In general I'm OK to break things if that reduces breakage in the future like:

implementing a very common feature the right way so there is high change people needing this feature too would be happy to discover that the engine already supports it end they don't need to break it themselves.
redesign some stuff in a way newer changes can be done without breaking things.

Basically for this PR I had the choice to implement buildtimer or weaponcharge in playerstate, but buildtimer is unvanquished-specific (game-specific) while weaponcharge is fps-specific (genre-specific), that's why I did weaponcharge.
I sometime think about the fact maybe some parts of the engine may be doable in a nexe. Those would not be updatable and never read from dpk archive, but a game developer would be able to tweak some things on the engine side without having to taking care of the operating system or the achitecture.

If I'm right, this was obsoleted by #341.


Getting "Not Found" when trying to log in

doesn't really do anything. just getting "not found" when i try to log in. running on a headless machine on my lan.
this is what my lnd.conf looks like atm:
`[Application Options]
lnd.debuglevel=trace
lnd.maxpendingchannels=10
lnd.rpclisten=localhost:10009
lnd.restlisten=<IP_ADDRESS>:443
[neutrino]
lnd.neutrino.feeurl=https://nodes.lightning.computer/fees/v1/btc-fee-estimates.json
[autopilot]
lnd.autopilot.active=false
[watchtower]
lnd.watchtower.active=true
lnd.watchtower.listen=<IP_ADDRESS>:9911`
-httpslisten set to <IP_ADDRESS>:443 and --uipassword is a 30 character random alphanumeric password

What URL are you trying to log into? It should be https://10.10.1.x:443 in your case. Do you see anything in the log?

Also note that if you want to set ``httpslistento port 443 then you should not setlnd.restlisten=<IP_ADDRESS>:443`. The LND REST API cannot listen on the same port as LiT's https server.

I had the same issue ... solved by creating a wallet via lncli create

Closing due to inactivity. @richieledude please feel free to respond if you would still like to troubleshoot this issue.


Optimizing variables in arguments of built-in functions

There are built-in functions that have optional arguments. Unfortunately, when such arguments are omitted, unnecessary temporary variables are created in their place. This makes debugging the program more difficult due to the increased number of useless variables. It would be nice if such variables were replaced with zero at least on advanced optimization.
Example:
found1 = ulocate(building, core, 1, outx, outy, building);
found2 = ulocate(building, core, 1, outx, outy);
found3 = ulocate(building, core, 1, outx);
found4 = ulocate(building, core, 1);
ulocate(building, core, 1, outx, outy, building);
print(found1, found2, found3, found4, building);

Compiled result:
ulocate building core 1 @copper outx outy found1 building
ulocate building core 1 @copper outx outy found2 __tmp2
ulocate building core 1 @copper outx __tmp4 found3 __tmp5
ulocate building core 1 @copper __tmp7 __tmp8 found4 __tmp9
ulocate building core 1 @copper outx outy __tmp10 building
print found1
print found2
print found3
print found4
print building

Desired result:
ulocate building core 1 @copper outx outy found1 building
ulocate building core 1 @copper outx outy found2 0
ulocate building core 1 @copper outx 0 found3 0
ulocate building core 1 @copper 0 0 found4 0
ulocate building core 1 @copper outx outy 0 building
print found1
print found2
print found3
print found4
print building

You're right, it is better to avoid creating these unnecessary variables. The replace will be done by the Temporary Variables Elimination optimization so that it could be switched off. I don't expect any problems with this, though, Mindustry Logic clearly doesn't mind using zero as an output variable.
I considered using null for these unused outputs, but 0 is both shorter and fits better into the "mlog style".
The fix will be in the next release. Until that time, you can use the new #set sort-variables;, it makes inspecting them much much easier in the processor.


Hawkular 654

The triggers list portion of 654.

Not yet forwarding to a details page.

LGTM for first iteration. I agree with @lucasponce  comment above about detail link.

Merged #592


v2.0.0

major improvements to the http and mqtt transports
includes the beta threat hunting actuator profile for message validation
streamlined startup
introduction of configurable features via the config.toml
basic kestrel integration examples
updated readme documentation with clearer instructions, examples and OpenC2 goals

merged request approved


Evaluate current runtime approach of building for select TFMs

According to https://github.com/dotnet/source-build/pull/2178#discussion_r638340576 we should now be able to build for all configurations in runtime instead of picking and choosing working ones.
See also #2232.

[Triage] Removal of net4.* tfms is tracked with #3014.  With the Unified Build work in .NET 8.0 project with build with the latest tfm or a net standard tfm.


Fix wrong force keyword.

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew tests with your changes locally?
[x] Have you successfully run brew man locally and committed any changes?

Critical to unblock https://github.com/Homebrew/homebrew-cask/pull/94046.

Review period skipped due to critical label.


Minor release v48.2.0

[ ] The purpose of this PR is explained in this or a referenced issue.
[ ] The PR does not update generated files.

These files are managed by the codegen framework at Azure/autorest.go.

[ ] Tests are included and/or updated for code changes.
[ ] Updates to CHANGELOG.md are included.
[ ] Apache v2 license headers are included in each file.

attestation package path is problematic, temporarily closed


Wrong update item in IOS

https://github.com/marazmone/ios-compose-showcase/assets/7988862/be36fcaf-7742-4909-b5e1-1966ddb09640

It is necessary to include a key for the items in the LazyColumn. This particular situation is intriguing because, in Android, this step is not required for the method to function properly.


Fixing CUDA compiler errors

The current CUDA builds are broken. This is an attempt to fix this...

@sithhell should we add -Wno-noexcept-type to take care of the errors pycicle is reporting? They are benign as far as I can tell, as long as one doesn't mix C++11/14 and 17 code...

@sithhell should we add -Wno-noexcept-type to take care of the errors pycicle is reporting? They are benign as far as I can tell, as long as one doesn't mix C++11/14 and 17 code...

or we remove the 'throw ()' functions?

Also fine.

@sithhell should we add -Wno-noexcept-type to take care of the errors pycicle is reporting? They are benign as far as I can tell, as long as one doesn't mix C++11/14 and 17 code...

or we remove the 'throw ()' functions?

Yes, we should be able to safely replace those with noexcept nowadays. I'd suggest to solve this in a separate PR, however (see #3778).

@msimberg, @sithhell: I addressed the review comments, let's get this merged asap to get the CUDA builders under control.

I have manually merged this as GitHub refused to merge it through the web interface. Closing...


Error on "is type" statement, but looks legal

Got an error trying to constrain ImmunizationGiven.Outcome to ref(AdverseReaction) as follows:
Element:		Outcome
Value:		ref(Finding)
EntryElement:	AdverseReaction
Based on:	Finding
EntryElement:	ImmunizationGiven
Based on:	ImmunizationAction
Outcome is type ref(AdverseReaction)  // field inherited from ImmunizationAction
Resulting error message:
ERROR shr: Cannot constrain type of IdentifiableValue<shr.base.Outcome> to shr.adverse.AdverseReaction. ERROR_CODE:12014 (module=shr-expand, shrId=shr.immunization.ImmunizationGiven)
Looks like this is legal construct, isn't it?

I don't think this is legal.  AdverseReaction is based on Finding, but neither AdverseReaction nor Finding are based on Outcome.  Outcome's value is Finding, but that doesn't mean you can cast a Finding to an Outcome.  Finding would have to be based on Outcome for that to work.

The statement should have been "Outcome value is type ref(AdverseReaction)".
My mistake. No bug here.


Update tools.yml

Have added my product to the tools list: COFIQ editor
https://github.com/adraab/openapi.tools/commit/585179f733fa2ed7365c7bcd1e559631dd8e683f

my first github experience, so please be patient with me!


Resolved compiler warnings for incorrect @param in function documenta…

sorry,just test

Current coverage is 74.55%

Merging #3393 into master will decrease coverage by -1.72% as of a7127ac

@@            master   #3393   diff @@
======================================
  Files            5       5
  Stmts         2491    3002   +511
  Branches         0       0
  Methods
======================================
+ Hit           1900    2238   +338
  Partial          0       0
- Missed         591     764   +173

Review entire Coverage Diff as of a7127ac

Powered by Codecov. Updated on successful CI builds.


no video in IOS simulator

In the IOS Simulator, It plays the audio but not the video is this normal ?
In android si it works fine.
So, I tried overriding in the pubspec but no difference.
dependency_overrides:
  video_player: ^0.6.4

Its a known issue with IOS sim :)
so closing.


Documentation - Initial

fixes #40

Current coverage is 13.58%

Merging #41 into master will not affect coverage as of 96fe9c8

@@            master     #41   diff @@
======================================
  Files            2       2
  Stmts           81      81
  Branches         0       0
  Methods         12      12
======================================
  Hit             11      11
  Partial          0       0
  Missed          70      70

Review entire Coverage Diff as of 96fe9c8

Powered by Codecov. Updated on successful CI builds.

fixes #3


Shift Page when Aside is opened as option.

"aside-open" should be added to  when the aside is opened so that it can be used to shift the content when the aside is opened (force over by the aside).  It would be awesome if that was an option on the $aside.open() method call.

There are template and templateUrl properties if you are talking about setting content of the aside. Sorry, but it's not clear.

what i am meaning is,
When you open the menu, it should shift the current view over.  So if the menu is 300px, the original page from before the aside is opened, should shift over 300 px and then the aside will show, giving the appearance that the aside "PUSH" the page over.

also i noticed that github processed <body> in my original post, that probably made it a bit more confusing.  lol

You should be doing this viewport positioning in your app by conditionally applying css rules with aside state.


Guardian mode support

Hi,
Is the abililty to support 'Guardian mode' on the roadmap at all? It would be great to be able to utilise the feature in a HA automation to automate when to track window/door events.

Never heard about this feature. Can you please explain? I can't see anything like it in the official Jaguar mobile app neither.

Never heard about this feature. Can you please explain? I can't see anything like it in the official Jaguar mobile app neither.

Sure thing! It’s a security feature that can be activated in the JLR remote app. When activated, it will alert you to any interaction with your vehicle such as doors opening etc. realistically, you’d want to enable it when your away from your vehicle or your going to asleep so your alerted immediately if someone is trying to gain access to your vehicle.
it’s mentioned here - https://www.landrover.co.uk/ownership/incontrol/touch/set-up-and-user-guides/using-your-emergency-and-security-features.html
I’ve also attached a photo of where it appears in the app.

Interesting, it does not appear in my app. Might be specific to car model or country?

It seems that the jlrpy supports the guardian mode, so it might be that we could include it in our integration as well.
Any thoughts, @msp1974?

Documentation of the feature here: https://documenter.getpostman.com/view/6250319/RznBMzqo#c18c47d3-606d-4e0c-97d0-75c5cb53ea46

Guardian mode support is in v3.0.0beta1 with both an on/off switch to set it for 24hrs (default but can be changed in config options) or via service call passing date/time to set it to.

Hey @msp1974
That’s cool. is the beta available now or something that’s coming shortly?

Available  now.  You will need to enable beta versions in HACs to see it.

Thanks @msp1974 I’ve got that installed now and can see Guardian mode, much appreciated.
However I’ve noticed when I go to toggle the switch to on I get the bellow error message.

Can you tell me if it works using the service call via dev tools?  Thx

Hi Mark,
yes, that works.
It’s just when you toggle Guardian Mode under Entities.


Is there a way to change color of placeholder and increase the height of the box

I have implemented multi-select-lite in my project is there a way to change the color of the placeholder and increase the width of primary box?

Add extra CSS to change placeholder color using this selector .msl-input:before
What do you mean by the primary box?


Lab05, Part II, #2

I am working on this question:
Accidents by Hour & Age
Create a new table of time of day and age group that reports the proportion of accidents at "7 AM" (hour12) for each age group (age)
I can follow the instructions and produce the table, but I am getting different values for "n" and for "n.hour" than what is shown in the example table (the "n.age" agrees). Is this correct or do I have something wrong here?
dat %>%
  count(age, hour12) %>%
  group_by(age) %>%
  mutate(n.age = sum(n)) %>%
  group_by(hour12) %>%
  mutate(n.hour = sum(n))%>%
  filter( hour12 == "7 AM")

Table returns as:

The preview table in the template and the instructions show:

I just saw the response in a separate issue thread, that the data agrees when using  ( hour ) instead of ( hour12 )


Belinda_Android_Sprint_2_xx_individual_product_page_and_delete

It may say this branch is behind main, but I've updated it to include all the updates from Muhammad's delete-add-product-button branch and the latest update to main.
Adds an individual product page and deletion as Taylor described on Slack.
Here's a video demoing my changes

Looks great but let's resolve the merge conflicts.
Here's a helpful doc for that: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/addressing-merge-conflicts/resolving-a-merge-conflict-on-github

Conflicts Resolved

This is where the bug @MuhammadNSC is originating. In the Update.kt file. Can you look into this @nickolasram ? I will check back in later this week to see how it's going and support if needed. Looks like something buggy cropped up when resolving the merge conflict. My bad for not looking at this closer before force-merging. Thanks for bringing this up, Muhammad!

This is where the bug @MuhammadNSC found is originating. In the Update.kt file. Can you look into this @nickolasram ? I will check back in later this week to see how it's going and support if needed. Looks like something buggy cropped up when resolving the merge conflict. I should've looked at this closer before force-merging. Lesson learned. Thanks for bringing this up, Muhammad!

Bugs should be fixed. Hotfix branch pushed and pull request created


Refactor web tests

What this PR does / why we need it:

Use msw to make it closer to the actual behavior.
Remove unnecessary spyOn

Which issue(s) this PR fixes:
Fixes #
Does this PR introduce a user-facing change?:

NONE

/lgtm

/approve


matrix-synapse-unwrapped: 1.98.0 -> 1.99.0

Diff: https://github.com/element-hq/synapse/compare/v1.98.0...v1.99.0
Changelog: https://github.com/element-hq/synapse/releases/tag/v1.99.0
Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

meta.licenseneeds an update

It was updated to agpl3Plus in https://github.com/NixOS/nixpkgs/pull/274088

meta.licenseneeds an update

It was updated to agpl3Plus in #274088

Oh I see. That was too early I guess but whatever.


[Roadmap] Update roadmap in Box and Wiki

User Story

As a [user role below]:
Jeff

I need to:
have a place where I can upload an image

so that I can:
use it in the Github wiki (SMH)

Additional information

Upload the image in this issue
Use the image reference in the wiki
???
Profit!

Acceptance criteria

[ ] Have an image url for an image in the wiki


Angular 2.0 Integration

Could you please provide a way to integrate with Angular 2.0. Basically our requirement is to include the dynamic form only for few pages and static for few. Could you please let us know how we can leverage this using the angular-schema-form.

@anupamarachuri as Angular 2 has not been released we haven't started migrating the framework to ng2 yet. If you had ng2 with the ng1 compatibility module I would expect that to work, but I haven't tried so I can't be sure. We will be trying to implement for ng2 but there's no timeline on that yet.
Please re-open if you do not feel the question has been answered sufficiently.

@anupamarachuri there is work being done by @nicklasb on a demo of how to use the current version in an ng2 app and we have started work on making the code re-usable for ng2, but no timeline just yet.

I will start doing that soon™, meanwhile the documentation describes how it works:
https://github.com/OptimalBPM/of-admin/wiki/App-Entry-&-Bootstrapping


feat(shortcuts): add shortcuts test

Fixes #243

Feel free to tell me how my code is noob-y. 😆

Should this not be in the CLI instead of part of e2e?

Ha! @helgi that's a good point, and probably would lead to faster fixes if there ever were problems. I'll move it over.


Ritika Das

Love this one.

Love this one.

Thanks @NirmalSilwal !

Due we close new issues on PR merge and voting is still possible on closed issues I will close this one.


[FLINK-3748] [table] Add CASE function to Table API

This PR adds an evaluation function to the Table API that allows for if/else branching. It also allows CASE WHEN syntax in SQL.
I will update the EBNF grammar in the documentation as part of FLINK-3086 next.

Looks good to me.

Thanks for reviewing @fhueske! I will merge this...