Add support for flatpack version of godot

There is an unofficial flatpack package of Godot called "org.godotengine.GodotSharp". It also can be used to debug from Visual Studio Code, but launch.json will looks a bit different:
    {
      "name": "Launch",
      "type": "coreclr",
      "request": "launch",
      "program": "flatpak",
      "preLaunchTask": "build",
      "args": [
        "run",
        "org.godotengine.GodotSharp",
        "--path",
        "${workspaceRoot}"
      ],
      "cwd": "${workspaceFolder}",
      "console": "internalConsole",
      "stopAtEntry": false
    },
    {
      "name": "Launch Editor",
      "type": "coreclr",
      "request": "launch",
      "program": "flatpak",
      "preLaunchTask": "build",
      "args": [
        "run",
        "org.godotengine.GodotSharp",
        "--path",
        "${workspaceRoot}",
        "--editor"
      ],
      "cwd": "${workspaceFolder}",
      "console": "internalConsole",
      "stopAtEntry": false
    },

Have been banging my head on this for a few hours, solution worked perfectly. Thank you!! <3


Integrate the Pythia test harness

We will need to test our models against common, industry-standard benchmarks. Pythia is what everyone uses today:
https://github.com/EleutherAI/lm-evaluation-harness
The process will involve:

Updating test.py to load the model with the Transformers API
Overwriting it with your model's latest checkpoint
Then, executing Pythia tests from the same script

I added an eval.py script, which covers most of this work, but it doesn't seem to work right. For some reason, eval suites tend to fail almost immediately, with weird tokenization errors. I'm not sure if that's because of a poorly-trained tokenizer, or an under-trained model, or because of the custom architecture - and I'm not sure how to fix it, right now.

I was not aware of the evaluate library. This looks pretty nice, and since we use the Huggingface Transformers API already - would probably be easy to setup. We should try this one.


Fixed a couple of very minor issues.

Great gem!!
Just fixed a couple of minor issues - one that was preventing me from using it:
https://github.com/gryphon/simple_datatables/issues/6
And to help the newbies like me:
https://github.com/gryphon/simple_datatables/issues/4
Cheers

Thank you! Great work!


[1] velero - backup storage aws not found error

Describe the bug
The Velero pods generate a lot of error:
time="2022-01-11T13:11:54Z" level=error msg="Error in syncHandler, re-adding item to queue" controller=gc-controller error="error getting backup storage location: BackupStorageLocation.velero.io \"aws\" not found" error.file="/go/src/github.com/vmware-tanzu/velero/pkg/controller/gc_controller.go:145" error.function="github.com/vmware-tanzu/velero/pkg/controller.(*gcController).processQueueItem" key=velero/velero-daily-backup logSource="pkg/controller/generic_controller.go:140"

To Reproduce
Steps to reproduce the behaviour:

Connect to a cluster
And check the logs for the velero pod

Expected behaviour
Velero not to generate this error
Version (add all relevant versions):

Compliant kubernetes apps version v0.18.2

Additional context
I think this problem was addressed in #557
Definition of Done

the error is not generated any-more

Investigate if this issue is fixed with this one https://github.com/elastisys/compliantkubernetes-apps/pull/557


Updated Tightly coupled Architecture's

solves issue  #148
Changes I Made

[x] Updated the problem it solves part.
[x] Checked the entity and for Typos & Grammatical mistakes and made the changes accordingly.
[x] Asked my friends and peers from technical and Non-Technical background if they were able to understand all the sections, To which i had a positive response.

I've made only a few necessary change to the definition section. As for the Problem it addresses section I've almost rewritten the entire section because it was addressing the disadvantage of another term.

Before it was like this

A major disadvantage of Loosely Coupled Architecture is that it has a very low data rate and is very inconsistent.

After the changes it looks something like this

Tightly Coupled Architecture's mainly addresses problems related to having a very low data rate and inconsistency.

As I wasn't able to find a clear answer as to  what are the problems addressed by Tightly coupled architecture

Please let me know if i any changes are required

Thanks for your suggestions, @TakshakRamteke, but it needed a lot more work so we rewrote it. Btw, tightly coupled architecture is a property and doesn't require the three sections per style guide. We do appreciate the PR and patience!

Thank you Would try to do better next time


packemon build doc is outdated?

packemon v2 removed multiple formats option.
https://packemon.dev/docs/migrate/2.0#removed-multiple-format-support-when-configuring-a-package
https://github.com/milesj/packemon/pull/119
However, packemon build docs has referred multiple formats option.

"formats": ["lib", "esm", "umd"],
https://packemon.dev/docs/build#how-it-works

Actually, This example configuration looks like invalid.

Reproduce: https://github.com/azu/packemon-build-formats

❯ npm run build

><EMAIL_ADDRESS>build
> packemon build

 ERROR

 The following validations have failed for `packemon-build-formats`:
   - Unknown fields: formats.

The docs were wrong, just fixed, thanks.


Allow users to change options

At the moment the app using this plugin has to be forcibly restarted to change the options such as authenticationValidityDurationSeconds.
It would be good to be able to allow the storage options to be changed after the first initialisation.
The solution I currently use in Kee Vault is to change the behaviour of the forceInit flag, as can be seen in this commit: https://github.com/kee-org/biometric_storage/commit/59f28197995d215dacd5d584f10e13f800d0004a
However, I misunderstood the intention of that flag so it is probably not the right approach for us to take in the long run.
@hpoul has suggested "something like a deleteAndDispose which deletes the storage, clears the key and removes the storage file from the dictionary.. and after disposing you have to initialize it again".
I think that could work but maybe it would be neater if the consumer didn't have to keep track of the state of whether the storage file dictionary has been initialised. Although in my current implementation in Kee Vault I effectively already do this by caching the storage instance, there might be some app architectures for which it would be necessary for this library to handle that state without throwing an exception.
The library could store the currently active InitOptions and, perhaps gated by a new flag on getStorage, reinitialise the storage if a different set of options are supplied?

Is this implemented in any way?
This feature is really needed!
let's take the following scenario:
using OAuth, the user logs in, biometric auth request will be permitted to save the refresh_token,
after 5 minutes, the user is still in the app playing around!, the access_token expired,
assume we have the refresh_token stored in a variable and we don't need to read it again from storage (reduced 1 authentication request),
new access_token & refresh_token are received!,
we need to save the new refresh_token for future usages (if the user closed the app and reopened it), here the plugin requests a new biometric authentication & we must authenticate to write the new value.
implementing the feature of dispose the storage and reinitialize it again with different options will be great


Proxying only non-text/html requests to api server prevents OAuth login flow in development

The OAuth login flow using both front end and back end requests and callback URLs requires the host to be identical for both and identical to the host given in the application setup with the provider. Since the proxy setting only redirects non-text/html requests to unknown routes, the final callback in the token exchange phase is prevented.
I.e. a typical /api/auth/provider/callback route must be located on the same host/port as the one initiating the provider.com/oauth/authorize call. Since this final callback from the provider does not have content-type text/html it never hits the backend.
Possible solution: Support a more detailed config object in proxy parameter that allows for certain route matching patterns to always be redirected to back end regardless of content-type. Simple String/Object data type switching could allow for the change to be implemented without breaking support for the current url string version of the proxy parameter.

I’d like to avoid implementing a more complicated proxy system because people will keep asking for more, and the surface area of potential issues will increase.
This has been brought up before, and I asked something that I don’t think I got a good answer to. My question is: can we reliably detect OAuth flow in particular somehow, and add a special case for it?

I apologise for bringing up old questions. I have been googling for days and also read the entire issue thread leading up to the initial introduction of the proxy parameter but found nothing relevant to my problem.
I absolutely see your point about not complicating the proxy system any further and the desire to keep with the original intent of the project. At the same time being able to run both the api server and the development server concurrently seems like a common scenario. As is implementing OAuth and similar token exchange procedures that require both browser redirects and backend fetches over the same port.
I am not really sure if there is a way to reliably identify OAuth requests. On the other hand I suspect that adding a special case just for OAuth would be brittle. Especially since manually whitelisting certain paths (as originally suggested here https://github.com/facebookincubator/create-react-app/issues/147#issuecomment-235021655) would be much simpler and work for many scenarios other than OAuth as well.

Looking at https://github.com/facebookincubator/create-react-app/blob/master/packages/react-scripts/scripts/start.js I see that it would be possible to extract the middleware-creating lines from https://github.com/facebookincubator/create-react-app/blob/master/packages/react-scripts/scripts/start.js#L172 into a function that could be called for each entry in a proxy object. This would be a relatively easy and safe way to do it apart from the fact that http-proxy-middleware allows a function for route matching while express doesn't.
I see the need to run the regex to prevent proxying hot reloading and index.html. As of now this is the only regex used for route matching in the proxy (https://github.com/facebookincubator/create-react-app/blob/master/packages/react-scripts/scripts/start.js#L209) and in WebpackDevServer/Express (https://github.com/facebookincubator/create-react-app/blob/master/packages/react-scripts/scripts/start.js#L236). This is why you are able to pass the same regex to Express as you use for match testing in the proxy (talking to myself here..)
Would it be possible to try something like:
var pathFilter = function(pathname){
  return mayProxy.test(pathname) && pathname.match(pathFromProxyObject);
};
var hpm = httpProxyMiddleware(pathFilter, { // ...

And then implement the Express middleware like so:
devServer.use(function(req, res, next){
  if (pathFilter(req.path)) {
    return hpm(req, res, next);
  } else {
    next();
  }
});


HHH-13879 - Slow query log should use System#nanoTime not System#currentTimeMillis

https://hibernate.atlassian.net/browse/HHH-13879

@sebersole Thanks for merging it.


When vlan4093 is not used for the mlag ibgp peering, BGP neighbour addressing is not updated

Issue Type

[ ] Bug Report

Summary
Even when setting the option below to the vlan4094, SVI 4093 and L2 4093 vlan is not defined in the config files. However, addressing of vlan4093 is still used to peer over the MLAG:
Steps to reproduce

Set vlan 4094 for the mlag peering:

mlag_peer_l3_vlan: 4094

BGP Peering should use the addressing now of vlan 4094, but still uses the one from vlan 4093 even if its not defined:

neighbor <IP_ADDRESS> peer group MLAG-IPv4-UNDERLAY-PEER
neighbor <IP_ADDRESS> description DC1-LEAF1B

Expected results
Addressing from vlan4094 should be used instead:
interface Vlan4094
   description MLAG_PEER
   no shutdown
   mtu 9214
   no autostate
   ip address <IP_ADDRESS>/31

Actual results
Addressing defined in the fabric.yml file for vlan 4093 is used instead.

@dgonzalez85 This is not something we claim support for today, but it may still work, if you set the same IP scope for both vlans.
From the Fabric Variables documentation:
# IP address summary used for MLAG Peer Link (control link) and underlay L3 peering | *Required
# * When MLAG leafs present in topology.
# Assign range larger then total: L3 Leafs + 2 ]
mlag_ips:
  leaf_peer_l3: < IPv4_network/Mask >
  mlag_peer: < IPv4_network/Mask >

You may see wrong descriptions on vlan and SVI with this trick, since the data model for peer_vlan is simply overwritten by the l3_peer_vlan definitions.
We can convert this to a feature request, if we should build a more robust implementation. Example we could detect if the vlan is the same and adapt the generated config.

Perfect, that's a nice workaround too :)

That's works for me for now, but yes i agree with you, it might be something that can be a separate RFE so we do not need to define both ranges, and descriptions are consistent etc. Thanks!


Incorrect use of GetHostByName($env:computerName) as New-SelfSignedCertificate input

"[System.Net.Dns]::GetHostByName($env:computerName)"

This does not yield FQDN of host:
"[System.Net.Dns]::GetHostByName($env:computerName)"

because GetHostByName returns a collection of values, so this is the correct way:
("{0}" -f [System.Net.Dns]::GetHostByName($env:computerName).HostName)

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

ID: cf384fb2-ea82-4956-5199-bf8d1c305a20
Version Independent ID: cf384fb2-ea82-4956-5199-bf8d1c305a20
Content: Configure SQL Server Database Engine for encryption - SQL Server
Content Source: docs/database-engine/configure-windows/configure-sql-server-encryption.md
Service: sql
Sub-service: configuration
GitHub Login: @sevend2
Microsoft Alias: v-sidong

@mloskot - Mateusz, thank you for your correction here.  I am updating the article.

@sevend2 -- please review this change in private PR 26999.


HDDS-4769. Simplify insert operation of ContainerAttribute

What changes were proposed in this pull request?

Keep the original value in ContainerAttribute when inserting ContainerId.

What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-4769
How was this patch tested?
Unit test in TestContainerAttribute.testInsert() validates the correctness.

@elek @adoroszlai
Could you have a review? Split the update operation to make it more readable. And if we are going to add more information to ContainerID, it will be easier to modify the update operation.

I think it will return false when the ContainerID already exists in the NavigableSet?

I think normally the operation of insert needs to update the old value in the set, but for ContainerID, since it doesn't contain information other than ID, it's kind of a special case here that we don't need to update the old value.

But I'm not sure if there will be new fields added in ContainerID, maybe timestamps or versionID, then the special case for ContainerID would not exists. I added the method of update(key, value) so that we can update more easily.

I think it will return false when the ContainerID already exists in the NavigableSet?

You're right.  Then:
attributeMap
    .computeIfAbsent(key, any -> new TreeSet<>())
    .add(value);
return true;

Anyway, I don't think we need to prepare now for fields maybe being added to containerID in the future.

@adoroszlai The insert operation has been simplified according to your last suggestion, could you have a check?

The insert operation has been simplified according to your last suggestion, could you have a check?

Thanks @symious for the update, it LGTM.  I would like someone else to review it, since it's exactly my suggestion and I could be wrong.

Merging it. Thanks the patch @symious and the review @adoroszlai


5.2: Character "L" gets stripped from the player tag in the thumbnail generator if found at the end of the tag

When inputting a player name finishing with the character "L", the "L" will be stripped from the name in the thumbnail generator
Note: This doesn't affect the lowercase "l" and a workaround can be to add a space after the tag to make the "L" reappear

Fixed in MR #271, pending a new release

Closed because the fixed was merged on main


Allow usage of MPSolver.SetStartingLPBasis in Python API

This PR allows using the MPSolver::SetStartingLPBasis in Python, via the Swig interface.
The trap is that Python converts the MPSolver::BasisStatus enums to integers (for example, MPVariable::basis_status returns an 'int' in Python, see new test file).
So we have to define a custom function that converts back integers to MPSolver::BasisStatus.

@lperron @Mizux what do you think about this change request?

hello, any update this?


[BUG] data missing for quickstart

Describe the bug
The quickstart-from-source.sh script does not download data.
To Reproduce
Steps to reproduce the behavior:

clone this repo
configure java and intellij according to contributing guidelines
run ./quickstart-from-source.sh
Error Exception in thread "main" java.lang.IllegalArgumentException: data/sources/lake_centerline.shp.zip does not exist

Expected behavior
quickstart-from-souce.sh should download the required assets.
Environment (please complete the following information):

Hardware: dell xps 2021
OS: ubuntu
Java version and distribution: java 17.0.1 2021-10-19 LTS

Maybe we could add instructions somewhere that one needs to download the stuff first with something like:
sudo docker run -e JAVA_TOOL_OPTIONS="-Xmx1g" -v "$(pwd)/data":/data ghcr.io/onthegomap/planetiler:latest --only-download --area=monaco

Good catch! We'll add some more scripts for different run variations (docker/jar/maven) in #74 but for now I think all that might be needed is adding the --download option here


[gulp] Initial set up + dist

Set up gulp to:

compile sass -> css
compile jade -> html
concatenate and pack css and js
prepare dist folder as a static ready-to-deploy site.

Implemented


Sales forecasting end to end

Moved folder to root of truera-examples repo. Updated readme based on Josh's feedback.

Wow, this is a great resource - very comprehensive view of how to use our APIs.
Say you weren't using diagnostics to begin with - what would be your plan of attack for ingesting a dataset purely so you have a baseline to compare production data for drift calculations?

Also, at what step between 1-5, would you expect a customer to look at the Diagnostics UI?

Looks like you currently have the end2end folder in both the quickstart and root folder.
Can you also add this to the README in the root?

Also, at what step between 1-5, would you expect a customer to look at the Diagnostics UI?

likely after step 1 -- at that point, the development data and models would be fully populated.
In an alternative scenario, one might choose to use elements of script 5 to ingest existing data (i.e., all I/Os available, including pre-computed FIs) for whichever data is available to them.

Looks like you currently have the end2end folder in both the quickstart and root folder.
Removed
Can you also add this to the README in the root?
Will do

Wow, this is a great resource - it is a very comprehensive view of how to use our APIs.
Say you weren't using diagnostics to begin with - what would be your plan of attack for ingesting a dataset purely so you have a baseline to compare production data for drift calculations? Would you expect a customer to already have predictions stored somewhere so perhaps they wouldn't need the model object at all? Or, would you expect them to use the model object to generate the predictions?
Also, how strictly do customers follow the flow you took for Diagnostics?

upload features/labels
train a model
use model artifact to generate predictions and fis for each model
ingest predictions and fis for the models
Repeat 2-4

Are we always used directly within the model training/experimentation process, or do customers also use us after models are already fit?

@goyetc Would be interested to know about your replies on @rohitrastogi's above questions
My view is that,
We are used after the model is fit.

For simple models, we have the option for the user to ingest the pre-data and then use our compute functions to generate predictions and influences and then upload them (without having to define the schema etc...)

For complex models or cases where the user doesn't want to upload the model binary for whatever reasons (IP protection, too large etc...), we allow users to ingest predictions and influences computed by them using external functions and then use add_data or the specific functions like add_model_predictions or model_model_feature influences to upload

The idea being by being flexible, this should cover all the use-cases.

@Avinash Joshi @.***>,
Rohit and I synced on Friday. Your view is in line with our discussion.
Some additional questions came up, including

Do users typically add a model or add data, first? Does this
matter/are there reasons to do one before the other?
In the latter case you describe, (pre-existing I/Os or complex model
not conducive to TruEra execution), how can we make this easier for users?

'virtual model' concept -- how can this be more user-friendly /
less jargon heavy?
what additional utilities can we provide in SDK to help users, e.g.,
(1) utility function to create or recall column spec
(2) utility function to creat or recall model output context
(3) more help or clarity on how or why to create a feature map

A discussion of the differences between a pre-prod and prod
ingestion scenario, wrt what inputs will (almost always) be available

pre-prod/Diagnostics: previously, we hardened the software to
accommodate scenarios where labels are not available. In a
sense, this was
because we were promoting the use of Diagnostics for the purposes of
production monitoring. We also did not require predictions, in the prior,
model-runner-based world.
Now that we have Monitoring -- is there ever a scenario where a
split loaded directly into Diagnostics would not have labels? In
other words, in current state of software, is the only way a label-less
split would enter Diagnostics via the Monitoring-->Diagnostics workflow,
via a time range split? It seems so.
There appear to be opportunities for streamlining the SDK /
ingestion process based on (2) above.

cc @Rohit Rastogi @.***> for further discussion.
Colin
--
TruEra | +1<PHONE_NUMBER>
On Mon, Oct 16, 2023 at 5:54 AM Avinash Joshi @.***>
wrote:

Wow, this is a great resource - it is a very comprehensive view of how to
use our APIs.
Say you weren't using diagnostics to begin with - what would be your plan
of attack for ingesting a dataset purely so you have a baseline to compare
production data for drift calculations? Would you expect a customer to
already have predictions stored somewhere so perhaps they wouldn't need the
model object at all? Or, would you expect them to use the model object to
generate the predictions?
Also, how strictly do customers follow the flow you took for Diagnostics?

upload features/labels
train a model
use model artifact to generate predictions and fis for each model
ingest predictions and fis for the models
Repeat 2-4

Are we always used directly within the model training/experimentation
process, or do customers also use us after models are already fit?
@goyetc https://github.com/goyetc Would be interested to know about
your replies on @rohitrastogi https://github.com/rohitrastogi's above
questions
My view is that,
We are used after the model is fit.

For simple models, we have the option for the user to ingest the
pre-data and then use our compute functions to generate predictions and
influences and then upload them (without having to define the schema etc...)
2.
For complex models or cases where the user doesn't want to upload the
model binary for whatever reasons (IP protection, too large etc...), we
allow users to ingest predictions and influences computed by them using
external functions and then use add_data or the specific functions like
add_model_predictions or model_model_feature influences to upload
The idea being by being flexible, this should cover all the use-cases.
—
Reply to this email directly, view it on GitHub
https://github.com/truera/truera-examples/pull/21#issuecomment-1764121502,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFLYBEDHCY44YTPJS3VWAZTX7T74PANCNFSM6AAAAAA5SZOENE
.
You are receiving this because you were mentioned.Message ID:
@.***>


Persistant notification of all devices and their status

Inspired by the calendar widget notification app I have, a persistent notification that lists your devices in it.

Closing for now. I have not seen this feature requested. It was just an idea at first.


Update 0034-PROB-prob_weighted_liquidity_measure.ipynb

SLA 001 and 003 removal

This is the correct one https://github.com/vegaprotocol/specs/pull/1921 .


Exercise 6 / 22 / 26: Remove deprecated attributes from Manifest

remove host key
buildpack: don't refer github version #13
remove timeout

Reference

https://docs.cloudfoundry.org/devguide/deploy-apps/manifest-attributes.html

You cannot just remove host. the host key is marked deprecated for CF manifests. AFAIK the usage of host might need to be replaced by routes.
Concerning buildpack references in CF manifests. There a several ways to do this. There should be clear guidance in such tutorials, how to do it best in the context of productive SAP solutions on that platform. Most likely there have been technical reasons to reference a buildpack from github. If so, this should be mentioned (which it isn't).

Due to CF still evolving and changing over time, we need tutorials/documents that are continuously aligned with those changes.


test(weaver): added missed fabric-ca-server config

To extend cert expiry to 10 years
These files were missed when we were merging Weaver into Cacti, due to a wrong .gitignore.
Pull Request Requirements

[ ] Rebased onto upstream/main branch and squashed into single commit to help maintainers review it more efficient and to avoid spaghetti git commit graphs that obfuscate which commit did exactly what change, when and, why.
[ ] Have git sign off at the end of commit message to avoid being marked red. You can add -s flag when using git commit command. You may refer to this link for more information.
[ ] Follow the Commit Linting specification. You may refer to this link for more information.

Character Limit

[ ] Pull Request Title and Commit Subject must not exceed 72 characters (including spaces and special characters).
[ ] Commit Message per line must not exceed 80 characters (including spaces and special characters).

@sandeepnRES A couple of them are still failing unfortunately :-(
Test Asset Exchange Fabric / asset-exchange-fabric-local (pull_request) Failing after 6m
Test Asset Transfer / asset-transfer-local (pull_request) Failing after 13m

It seems the fabric certificates for test network have expired, will re-issue them in this PR only.

@sandeepnRES Please squash the commits together if possible

Done.


rhcos puddle job -> el8 rebuilds job

Whenever openshift and openshift-clients packages are built for el7, this job
takes the same content and rebuilds it under an el8 buildroot. RHCOS is the
only consumer of these builds.
NOTE: There are no concurrency controls because it is expected to be called
from within a job that already blocks concurrency per version.

Have never been aware that this roundabout is necessary and in use :)
I was wondering if there is anything against this approach. I am about to find out :crossed_fingers:


uaccess attribute error for AnonymousUser - release candidate 1.10

Needs to be a hotfix.

This seems to be fixed on beta. Will close after www deployment.


Fixed a bug where using PINRemoteImageBasicCache would cause images to be decoded on the main thread.

Issue reference: #451

PINRemoteImageBasicCache disk-cache methods are now no-op.
Changed cancelTaskWithUUID: documentation to say disk cache, and not memory cache.

Feel free to (of course) to pick on anything that seems weird. One decision that may feel odd is that I left the global-queue code in objectFromDiskForKey:completion:. My thought-process was to leave the code functioning as it did before as to not cause any unknown side effects. In this case, the completionHandler will continue to return from a global queue (as before), but it will just always return a nil for object, instead of just sometimes.

          1 Warning

:warning:
Any source code changes should have an entry in CHANGELOG.md or have #trivial in their title.

  Generated by :no_entry_sign: Danger


Debit note creation

[x] Place of supply function not properly working (Non vat registered)

[x] Set sales account in default (in select account field)

[x] Set 'default tax' rate in in creation screen

issue fixed !


[PE-D][Tester E] Invalid address inputs for add

Description
Currently, there is no restriction on the address inputs (all strings are valid). This means that an address like a/Commonwealth c/loseblock123 should technically be allowed by TA Assist (since addresses like a/Commonwealth b/loseblock123 are allowed), and there is no restriction on the address specified in the UG or command parser.
However, quite clearly, there would be issues parsing an addressa/Commonwealth c/loseblock123 because the command is ambiguous, and parser would pick c/loseblock123 as a class parameter rather than being part of the address.
Since most addresses generally do not have a "/" in them, perhaps it would be better to impose character constraints on addresses, just like how other parameters do not allow slashes in them?

Labels: severity.Medium type.FunctionalityBug
original: chloeelim/ped#8

Address will validate against any /. More UG info will be added in this regard.

Actually, this seems like a pretty niche problem, having <white_space>x/ for some character x for address seems pretty unlikely to be done by a user, validating in this scenario would only highly likely cause proper addresses to be invalidated, i.e. in Indonesia, Jalan Sumatra No. 10/20C is a perfectly valid address.
We'll add some notes regarding this issue on the appending on parameters and its constraints. Nice spot however!

Added note on documentation due to its relatively "niche" issue.


Use relative import path between core modules

What does this change?
Use relative import, not doing this has led to a strange failure in frontend https://github.com/guardian/frontend/actions/runs/7478681099/job/20354162665?pr=26808

Good catch. Any way we can avoid it cropping up in the future?

It may actually because I removed "rootDir" from the tsconfig, I've also restored that now.


A less invasive vite-jsx-plugin

Directives

v-if
v-else-if
v-else
v-show
v-text
v-html
v-model

https://github.com/cereschen/vite-jsx

Cool stuff, but instead of embedding it into the Vite core, you can add it and be featured on awesome-vite :)


"See" referencing style validation

"see" is only permitted before informative references and sections. ISO/IEC DIR 2, p. 45, 58

Turns out this is a lot easier to implement if we have #68


Multiple password request for p2p commands

Expected Behavior
Enter user password once for p2p actions
Current Behavior
2-4 times requests password
Context (Environment)
Linux

Fixed in Master.

Fixed in Prod.


This should be a CLI

I'm not a fan of the extra config file. Could be convert this is the a CLI. Where from and to are parameters. That way uses can just install and use this in their script easily.
What do you think?

Nice. Quick turnaround :+1:


Won't recieve any button press in Radio -> Logs

As everybody else I want to open/close my sunscreen via your project, from what I've seen it looks amazing.
I've connect my CC1101 radio & the ESP32 and at first glance everything seems fine. The radio is initialized, so far so good.
But when I go to the logs, I don't see any entry after I press any button on my Somfy remote.
I assume the Tx pin on the CC1101 is responsible for this action, it's connected to the Rx on the ESP32 (GPIO12).
Tried all kinds of different combinations and inputs on the ESP but nothing seems to work, i just can't receive any commands from the remote,
Do you have any suggestion(s) how to tackle this?
Hardware used:
CC1101 -> https://www.amazon.nl/gp/product/B07YX92NMP
ESP32 -> https://www.amazon.nl/dp/B0BS6R68H6

Which Somfy remote do you have?  Please verify that this is not a IO remote and is an RTS.

On the back of the remote it says Somfy 74300. Google learns that it's an (older) type off Somfy RTS.

That is a Telis remote and it should work well.   The hardware you selected should also do the trick.  For a radio comms like this (it is not a serial connection) you should not swap the TX to the RX.  The TX pin should match the GDO0 pin on the transceiver and the RX pin should match the GDO2 pin on the transceiver.  It is never a good idea to use the TXD, TX0, RXD, or RX0 labeled pins on any ESP32 if you have them.  These are serial UARTS that are likely in use for USB or they are attached to a JTAG for debugging.

Hello, I tried my remote today after building 2 devices and testing sending from one and receiving from another, it worked very well!
But when testing with the real remote at my friend's house, with a Tellis, it did not work. I downgraded to 2.2.3(was on 2.3.0), and it picked it right away!! Working very well.

Thanks both for the replies but I really can't get it to listen to my remote.
I now reflashed with 2.2.3 hoping it would solve my issue but again not button presses received although radio is initialized.
It's driving me crazy. Is there anything I can do/check to see why it doesn't work @rstrouse ? Want me to make some pictures of the connections ?

Want me to make some pictures of the connections ?

Yes

Let's see it this helps:
Below my connection schema, as you can see MOSI is connected to MISO and MISO is connected to MOSI
that way the radio is initialized, the other way miso -> miso and mosi -> mosi does not work
ESP32	Radio	Color
3.3V	        pin 2	Red
GND	pin 1 	Brown
GPIO5	pin 8	Purple
GPIO18	pin 4	Blue
GPIO19	pin 3	Grey
GPIO23	pin 5	Orange
GPIO13	pin 6	Green
GPIO12	pin7	        Yellow

https://github.com/rstrouse/ESPSomfy-RTS/assets/54189854/09d452f8-6a43-4c93-a2d8-0b987f18dc55

I took a look at your pics and you only have the GND and 3v3 pins connected correctly.  Unfortunately, I think you fell prey to the crappy way the transceiver is printed.  If you leave the wires connected to the transceiver the way have have them move the wires to the specified pin as I have indicated below.
1 - Brown - GND - OK
2 - Red - 3v3 - OK
3 - Gray - GDO0 TX - currently GPIO19 move to GPIO13
4 - Blue - CSN - currently GPIO18 move to GPIO5
5 - Orange - SCK - currently GPIO23 move to GPIO18
6 - Green - MOSI - currently GPIO13  move to GPIO23
7 - Yellow - MISO - currently GPIO12 move to GPIO19
8 - Purple - GDO2 RX - currently GPIO5 move to GPIO12

Again, thanks for all the effort you put in to help but unfortunately it still does not work. The radio does not even initialize and in my first setup it was.
Is there anything I can still do? Do you have a link for a proven working radio so I get buy a new one, just to find out if mine is maybe faulty?

Hello, I have this one: https://a.aliexpress.com/_EwltBKz
Also this one:
https://a.aliexpress.com/_EyySxIN
Both work, it did not work for me when I selected the wrong wiring, but once I selected the right wiring for the boards (E07-M1101D), they worked.
I hope those ones help you!
Edgar Salgado

From: Ed @.>
Sent: Friday, January 12, 2024 12:06:55 PM
To: rstrouse/ESPSomfy-RTS @.>
Cc: Edgar Salgado @.>; Comment @.>
Subject: Re: [rstrouse/ESPSomfy-RTS] Won't recieve any button press in Radio -> Logs (Issue #228)
Again, thanks for all the effort you put in to help but unfortunately it still does not work. The radio does not even initialize and in my first setup it was.
Is there anything I can still do? Do you have a link for a proven working radio so I get buy a new one, just to find out if mine is maybe faulty?
—
Reply to this email directly, view it on GitHubhttps://github.com/rstrouse/ESPSomfy-RTS/issues/228#issuecomment-1888894438, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAHOVTFWR335I7YFKSS7XADYOEKM7AVCNFSM6AAAAABBQPN37OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOBYHA4TINBTHA.
You are receiving this because you commented.Message ID: @.***>

Did you make the wiring changes as I suggested if you did, please make sure you set the pin configuration as I show in the screenshot below.  Click the enable radio and press Save Radio.  The hardware you have should work just fine unless it is defective.

Finally had some time to play around again and compared your last screen with my setup. Apparently MOSI and MISO were switched. Now I'm seeing the responses from my remote!!
Next phase is configuring the setup. Thanks for the patience and the help.

Awesome.  I am going to close out this issue for now.


[False Alarm.. but maybe..] GTA SA Mission Passed

I was going to say that the mission passed mod doesnt work the notify when someone gets a star, but turns out, only works in the same map where a player is but sometimes it doesnt play the jingle when someone got a star and the music remains in silence before playing again.

To be honest I'm not sure on why that may happen, but if I ever figure it out, I'll fix it, thanks for reporting the issue!


Docs for v5

Hi, I've just found AudioKit and am eager to use it. As new users we're encouraged to use v5. However, the docs and playgrounds are all for v4. Old sample code I'm trying to follow doesn't work with v5 either - why are there no proper docs for v5 yet?

Hi Yusuftor,
There is a migration guide for v5: https://github.com/AudioKit/AudioKit/blob/v5-main/docs/MigrationGuide.md
Also, feel free to contribute documentation as you see fit, this is a collaborative effort :D

@maksutovic Hi, I saw that but it didn't really help me. It tells me what I need to do to convert my app to V5, but doesn't explain the fundamentals.
For example, someone has created a tuner here, but none of what they use is available in V5, and just searching for an AKOrchestra or AKAnalyzer in the guide doesn't bring up any results. I can see that there's a PitchTap that I can attach to the microphone, but the docs don't explain what the PitchTap returns in its handler.

@yusuftor Have you seen https://github.com/AudioKit/Cookbook ?

@aure Ah that's perfect, thanks!


Add update command

What should it do? Re-lock? Bump the .in file? Update a single dep version?

@yehonatanz update single dependence is feature in piptools


Checkpointing utilities for TP

As per title.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


For some reason the title References is duplicated in the pdf version

Hello, thanks for creating this template.
I've seen that when rendering aft-html: default there is no problem with the Reference part. But when using aft-pdf at the end the title References appears double. You can see it in the demo pdf that you've published:
https://quarto-journals.github.io/article-format-template/
Is this a bug? Let me know.
Thanks

Thanks for the report. I removed from the template as the PDF version uses Natbib which already includes a title.


Use case: Composition of robots

Mobile manipulators are now a day almost in every research lab for robotics. These are basically multiple robots assembled into one mechanical construction. If each component is already ros2_control enabled there should be a simple way to create a complex robot (in ROS1 CombinedRobot).
Toward controllers, this construction should be transparent in terms that they automatically gain access to every one of them. This could be achieved using the DynamicJointState message.

Example 5 - 7


Correctly handle rpsLimits set to zero...

...by effectively disabling rate limits.
I noticed that our test suite at least was provoking rate limiting for total
rps limit set to 0.  Whether or not the tests should be running with rps limit
0, I feel this is more correct implementation.
r @raynos @rf

lgtm.

lgtm.


Create new empty color boxes

Ask to be assigned the issue
Add 4 more empty color boxes under the existing boxes

@OindrilaDas27 Please assign me this task?

@OindrilaDas27 I can do this task instantly. Please assign me.

@sidmadke you've been assigned the task. Complete it in 48hrs or it will be assigned to the person next in line

@OindrilaDas27 I have commented first

@OindrilaDas27 I have commented first, it was fcfs basis right?
Hi, I have already assigned you in one issue. It'd be fair to give others a chance too

@OindrilaDas27 I have made the PR #13
Do consider merging it


Refactored app initialization

In this PR I've refactored app intialization:

extracted separate App component which incapsulates routing stuff.
extracted separate routerMiddleware for web and native.

This has https://github.com/jitsi/jitsi-meet-react/pull/35 inside, so should be merged only after it.

@lyubomir fixed the pr

Updated the PR after MiddlewareRegistry was introduced in #47


Separate dynamic loading from static linking

@diyessi I'm approving but not sure what we want to support. Shouldn't nGraph have just two disjoint configuration: All backend static vs all backend dynamic? I don't see a usage case where we want only some backends to be static.


phnt: ntzwapi.h breaks builds with PHNT_VERSION prior to Red Stone 2

Building phnt with PHNT_VERSION set to anything prior to RS2 is currently broken due to conditional compilation hiding type definitions required for certain function prototypes inside ntzwapi.h (auto-generated).
Specifically type definitions under ntexapi.h:1213 as well as ntseapi.h:638 are causing issues for me. Pulling these outside of the #if seems to resolve the issue.

I've fixed this upstream and just waiting to sync with phnt


chore(batcher): rename done proposal -> executed proposal

This change is

#1477  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @alonh5 and the rest of your teammates on  Graphite


editableConfig.editableTagsInput.inputTpl added

added module.constant('editableConfig'. Can use other libraries for ngTagsInput.

With these changes other tags input libraries can be used with xeditable.
myApp.config(function(editableConfig) {
    editableConfig.editableTagsInput.inputTpl = "<chips>...</chips>";
})

And then use editable-generic-tags-input instead of editable-tags-input

That's all.
I believe this approach can also be used to make other views more extensible.

@aecepoglu please include only src folder in your PR

Hi @aecepoglu, thanks for the pull request.  A couple of things need to be changed.

Remove all generated files from your request.
Move your new directive to it's own file.
Update the documentation.
Add a test
Do all of the existing tests pass?

I assumed because they weren't .gitignore'd they were wanted.
Anyway, I'm going to need to extend the mixin props(items in docs/jade/reference.json because my items are objects and this mixin doesn't seem to handle them.
What would you like the output table to look like?
So this is what I'm trying to document:
editableGenericTagsInput: { inputTpl: "..." }

And the mixin outputs table of this form:
Name | Type | Description

So will this suffice?
editableGenericTagsInput | Object | An object of the form `{inputTpl: "...", otherAttributesMaybe: "..."}`.
                                          |             | *inputTpl: the html template to use for the tags input directive

For documentation you could just add a new folder to the "docs" folder and model it after an existing documented item.
Put the documentation in the desc.md file, an example in the controller.js/view.html file and a test in test.js file.  Then update structure.js to add your documentation, maybe in the "Techniques" section.

@aecepoglu can you update your pull request?

@aecepoglu can you update your pull request?

@aecepoglu do you need any help getting this pr updated?

Oh no no, I just can't find the time or motivation to do it; and I feel nobody is interested in the feature anyway. If you or someone else would like it added, I can make an effort. Otherwise we can close this.

Yeah, this has been here for a while and no one has commented that they wanted it.  I am fine with you closing it.


python: Convert os.open(…, O_EXCL) to open(…, "x")

(The "x" mode is new in Python 3.3.)
Testing plan: Ran tools/provision with and without var/remote_cache_prefix existing; checked that ./manage.py export works normally and fails when the tarball exists.

Merging due to reports that this works around a VirtualBox bug, I guess by opening with O_WRONLY instead of O_RDWR.


(fix) import typo in readme for nuxt

Small typo fix in the README in the nuxt example.
import redirectSSL from 'redirect-ssl' -> import redirectSSL from 'redirectSsl'

Thanks!


cel expression against a map in DynamicMessage cause CelEvaluationException about class cast

Hi,
First of all, I would like to apologize for that the issue #333 I posted was incorrect.
I have once again confirmed my program behavior, so let me ask a question again.
I tried to execute cel expression against a map in DynamicMessage built with a Descriptor which created from FileDescriptorSet at run time, and then that java process resulted into dev.cel.runtime.CelEvaluationException.
The exception message is like bellow.
 Unexpected exception thrown: dev.cel.runtime.CelEvaluationException: evaluation error at <input>:8: class com.google.protobuf.DynamicMessage cannot be cast to class com.google.protobuf.MapEntry (com.google.protobuf.DynamicMessage and com.google.protobuf.MapEntry are in unnamed module of loader 'app')

My question is that are there any way to execute a cel expression to map in DynamicMessage built with a Descriptor created at run time or are there any plan to support that case in cel-java.
Description
It seems that cel-java try to convert input DynamicMessage to a generated Message, probably a Message class generated from protobuf definition, with DynamicProto::maybeAdaptDynamicMessage but when build a Descriptor from FileDescriptorSet and use it to construct DynamicMessage to pass to eval method's argument, cel-java can not load a generated Message and so use DynamicMessage::Builder as maybeBuilder that is local variable in maybeAdaptDynamicMessge.
I'm not familiar with the DynamicMessage implementation, but from the behavior it appears that DynamicMessage::Builder treats the map field as a list of DynamicMessages, then the map field in Message returned by maybeAdaptDynamicMessage will also be the list of DynamicMessages and cause exception of class cast.
If use a generated Message's Builder to build Message or use generated Message's Descriptor to build DynamicMessage and pass it to eval method as argument, cel-java will be able to acquire a generated Message's Builder at DynamicProto::maybeAdaptDynamicMessage method and it treat map field as Map, so Exception will not happen.
To Reproduce
To reproduce, create descriptor and execute cel expression with following steps.

define protobuf message and convert it to FileDescriptorSet format using protoc as preparation
load the FileDescriptorSet and build a Descriptor from it in java code
build CelRuntime::Program using the Descriptor created in step 2
create DynamicMessage::Builder with the Descriptor which generated in step 2, and parse json to DynamicMessage
call CelRuntime.Program::eval and pass DynamicMessage built at step4
Exception will be raised

To do that, I write test code and put here.

Hi,
First it'd be helpful for me to understand the larger context of what you're trying to achieve here. From the test code, it seems like you're trying to deal with JSON data. The canonical way of achieving that is to leverage google.protobuf.Value (or google.protobuf.Struct). There will be a section added for this in the codelabs in the near future (currently being reviewed: https://github.com/google/cel-java/pull/349). I think this alone may make this question moot.
To answer the question -- the reason why you're running into this specific issue is because two semantically equal descriptors that differ in binary are present in the runtime: https://github.com/google/cel-java/blob/main/common/src/main/java/dev/cel/common/internal/DefaultInstanceMessageFactory.java#L78-L82. When constructing a default protobuf message, we rely on java reflection to lookup the descriptor to construct a default instance with but we try to ensure we're using the right descriptor for this work. The fix here is to not load the same file descriptor when you already have it available in memory (I realize you're probably doing this to reproduce the issue, but this work shouldn't be necessary).
As an aside -- I noticed that you're enabling CelValue and optional syntax in CelOptions. Keep in mind that 1) CelValue is currently experimental, as noted in the option, so you may run into some rough edges (but it is something we want to make the default for the runtime in the future) and 2) the preferred way of enabling optionals is through adding the library to your environment.

@l46kok
Thank you for your reply.
At first, sorry for not explaining enough my situation.
Let me explain again. What I want do is  following.

schemas written with protocol buffer is managed outside of java process
java program receive json value as a target value of cel expression and dinamically load a Descriptor for the value at runtime from schemas describe in step 1
build a DynamicMessage from the json value and CelRuntime::Program using the Descriptor loaded at step 2.
call eval method of the CelRuntime::Program and pass the DynamicMessage built at step2 as argument.

In step 3, input value’s schema is defined by a Descriptor and so I parse the value and convert it to DynamicMessage with the Descriptor.
In my use case, runtime process dynamically load Descriptors and does not include generated Messages, so can not load a Descriptor from a generated Message like Test.getDescriptor(). In the repository for reproduction, I added generated Messages under the directory src/main/java/org/example/proto to write the test code for the normal case, but it is not necessary to describe my problem situation. Also, in my test code I made CelForDynamicMessageTest not depends on src/main/java/org/example/proto in src/test/java/org/example/cel/dynamicmessage/BUILD.bazel setting, so it was intended to be that test process does not load generated Messages at runtime and resulted to same situation described above. However, this settings was difficult to understand and it made confusing. I apologize for this. I simplified my test code and push it here.
In my use case conditions, DefaultInstanceMessageFactory will not be able to load defaultInstance and it will be null:  https://github.com/google/cel-java/blob/v0.5.1/common/src/main/java/dev/cel/common/internal/DefaultInstanceMessageFactory.java#L231-L234, https://github.com/google/cel-java/blob/v0.5.1/common/src/main/java/dev/cel/common/internal/DefaultInstanceMessageFactory.java#L75-L77.
As a result, DynamicProto::maybeAdaptDynamicMessage will use DynamicMessage::Builder as maybeBuilder: https://github.com/google/cel-java/blob/v0.5.1/common/src/main/java/dev/cel/common/internal/DynamicProto.java#L104-L115. That is probably why, a map field remain as a list of DynamicMessages and eval method ends with cast error.
Sorry if I’m wrong, but It looks that a map field in DynamicMessage constructed using preveious described steps is consists of list of DynamicMessage which contains key and value fields, so to cast it to map, it may be necessary to execute some kind of conversion process, for example to take a key and value from each element of the list and convert it to an MapEntry. Because of these reasons, I asked that if there is a way to execute the cel expression in my use case or if cel-java plan to support this situation.
Thank you also for your advise about CelOptions. I tried several settings, and remaining enableOptionals option but it is not necessary. Regarding CelValue, if I set enableCelValue to false, result of cel expression is like this and looks same as when set it true.
Unexpected exception thrown: dev.cel.runtime.CelEvaluationException: evaluation error at <input>:eight: class com.google.protobuf.DynamicMessage cannot be cast to class com.google.protobuf.MapEntry (com.google.protobuf.DynamicMessage and com.google.protobuf.MapEntry are in unnamed module of loader ‘app’)
When disable CelValue, the exception is looks raise around here: https://github.com/google/cel-java/blob/v0.5.1/common/src/main/java/dev/cel/common/internal/ProtoAdapter.java#L216-L220.
With set enableCelValeu to true, exception was happen here: https://github.com/google/cel-java/blob/v0.5.1/common/src/main/java/dev/cel/common/values/ProtoCelValueConverter.java#L205-L207.

Oh I follow now, thanks for elaborating. CEL for the most part expects a non-dynamic descriptors as constructing protobuf messages (structs) in expressions is baked into the specification (example: Test {} would return Test.getDefaultInstance()). We only support dynamic messages to the extent where a concrete descriptor is present in the environment. So to support your case of mapping a dynamic message into the fields of dynamic descriptor would also mean we have to be able to somehow construct a default instance from it, and I don't really see a great way of achieving this. One area we could improve on is providing a more descriptive error message to make this clear.

Thinking about this further -- what I've mentioned regarding CEL-Java expecting concrete descriptors has been traditionally true, but we might be able to extend the support to dynamic descriptors as well. In discussing with the team, support for fully dynamic messages and descriptors may already be there for golang and C++ stacks via protobuf reflection. The biggest challenge would be to ensure that the usual CEL semantics is fully supported in CEL-Java. We'll explore this further and see what can be done.

Thanks for your explanation for the specification and suggesting ways to support my situation.

CEL for the most part expects non-dynamic descriptors as constructing protobuf messages (structs) in expressions is baked into the specification (example: Test {} would return Test.getDefaultInstance()).

I may not fully understand or misunderstand the cel-spec. If it's alright, let me ask about this comment. Does this mean cel-spec define that to execute a cel expression against a message, a runtime should load generated messages and should be able to call getDefaultInstance method correspoinding to that message?
With dynamically loaded Descriptors, I tried to run cel expressions which access to various types of fields, for example enum, repeated, and map, and those were success in case except map. Therefore it would be very appreciated if cel-java support to run the expression for a map in a DynamicMessage with Descriptors loaded at runtime.

Ah I meant to say CEL-Java on that, my mistake. CEL-Spec only specifies that an expression M {} must return an empty message. For CEL-Java, that historically meant ConcreteType.getDefaultInstance(). In cases where dynamic messages are provided as an input, we make all possible attempts to convert it into a concrete type as long as we have the descriptor with generated message available for it.
I think your observations about dynamic message is correct -- I think it should behave the same except for map (and only for cases where JSON is parsed into dynamic message, because it converts the json map into a dynamic message to preserve ordering post-serialization). I just need to verify the behavior against few other things such as function dispatch before we can canonically support this case.

Thank you for your response. I understood what you are saying about cel-java specification.
Also, I really appriciate for your handling to support the situation I reported.


Add support for building 2.1 based Server.IIS package

https://github.com/aspnet/IISIntegration/issues/1522
/cc @natemcmaster

Do we care that tests are publishing for 2.2 instead of 2.1?

Also is this using the 2.1 build in tests?

Do we care that tests are publishing for 2.2 instead of 2.1?

We'll just check when giving the build away.

Also is this using the 2.1 build in tests?

It uses the project reference so yes.


RocketChat Desktop client -Server reload

My Setup

Operating System: Windows 10 Enterprise
App Version: 2.15.3
Installation type: exe

[x] I have tested with the latest version
[x] I can simulate the issue easily

Description

Everytime i logout and relogin get the error "Server failed to load"

.
Current Behavior

Need to click retry to login to rocketchat desktop client.
Expected Behavior

It should login without throwing "Server Failed to load" error.

Please test if the latest release did solve this. If not please reopen the issue or create a new one and tell us. I will be closing this one for now.
Thank you!


Migration to firefox-devtools org

Firefox DevTools wants to support the development of this extension. Joining forces with @hbenl, who will remain a maintainer, we'll continue with shared governance!
Apart from @hbenl and me, the Firefox DevTools team members that will be involved are @janodvarko and @jasonLaster.
How will shared governance help this project?

Keep the extension in sync with Firefox's debugger server, remote protocol changes and upcoming features (e.g. dfddc7a8134b3672fd9857f4c1d39a56a582ef02, 5f86bdbd2c27137e55f3bfd3352a95abb7a2cb6a, etc)
Simplify setup and integration with DevTools
Make this the official Firefox extension for VSCode, with proper branding (e.g. #81)
Do more joint outreach to spread the word about the extension and how to set up

What's next?
🚧 Draft

[ ] Add Mozilla's contributing.md, CODE_OF_CONDUCT.md
[ ] Transfer repo to https://github.com/firefox-devtools

[ ] Make @hbenl and @jasonLaster, @janodvarko & @digitarald maintainers of that repo
No need to update Copyright/License

[ ] Update references

[ ] VSCode extension marketplace

[ ] Update branding

[ ] New icon
[ ] Updated description

[ ] Ship a new release
[ ] Announcement

[ ] Blog post

I have one more item for the TODO list: every maintainer should be able to publish new versions of the extension. There seem to be 2 options for this:

move the extension to a new publisher account and add all maintainers to this publisher
add other maintainers to the current publisher account

The big downside of the first approach is that this would change the extension ID in the VS Code Marketplace and hence all current users of the extension would have to reinstall it - it seems there is currently no way to automatically migrate them to the new extension ID.
The downside of the second approach is that the current publisher account is also used for my other extensions and I would give you the ability to manage those extensions as well. But I'd trust you to not abuse this, so I'd be OK with that.
There doesn't seem to be any documentation on how to do this, all I could find so far are these issues:
Transfer Publishers in Marketplace
Guidance for transferring ownership of extension

Good point @hbenl ! I have been in touch with the VSCode team and they offered to change the publisher ID. I will add those items to the list then.

The migration of the extension to the firefox-devtools publisher was successful and I already made a bugfix release with the new publisher.


feat: mount redfish credentials

This PR mounts redfish credentials to the kepler exporter pod

/hold

closing this since #364 is merged 🎊


[BUG] Flaky test in DataFileLoaderTest

Describe the bug
> Task :data-prepper-plugins:dynamodb-source:test

DataFileLoaderTest > test_run_loadFile_correctly() FAILED
    org.mockito.exceptions.verification.WantedButNotInvoked at DataFileLoaderTest.java:139

Example failure:
https://github.com/opensearch-project/data-prepper/actions/runs/6470850267/job/17567944366?pr=3469
To Reproduce
This is flaky, so this might require some investigation.
Expected behavior
This test should consistently pass.

@daixba , Could you take a look at this flaky test?

Sure, I will work on this.
I have seen this before, it's strange that it does not fail all the time and I can't reproduce this failure in my local.  My best guess is that there is no enough 'time' before the 'expected invoke' to happen due in the github ci instance.

Recently failing test running on Java 11:
> Task :data-prepper-plugins:dynamodb-source:test

DataFileLoaderTest > run_loadFile_with_acknowledgments_processes_correctly() FAILED
    org.mockito.exceptions.verification.WantedButNotInvoked at DataFileLoaderTest.java:222

This is a new test after end to end ack is added. The previous test test_run_loadFile_correctly was successfully.  This one was failed because the data mapping issue, I believe I have fixed it.
I will keep an eye on this issue. This ack is not added by me, I can do some check and try to provide a fix. Please leave this issue open for now.

The Flaky issue (DataFileLoaderTest.java:222) regarding flush() wanted but not invoked is because there was
private final int total = random.nextInt(10);
in the test code to generate random number of test records.
However, I just noticed that the 0 is inclusive for nextInt() method, hence, there is a small chance that the total records is 0. In which case, the flush() was not invoked.
I have run the test again and again and confirm that only when total == 0, the test failed. I was trying to fix this issue by adding at least 1 to the total count.  However, this test was already modified.
Let's see if this issue still exists.

See change as below:
https://github.com/opensearch-project/data-prepper/pull/3592/files#diff-d0164dc5e11eb3ef88aa4fabf7ae3fd4b0ebda3c17120c2fcc7849a4d43578c4L96-R96

Yes, roughly 1 out of 10 times that condition should occur, so I think that is likely the fix. For the future, would be nice to have had a separate PR so that such a nice fix is not blocked by other PR comments.

Resolved by #3592.


Create several survival functions with detailed explanations of the function.

Try to add important functions

[ ] the Kaplan-Meier Estimate,

[ ] hazard rates using Nelson-Aalen

[x] cumulative hazards using parametric models(Exponential, Log-Logistic, Log-Normal ,Splines and Nelson-Aalen)

[x] Cox’s proportional hazard model

[x] Custom regression models(if possible)

Here is my progress so far -
Dashing-Again.zip
I will update the explanation part for each section and will update the styling as well.
Currently, I am trying to fix some errors with Nelson-Aalen and Kaplan-Meier
Here are some screenshots :

Try to change to this for clear background
plot_bgcolor : rgb(102, 204, 255)
paper_bgcolor : rgb(255, 255, 255)
Well it's looks fine.

Okay, I will add the explanation part soon and will change the plot and paper colours.

My progress till now:
Still trying to fix some minor glitches.
Dashing-Again.zip


Investigate why benchmark cleanup does not work on windows

In #17 we had to introduce file chmod-ding in order to be able to clean-up benchmark repos under the Windows.
This is strange, as it seems that TemporaryDirectory should do that by itself:
https://github.com/python/cpython/blob/fa919fdf2583bdfead1df00e842f24f30b2a34bf/Lib/tempfile.py#L792
Investigate why do we receive WinError 5 (Access Denied) on windows runner upon benchmark directory cleanup.

Closing as stale.


chore(version): Bump traitobject

[X] The commit messages match the guidelines in https://github.com/hyperium/hyper/blob/master/CONTRIBUTING.md#git-commit-guidelines

Noticed that a project was pulling in two versions of traitobject.


Implement FlowDirection - Bidirectional layout

What does the pull request do?
Closes  #7345. implement FlowDirection as wpf, that is, bidirectional layout.
What is the current behavior?
FlowDirection implemented only for Bidirectional text layout.
What is the updated/expected behavior with this PR?
FlowDirection implemented for all controls.
Todo

[x] ComboBox - content appears inverted until open the list.
[x] Conflict with RenderTransformOrigin - when it isn't setting to 50% 50%, things get wrong.
[x] ~Notification shuold appears in right side.~ - Does not require intervention, can be configured in WindowsNotificationManager.FlowDirection property.
[x] Popups - including Menus Flyouts etc.

How was the solution implemented (if it's not obvious)?
I added to all the controls that should appear inverted visually, RenderTransform fits, and if their children does not need to appear inverted, then i inverted them again.
I bypassed the RenderTransform mechanism and the effect is applied while processing the scene, so as not to conflict with RenderTransformOrigin.
I was trying to follow this document:
https://docs.microsoft.com/en-us/dotnet/desktop/wpf/advanced/bidirectional-features-in-wpf-overview?view=netframeworkdesktop-4.8
The concept is basically applied in WPF in a similar way, can be seen here:
https://github.com/dotnet/wpf/blob/69a406c7f896042fba29a83943d58947cc8b11c3/src/Microsoft.DotNet.Wpf/src/PresentationFramework/System/Windows/FrameworkElement.cs#L3961
Important
This is my first PR in GitHub, And I'm not very familiar with the conventions of Avalonia either, so please try to be gentle (;
If anyone can help with this or any ideas for implementation, I'll be happy, it's a very important topic that opens up a world of new options for software aimed at RTL languages.
Screenshot

Checklist

[ ] Added unit tests (if possible)?
[x] Added XML documentation to any related classes?
[ ] Consider submitting a PR to https://github.com/AvaloniaUI/Documentation with user documentation

Breaking changes
Unknown
Fixed issues
#7345

You can test this PR using the following package version. 0.10.999-cibuild0019307-beta. (feed url: https://nuget.avaloniaui.net/repository/avalonia-all/index.json) [PRBUILDID]

@hez2010 I know, it's related to a platform dependent API, it can be done in a separate PR

@danielmayost You are my hero. Thanks for your contribution. Will have a look shortly.

You can test this PR using the following package version. 0.10.999-cibuild0019326-beta. (feed url: https://nuget.avaloniaui.net/repository/avalonia-all/index.json) [PRBUILDID]

You can test this PR using the following package version. 0.10.999-cibuild0019333-beta. (feed url: https://nuget.avaloniaui.net/repository/avalonia-all/index.json) [PRBUILDID]

Keep in mind that each individual visual can have a different FlowDirection so if a parent is RTL but a child is LTR the LTR control needs to be mirrored.

Keep in mind that each individual visual can have a different FlowDirection so if a parent is RTL but a child is LTR the LTR control needs to be mirrored.

If I understood you, then that's exactly what I did.

You can test this PR using the following package version. 0.10.999-cibuild0019338-beta. (feed url: https://nuget.avaloniaui.net/repository/avalonia-all/index.json) [PRBUILDID]

It seems like I'll have to get involved in the scene update process, so as not to create conflicts with RenderTransformOrigin, I'll add another IVisual property.
I'll work on it today or tomorrow.

I always though it would require layout system updates, including all panels, instead of simple mirror transformation. Surprised WPF did it this way. Well, I guess it's proven with time.

IMO, I'm not a fan of applying the mirror transform directly to RenderTransform (even if that's what WPF did), as if a transform is set and then it's read back, a different transform may be returned if in RTL mode. It'd be better to apply the mirror transform in the layout engine or in the renderer, invisible to the end user.

IMO, I'm not a fan of applying the mirror transform directly to RenderTransform (even if that's what WPF did), as if a transform is set and then it's read back, a different transform may be returned if in RTL mode. It'd be better to apply the mirror transform in the layout engine or in the renderer, invisible to the end user.

That shouldn't be too difficult in principle I would think. At the last step when the RenderTransform is being applied during layouting combine it with any Mirror transform as required. Don't modify any properties -- just apply both transforms together each time.
That said, an argument can be made from the opposite viewpoint. Some applications may want to know the actual RenderTransform with any framework modifications as well. Any examples would be contrived but a case can be made that the RenderTransform property should always be the actual applied RenderTransform.

That shouldn't be too difficult in principle I would think. At the last step when the RenderTransform is being applied during layouting combine it with any Mirror transform as required. Don't modify any properties -- just apply both transforms together each time.

This is what I planned to do.

How does it feel now?

You can test this PR using the following package version. 0.10.999-cibuild0019346-beta. (feed url: https://nuget.avaloniaui.net/repository/avalonia-all/index.json) [PRBUILDID]

You can test this PR using the following package version. 0.10.999-cibuild0019348-beta. (feed url: https://nuget.avaloniaui.net/repository/avalonia-all/index.json) [PRBUILDID]

You can test this PR using the following package version. 0.10.999-cibuild0019349-beta. (feed url: https://nuget.avaloniaui.net/repository/avalonia-all/index.json) [PRBUILDID]

You can test this PR using the following package version. 0.10.999-cibuild0019404-beta. (feed url: https://nuget.avaloniaui.net/repository/avalonia-all/index.json) [PRBUILDID]

@Gillibald I'm done, everything seems to be working.
If you have naming comments, check me out.
There are some bugs related to #7502, I will write there later.

You can test this PR using the following package version. 0.10.999-cibuild0019421-beta. (feed url: https://nuget.avaloniaui.net/repository/avalonia-all/index.json) [PRBUILDID]

You can test this PR using the following package version. 0.10.999-cibuild0019425-beta. (feed url: https://nuget.avaloniaui.net/repository/avalonia-all/index.json) [PRBUILDID]

You can test this PR using the following package version. 0.10.999-cibuild0019440-beta. (feed url: https://nuget.avaloniaui.net/repository/avalonia-all/index.json) [PRBUILDID]

You can test this PR using the following package version. 0.10.999-cibuild0019514-beta. (feed url: https://nuget.avaloniaui.net/repository/avalonia-all/index.json) [PRBUILDID]

I don't know if I'm using this right or no. But the TextBox control has a problem in RTL layout in all the package versions named here, including 0.10.999-cibuild0019514-beta. When the FlowDirection is set to RightToLeft backspace and delete keys won't work without a selection. And selection with arrow keys doesn't work as expected too.

@koosha72 All the problems you mentioned (and there are more), are related to text layout, and it is addressed in #7746.
If you know of any new issues in this topic, you can report in #7502, a concentration of issues on the topic.
Feel free to report RTL layout issues here.

@koosha72 All the problems you mentioned (and there are more), are related to text layout, and it is addressed in #7746. If you know of any new issues in this topic, you can report in #7502, a concentration of issues on the topic.
PS If you mean the selection using the arrows is reversed, it should be like this.
Feel free to report RTL layout issues here.

Thanks for your explanation. I thought the problem is here because on other versions of Avalonia TextBox doesn't have problems with rtl text. Only the versions with rtl layout support has problems. May be it's because the commits are not merged, I'm not sure.

You can test this PR using the following package version. 0.10.999-cibuild0019648-beta. (feed url: https://nuget.avaloniaui.net/repository/avalonia-all/index.json) [PRBUILDID]

anybody?

You can test this PR using the following package version. 0.10.999-cibuild0019660-beta. (feed url: https://nuget.avaloniaui.net/repository/avalonia-all/index.json) [PRBUILDID]

You can test this PR using the following package version. 0.10.999-cibuild0019692-beta. (feed url: https://nuget.avaloniaui.net/repository/avalonia-all/index.json) [PRBUILDID]

You can test this PR using the following package version. 0.10.999-cibuild0019715-beta. (feed url: https://nuget.avaloniaui.net/repository/avalonia-all/index.json) [PRBUILDID]

You can test this PR using the following package version. 0.10.999-cibuild0019740-beta. (feed url: https://nuget.avaloniaui.net/repository/avalonia-all/index.json) [PRBUILDID]

Great work! This was a surprise PR in several ways :)

Thanks

Amazing work! when will this be available? It seems like FlowDirection is still not available even when I use [11.0.0-preview4](https://www.nuget.org/packages/Avalonia/11.0.0-preview4).
cc @Gillibald

You may also want to test it in latest nightly builds, which is available via nuget. Alternative is to wait for preview5 which should drop in the next few days (planned)

for me it works in Preview4, just set FlowDirection="RightToLeft", what the problem?

@danielmayost it gives me a compilation error saying that FlowDirection doesn't exist. I updated to the Nightly builds it worked


Can not create config maps when install Harbor on Kubernetes ( v1.3.3 ), error message show " the ConfigMap  is invalid "

At first, modify some basic configs, included make/harbor.cfg,make/kubernetes/**/*.rc.yaml and so on.
Then I run this command
python make/kubernetes/prepare

and some files generated. But some errors happens when I exec
kubectl apply -f make/kubernetes/jobservice/jobservice.cm.yaml

Output is
[root@XXX ]# kubectl apply -f make/kubernetes/jobservice/jobservice.cm.yaml
The ConfigMap "harbor-jobservice-config" is invalid.

* data[MYSQL_PORT]: Invalid value: "MYSQL_PORT": must have at most 253 characters and match regex \.?[a-z0-9]([-a-z0-9]*[a-z0-9])?(\.[a-z0-9]([-a-z0-9]*[a-z0-9])?)*
* data[REGISTRY_URL]: Invalid value: "REGISTRY_URL": must have at most 253 characters and match regex \.?[a-z0-9]([-a-z0-9]*[a-z0-9])?(\.[a-z0-9]([-a-z0-9]*[a-z0-9])?)*
* data[SECRET_KEY]: Invalid value: "SECRET_KEY": must have at most 253 characters and match regex \.?[a-z0-9]([-a-z0-9]*[a-z0-9])?(\.[a-z0-9]([-a-z0-9]*[a-z0-9])?)*
* data[EXT_ENDPOINT]: Invalid value: "EXT_ENDPOINT": must have at most 253 characters and match regex \.?[a-z0-9]([-a-z0-9]*[a-z0-9])?(\.[a-z0-9]([-a-z0-9]*[a-z0-9])?)*
* data[GODEBUG]: Invalid value: "GODEBUG": must have at most 253 characters and match regex \.?[a-z0-9]([-a-z0-9]*[a-z0-9])?(\.[a-z0-9]([-a-z0-9]*[a-z0-9])?)*
* data[LOG_LEVEL]: Invalid value: "LOG_LEVEL": must have at most 253 characters and match regex \.?[a-z0-9]([-a-z0-9]*[a-z0-9])?(\.[a-z0-9]([-a-z0-9]*[a-z0-9])?)*

make/kubernetes/jobservice/jobservice.cm.yaml file output:
apiVersion: v1
kind: ConfigMap
metadata:
  name: harbor-jobservice-config
data:
  MYSQL_HOST: mysql
  MYSQL_PORT: "3306"
  MYSQL_USR: root
  MYSQL_PWD: "rponline"
  UI_SECRET: "xGmyDfwYbwUPLcVX"
  SECRET_KEY: "0GFvILdJ5Wj7oJkQ"
  CONFIG_PATH: /etc/jobservice/app.conf
  REGISTRY_URL: http://registry:5000
  VERIFY_REMOTE_CERT: "on"
  MAX_JOB_WORKERS: "3"
  LOG_LEVEL: debug
  LOG_DIR: /var/log/jobs
  GODEBUG: netdns=cgo
  EXT_ENDPOINT: "http:/XXX.com"
  TOKEN_URL: http://ui
  config: |
    appname = jobservice
    runmode = dev
    [dev]
    httpport = 80

Could anyone help me? Thanks very much.

@ScorpioCPH could you help?

@kdada

Please give the kubectl version:
$ kubectl version

Output
# kubectl version
Client Version: version.Info{Major:"1", Minor:"3", GitVersion:"v1.3.3", GitCommit:"c6411395e09da356c608896d3d9725acab821418", GitTreeState:"clean", BuildDate:"2016-07-22T20:29:38Z", GoVersion:"go1.6.2", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"3", GitVersion:"v1.3.3", GitCommit:"c6411395e09da356c608896d3d9725acab821418", GitTreeState:"clean", BuildDate:"2016-07-22T20:22:25Z", GoVersion:"go1.6.2", Compiler:"gc", Platform:"linux/amd64"}

@kdada

We have tested in kubernetes 1.4 and 1.5, The ConfigMap can be created successfully. May be it's a limitation of kubernetes 1.3 ( We have no cluster versioned 1.3 to test).
You can modify those keys in data to satisfy the requirements as a workaround.

Thanks for your reply. I will have a try on other k8s version.

Can create config maps in kubernetes 1.5.6, but errors occurred when created rc using these config maps.
# kc get configmaps
NAME                       DATA      AGE
harbor-jobservice-config   16        3m
harbor-mysql-config        1         3m
harbor-nginx-config        3         3m
harbor-registry-config     2         3m
harbor-ui-config           30        3m

Errors
# kubectl apply -f make/kubernetes/mysql/mysql.rc.yaml
The ReplicationController "mysql-rc" is invalid: spec.template.spec.containers[0].env[0].valueFrom: Invalid value: "": may not be specified when `value` is not empty

The harbor-mysql-config output is
# kc describe configmaps harbor-mysql-config
Name:		harbor-mysql-config
Namespace:	default
Labels:		<none>
Annotations:	<none>

Data
====
MYSQL_ROOT_PASSWORD:	8 bytes

The mysql.cm.yaml output is
# cat make/kubernetes/mysql/mysql.cm.yaml
apiVersion: v1
kind: ConfigMap
metadata:
  name: harbor-mysql-config
data:
  MYSQL_ROOT_PASSWORD: rponline

Kubernetes version
#  kubectl version
Client Version: version.Info{Major:"1", Minor:"5", GitVersion:"v1.5.6", GitCommit:"114f8911f9597be669a747ab72787e0bd74c9359", GitTreeState:"clean", BuildDate:"2017-03-28T13:54:20Z", GoVersion:"go1.7.4", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"5", GitVersion:"v1.5.6", GitCommit:"114f8911f9597be669a747ab72787e0bd74c9359", GitTreeState:"clean", BuildDate:"2017-03-28T13:36:31Z", GoVersion:"go1.7.4", Compiler:"gc", Platform:"linux/amd64"}

@kdada  Could you help me what's wrong with the problem ？ Thanks very much

@willstudy
Can you show me the mysql.rc.yaml ？

For deployment on K8S, please refer to the helm chart:
https://github.com/goharbor/harbor-helm


update code preview to look nicer

What: Change code preview to look nicer

Why: Because we care about looking nice

How: Pretty much copy all of styled-components 💅
Here's some gifs:

The box shadow looks funny in those gifs, looks much better IRL:


Can't have code blocks inside tags

$ node
> var m = require("marked");
undefined
> m("```\nThis is code\n```");
'
This is code\n
'
> m("
```\nThis is code\n```
");
'
This is code
'
> m("
\n```\nThis is code\n```\n
");
'
\nThis is code\n
'

The first example works as expected. The second and third just do , like it's inline. It's not tag specific, or least it doesn't appear to be...
> m("<abc>\n```\nThis is code\n```\n</abc>");
'<abc>\nThis is code\n</abc>'

Just any code fences inside markup is assumed to be inline code instead of code blocks.

#983


battery module

it will be nicer if the battery module handles the state where the laptop's battery is removed and is running on the AC adapter and also the transition in between plugin and out the battery ^^

Could you elaborate? What functionality would you like to see?

@NBonaparte
Case 1 :
if laptop battery is unpluged and the laptop is running on the AC cable we start polybar and it will drop the battey module thats normal because there is no battery here i want it to not drop the battey module and instead show or print message that we are running on AC only (like it handle this event ) and therefore i can put a nice lightning icon of it rather then having nothing in this case
Case 2 :
lets suppose that we are running on AC only and then i plug the battery i want polybar [ battery module ] to automaticly detect it and then show that we are in charging mode
sorry for my bad english :3
if its not clear enough i will do a small screencast to show you ^^
thank you

I'll take a look at implementing these in the next few days.


Should be able to select a subset of fields and populate the activerecord model

It should be possible to utilize select(array('subset', 'of', 'fields')) and also go through serialization stages that models go through.
In addition, using a specific formatter (e.g. ObjectFormatter) with select() fails for totally not-obvious reasons (it literally just goes through formatting expecting all the fields to exist)

There's no way to hydrate a object with limited informations yet. This would make it necessary that any column is dynamically lazy-load which is only in the data-mapper implemented.


Add functions to generate mass matrix probs

Open to however you want to do this syntax-wise. I made a new function but could put it in one of the old ones with a keyword argument if you'd prefer

I couldn't think of a good reason, so we're all in one now


[Rust]: Adapt existing benchmarks to have proper execution over AVX-512

Note: migrated from original JIRA: https://issues.apache.org/jira/browse/ARROW-10615
Some benchmarks are utilizing the same data which is easily predictable during execution, moreover, some of them the insufficient amount of data to utilize the power of the implemented operations.

I am not sure this ticket is actionable so closing


Steam VR Setup

Hello, a question about documentation in Wiki
On this doc: https://github.com/cboulay/PSMoveService/wiki/Steam-VR-Setup
In " Configure steamvr.vrsettings
Open C:\Program Files (x86)\Steam\config\steamvr.vrsettings"
The file is created automatically by "SteamVR_InitialSetupWin64.bat"  or i need to create a empty file on the folder after run the .bat ?

This is my config folder after run the .bat and driver .bat...
no steamvr.vrsettings  file !
http://i.imgur.com/VHRTI95.jpg

Found the new location: C:\Program Files (x86)\Steam\steamapps\common\SteamVR\resources\settings
Pls update the wiki. Thanks

I believe the settings file in C:\Program Files (x86)\Steam\steamapps\common\SteamVR\resources\settings is the defaults file in the event that no steamvr.vrsettings file exists. You have to make the steamvr.vrsettings file if none exists. The SteamVR_InitialSetupWin64.bat does not make that file since it could have other settings in there unrelated to psmoveservice. I should make that more clear in the wiki though.
Also there is a newer version of OpenVR out now (1.0.6) that changes a few things about you register plugins with SteamVR. After I finished up with the C-API work I was doing update the psmove plugin to use openvr 1.0.6 and fix up the SteamVR_InitialSetupWin64.bat script. I'll probably do another documentation pass then too.

ok, thank you.


RDoc for CORE

Create RDoc documentation for CORE Methods such as world, elements, pages, ledger, and logger.

While documentation for CORE would be nice, i don't necessarily think we should use RDocs for it.  I'd personally prefer to have a good suite of unit tests and then from that generate docs using a tool like relish.
The problem with manually written documenters like RDoc is that they frequently fall out of spec with the things that they describe.  (there's NOTHING worse than a magic comment that says it does something completely different than what the code under it actually does.) At least if you generate your docs from unit and integration tests then you get a notification when this happens.


:NextError jumps between top and bottom buffer

Describe the bug
The respectable error causing code is opened in different buffers. Switching between top and bottom in my case.
To Reproduce
Steps to reproduce the behavior:

Run :Recompile or :Compile
Note: There should be more than 2 compilation warnings/errors
Run :NextError multiple times

Expected behavior
The errors should be opened in the same window/buffer and not jump arround from top to bottom and so on. Also i would expect the compilation errors (the temporary buffer) to stay open for further error information.
Screenshots
If applicable, add screenshots to help explain your problem.
Neovim Version
Include the result of nvim -v:
v0.9.4

compile-mode.nvim Version
Specify what version of compile-mode.nvim you are using.
latest

Additional context
Add any other context about the problem here.
I use make and cc for compiling the code.
Also i am running neovim inside WSL.
C code for testing:
#include <stdio.h>

int main(void)
{

    int i = 17;
    int *ip = &i;
    char *cp;
    char *cp1;
    void *vp1;
    void *vp2;

    printf("ip1: pointer: %p, value: %d\n", (void *) ip, *ip);

    return 0;
}

Will give this a look and try to fix.

I hope this video illustrates the issue fine. :)

Just checking - are you using the same_window_errors config option? Maybe check it out and see if it's what you're looking for?

Sort of, is there an (automatic) option to use/open the respective buffer below the compilation output to keep the info of the compilations errors open? When running :Compile and :NextError e.g. while Netrw is open the info of the errors is lost. I guess it will be more of a convenience feature, than a bug. I would expect the temporary buffer to stay at the top and open, and :NextError to open the respecive buffer/location below.

I think retaning the focus on the buffer in which you run the command in combination with same_window_errors should fix the issue right there. Maybe adding that as a configuration option for that will improve the convenience (in my opinion). What do you think of that idea?

I see... So what you're looking for would be like the behavior of

Running :Compile with your command
Navigating back to the file explorer or some other buffer
Running :NextError there

Such that the top window shows the compilation buffer, and the bottom window shows the location of the error?

Actually, i'm looking for the option that the compilation buffer will stay open when i run :NextError, so if there are multiple compilation errors, i don't loose sight of whats the issue there. Since the buffer is opened vertically on top, i would expect :NextError to open the repective malicious code in the window below and keep the compilation buffer open at the same time.

Well, with same_window_errors, :NextError opens the error in the buffer which the cursor is currently on.
How about trying:
autocmd FileType compilation wincmd w

or
vim.api.nvim_create_autocmd("FileType", {
  pattern = "compilation",
  command = "wincmd w",
})

To not switch focus to the compilation buffer once its created? Try it and see if it fits what you're looking for?

Thats exactly what i was looking for. Thank you for the support!


Send bank_statement_evidence_pdf to CCMS as BSTMT document type

What
Send bank_statement_evidence_pdf to CCMS as BSTMT document type
BA confirms that this is the preferable document type to assign
for bank statement uploads. I had wrongly assumed that the
DocumentCategory#ccms_document_type, previously set for
bank_statement_evidence_pdf to be BSTMT, would be sent across
but it is a hardcoded value that is sent based on name.
Checklist
Before you ask people to review this PR:

Tests and rubocop should be passing: bundle exec rake
Github should not be reporting conflicts; you should have recently run git rebase main.
There should be no unnecessary whitespace changes. These make diffs harder to read and conflicts more likely.
The PR description should say what you changed and why, with a link to the JIRA story.
You should have looked at the diff against main and ensured that nothing unexpected is included in your changes.
You should have checked that the commit messages say why the change was made.

thanks @colinbruce - i have added a commit as i realised we were sending the wrong extension type

thanks @colinbruce - i have added a commit as i realised we were sending the wrong extension type

Looks good! Happy to let approval stand :+1:


Unable to save email address as username (since updating from 1.8.3)

Describe the bug
Since updating to the latest version (from 1.8.3 to 1.8.6) the plugin is not working anymore for me. It seems that I cannot save my email as username anymore... Was there a change in the regular expression? My email is something like<EMAIL_ADDRESS>I get the following error:
Failed to save add-on configuration, does not match regular expression ^([\w+\.]+@([\w-]+\.)+[\w-]{2,4})|(\+\d{5,20})$.

To Reproduce:
Steps to reproduce the behavior:

Go to the add-on
Click on 'Configuration'
Fill in all options (see my config below)
Click 'Save'
See error

Update Interval:
300
Babel Locale:
nl_BE
Timezone:
Europe/Brussels
MQTT Broker settings:
broker: core-mosquitto
port: auto_port
username: xxxx
password: xxxx

Volvo Configuration Options:
username<EMAIL_ADDRESS>password: xxxx
vin: xxxx
vccapikey:
  - xxxx
odometerMultiplier: 1
averageSpeedDivider: 1
averageFuelConsumptionMultiplier: 1

Expected behavior:
I expect the add-on to work like before, I was able to load all the details of my car before updating to the latest version
Screenshots:
If applicable, add screenshots to help explain your problem.
Version info:
Home Assistant: 2023.8.4
Supervisor: 2023.08.1
Operating System: 10.5
Add-on: 1.8.6

I need your real email address to check the ReEx validation.

I need your real email address to check the ReEx validation. Is this email address nearly the original<EMAIL_ADDRESS>
Yes, it includes the - and the .

created #98 with a possible fix

Should be fixed with version v1.8.7!. Thanks for your PR.

Yes, I'm able to save my configuration again! Thanks!


Update APIGroup to habana.ai

Signed-off-by: Fabien Dupont<EMAIL_ADDRESS>

Codecov Report

Merging #12 (9c0273d) into main (3e2dee1) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #12   +/-   ##
=======================================
  Coverage   72.00%   72.00%
=======================================
  Files           8        8
  Lines         643      643
=======================================
  Hits          463      463
  Misses        158      158
  Partials       22       22

Impacted Files
Coverage Δ

controllers/deviceconfig_controller.go
54.54% <ø> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Receiving a new trade offer not working?

I am using the default TradeOfferUserHander.cs and just trying to decline the offer instantly when the it receives a trade offer, in the log the notification is logged but nothing else is done, trade offer is not canceled and nothing is being logged. All I added was
Log.Success("TRADE OFFER RECEIVED IN TRADEOFFERUSERHANDLER.CS");
offer.Decline();

Kinda confused and would love some help.
If needed pastebins:
http://pastebin.com/aVFbiiQh - TradeOfferUserHandler.cs
http://pastebin.com/GjVs4bDu - Bot.cs
http://pastebin.com/2JdsneJK - settings.json

Are you using API key bound to that particular account?

I got it to work using SteamTradeOfferBot.

Does anyone use offer.Decline()?  Can we verify that it actually does work?
IIRC I've had some issues with this in my bot, but I never investigated to check if it's an issue with offer.Decline() or something else..

@BlueRaja check out https://github.com/waylaidwanderer/SteamTradeOffersBot/tree/SteamTradeOffersBot/SteamBot works almost out of the gate(minus setup)

Fixed in #981


Adding an Appropriate App Icon

App icon added, added all image files for android, and IOS.
also added some import statements which were causing errors and a package in pubspec.yaml file.

Thank you


Add support for offset_cc

Closes #159
It will be cleaner once the changes from #166 are merged

Rebased in order to make it readable (since the base branch changed after the PR)


Request for more details

I appreciate that your code can be used as a reference, but I don't know how to run them, could you please add more comments in the README file again? Thanks.

@qiwu000 Thank you for reporting this issue.
train.py is the main interface.

！！Thank you very much for your reply. I'll try to learn and do it！

I don't have a lot of foundation but would like to try to run your code and understand a little bit, can you give me some advice ? Thank you!


Combobox is not working for mobile device

Combobox is good for the desktop devices, but for mobile device it does not work, it does not open the select list

@soyabhussain21, Can you provide your configuration detail in which you are using the combobox component, like Select only, or Select with Autocomplete, MultiSelect etc. if possible can you provide your html code reference of the component.

@soyabhussain21, I further tested the component using the Salesforce Mobile App, specifically with the latest versions of iOS and Android, I have not encountered any issues with either the single select or multiselect variants. Some cosmetic changes are needed on the CSS side, which I will address later. However, there are no critical functionality errors or instances where the component is completely non-functional. To better assess your issue, I may require additional details. Please provide more information on where you tested the component and the configuration which you used.