Exec format error

Hello,
I am getting a "Exec format error" lately if I am trying to run Elite through the minimal ed launcher. The normal launcher does work and I tried several proton versions.
Launch options:
MANGOHUD=1 rxvt -e ./MinEdLauncher %command% /autoquit
Here is the log:
2022-09-25 17:23:03.214 +02:00 [DBG] User selected Elite Dangerous: Odyssey - FORC-FDEV-DO-1000 - edo 2022-09-25 17:23:03.333 +02:00 [INF] Launching Elite Dangerous: Odyssey 2022-09-25 17:23:03.485 +02:00 [DBG] Process: /home/arne/.local/share/Steam/steamapps/common/Elite Dangerous/Products/elite-dangerous-odyssey-64/EliteDangerous64.exe "ServerToken d0f...0d51cf...20c1c5...7f25e595ebe8f24eb9126 84c...fdc4de...201d00...efaf0a...966394...578175...f87324...195a64...894e53...fc8dd8...6ffe86...6dc5fc...116c91...bc49e6...c17111...4a3fed...f18269...e06c0c...39d33e...6fc79a...3e622b...2c8134...517c82...41c0a3...8700f2...48cfc6...f88f9c...42e4d8...e34220...1b30b2...425949...2fcde8...a7ee5d...3d1502...629d38...3da1de...7ba0d6...e98841...5df5a8...659827...c84ee9...b40b70...681fff...71215d...f5372c6 " /language German\\DE /steam /novr 2022-09-25 17:23:03.524 +02:00 [ERR] Couldn't start selected product: System.ComponentModel.Win32Exception (8): ErrorStartingProcess, /home/arne/.local/share/Steam/steamapps/common/Elite Dangerous/Products/elite-dangerous-odyssey-64/EliteDangerous64.exe, /home/arne/.local/share/Steam/steamapps/common/Elite Dangerous/Products/elite-dangerous-odyssey-64, Exec format error    at System.Diagnostics.Process.ForkAndExecProcess(ProcessStartInfo , String , String[] , String[] , String , Boolean , UInt32 , UInt32 , UInt32[] , Int32& , Int32& , Int32& , Boolean , Boolean )    at System.Diagnostics.Process.StartCore(ProcessStartInfo )    at System.Diagnostics.Process.Start()    at System.Diagnostics.Process.Start(ProcessStartInfo )    at MinEdLauncher.Product.run(FSharpOption1 proton, String args, RunnableProduct product)`

It's possible this is related to #60. Valve introduced a new process when launching via proton and it borked argument parsing. Can you try the latest CI build and let me know if it's still broken? If it is, can you post a full log entry?

Jup, thanks! That was it!
For those, who are reading this later:
You may need to do an
chmod +x MinEdLauncher
because the zip doesn't save permissions.


Error upon fresh import of plutus-metatheory into Agda project

Summary
Tried to use the UPLC agda from the metatheory and it failed at import.
Steps to reproduce the behavior

clone plutus repo
add name: Plutus to plutus-metatheory/Plutus.agda-lib
add absolute path to Plutus.agda-lib to agda libraries file
clone stand
add absolute path to standard-libary.agda-lib to agda libraries file
note that the name of the lib inside the agda-lib file is actually standard-library-2.1
create agda defaults file and add standard-library to it
create a new Agda project with the agda-lib file containing

include: src
depend: Plutus

create a new file src/HelloProof.agda containing

open import Agda.Primitive
open import Untyped.CEK

data _≡_ {a} {A : Set a} (x : A) : A → Set a where
  refl : x ≡ x

reflexive : {A : Set} {x : A} → x ≡ x
reflexive = refl

EDIT: commenting out everything but import Untyped.CEK didn't change the outcome.
10.  run agda on that file using vscode with the agda-mode extension
Actual Result
(...)/plutus/plutus-metatheory/src/Utils/List.lagda.md:109,47-110,79
_i_213 ≡ _k_215 !=< List A
when checking that the inferred type of an application
  _i_213 ≡ _k_215
matches the expected type
  List A

Expected Result
*All Done*
Describe the approach you would take to fix this
No response
System info
OS: MacOS Sonoma
Plutus (tried multiples with identical result):

commit @d1ef450 (latest)
tag <IP_ADDRESS>
commit @ec781723

We are building against version 1.7.3 of the stdlib, so it seems unlikely that it will work with version 2.1.

Thank you, that worked!


Update node-sass to 4.12.0

Fixes #719
The same three tests fail pre- and post-change, so this PR does not appear to have any impact.

LGTM

@dlmanning, could this be merged, please? Thank you!

After running system updates on my MacBook Pro over the weekend (macOS Mojave version 10.14.3), I was having a Node Sass error coming from gulp-sass when I tried to start my web server: Error: Node Sass does not yet support your current environment: OS X 64-bit with Unsupported runtime (67). A good old npm rebuild node-sass wasn't helping this time. I also tried to delete node_modules folder and re-install everything: didn't help either.
At the end, I realized that gulp-sass was not using the latest version of node-sass and using this fork, I was finally able to start my server as usual.
Thanks @simonua, can't wait to see this PR merged.

npm update node-sass is the non way of updating a transient dependency.
On Mon., 18 Feb. 2019, 7:28 pm Pierre-Michel Brown<EMAIL_ADDRESS>wrote:

After running system updates on my MacBook Pro over the weekend (macOS
Mojave version 10.14.3), I was having a Node Sass error coming from
gulp-sass when I tried to start my web server: Error: Node Sass does not
yet support your current environment: OS X 64-bit with Unsupported runtime
(67). A good old npm rebuild node-sass wasn't helping this time. I also
tried to delete node_modules folder and re-install everything: didn't
help either.
At the end, I realized that gulp-sass was not using the latest version of
node-sass and using this fork, I was finally able to start my server as
usual.
Thanks @simonua https://github.com/simonua, can't wait to see this PR
merged.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/dlmanning/gulp-sass/pull/720#issuecomment-464634051,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAjZWKldlEVHtge3t7rrMxLgmGHIaBrpks5vOmRAgaJpZM4YticH
.

@pmrotule, you're welcome. Your situation applied to me similarly on the Windows side. =P

hey @xzyfer really waiting for this! -> rgba, rgb and custom vars coming too
gulp-sass -> node-sass (even with 4.11 we need to wait until node-sass update to sass 3.56 with rgba vars) -> lib-sass
issue in node-sass

Would be great if this could get merged so we can get rid of the CSS import warning.

@dlmanning, just a ping to float this back to the top of your inbox, good sir. It would be great if this could be merged and released, please.
Thank you!

npm update node-sass
On Tue., 26 Mar. 2019, 3:56 am Simon Kurtz<EMAIL_ADDRESS>wrote:

@dlmanning https://github.com/dlmanning, just a ping to float this back
to the top of your inbox, good sir. It would be great if this could be
merged and released, please.
Thank you!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dlmanning/gulp-sass/pull/720#issuecomment-476287504,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAjZWN5QlOBFo3wwrZHHTzec0-qpD-5kks5vaP_CgaJpZM4YticH
.

I updated the PR with the latest version of node-sass released a few days ago that now supports NodeJS 12. @dlmanning, would be really, really great if this could be merged and released. Pretty please.

My project really needs this as well. Thank you so much!

As @xzyfer already said twice, this PR isn't necessary. The solution is to simply npm update node-sass in your project.

@TheDancingCode: it’s better to have safe defaults than hope people find that default buried in an issue tracker after hitting a problem because they mistakenly assumed that npm install gulp-sass would do the right thing.

True, not necessary, and the suggestion helps, but I agree with good defaults and good package maintenance. Such is a team sport.

If you are using yarn to install your npm packages, you can use this solution to force installing node-sass v4.12.0.

This is not needed because of how SemVer works. You can always update the transient node-sass with npm update node-sass if you wish.
nodesource.com/blog/semver-tilde-and-caret


[BUG] Cannot control log level in bpftime attach at least

I tried to use the system environment variable to control the log level of bpftime but it won't work, at least for the bpftime attach command:
$ SPDLOG_LEVEL=Error bpftime attach `pgrep victim`
[2024-01-27 15:11:15.313] [info] Injecting to 622200
[2024-01-27 15:11:15.322] [info] Successfully injected. ID: 1

In the current design, you should control the log level of the attached process at the process side, since the library was injected into process, so variables were read at the process side


firebase pipeline initial commit

Tell us what you do here

[x] contributing new pipeline (please create an issue here and link it below)
[ ] implementing verified pipeline (please link a relevant issue labelled as verified pipeline)
[ ] fixing a bug (please link a relevant bug report)
[ ] improving, documenting, customizing existing pipeline (please link relevant issue)
[ ] anything else (please link an issue or describe below)

Relevant issue
issue #
link
More PR info

WIP, currently only covers realtime database, next would be cover crashlytic.

@Anggi-Permana-Harianja looks like pyproject toml is incorrectly merged and still the test data is in pipeline code. we reorganized the pipelines code and I've moved your fork as our branch. https://github.com/dlt-hub/pipelines/tree/rfix/firebase-attempt
we'll continue to work on it later.


[REQUEST] Add the ability to view all reward items for ongoing missions

          Can you add the ability to view all reward items for ongoing missions?

Originally posted by @VictorChan7 in https://github.com/DorielRivalet/mhfz-overlay/issues/47#issuecomment-1411547464

For this feature to be added, it might be needed to find memory addresses related to the reward item ID along with its chance, which has yet to be found.


Object Localization and Detection with new my own dataset

I trained a my own dataset to detect car and i tested on Ultra96 via PYNQ 2.4 . The result was good but when I tried to follow the sample code on road-traffic , my code could not bound box object (car) good enough. And I want to know how I can modify your code and how to bound box object as well. Finally I want to bound box the car fast enough to be able use for real time .

This is my bounding box  ( tried to use non-maximum)
Sorry about my English and my unrelated content!
Thanks all!

Hi,
the network you trained is for image classification, not for detection. In out notebook examples, we show how it's possible to use an image classification neural network to detect objects, but the accuracy heavily depends on how many tiles (portion of the original image) are fed to the classification neural network.
In our example we used for s in [64,96]: meaning we take 64x64 and 96x96 pixel tiles, with a distance from each other of stride = s // 4 1/4 of the image size.
These values have to be carefully selected, depending on the smaller and the bigger car you want to identify in the original image.
Please note that the road-sign neural network has been trained with an extra "junk" class, as explained here, using background images to be classified as a different class. This approach actually increased the perceived accuracy of the object detection task.
Hope this helps

Thank you so much , I think I have some ideas to do and improve my junk class. Your advice is exactly what i need. This help me a lot !
Once again, Thanks you!


ci: use dependabot on GH actions

Inspired by: https://github.com/ipdk-io/ipdk/pull/146
Co-authored-by: Boris Glimcher<EMAIL_ADDRESS>Signed-off-by: Kyle Mestery<EMAIL_ADDRESS>

This will ensure the Dependabot pushes PRs once a week to update the versions of our GitHub Actions we use.

#38 needs to be merged first so this one can pass the CI.

@mestery I've merged #38. I like this idea - we had also been using dependabot at ONF for some projects. It might require some additional configuration of GHA, but let's see.

@mestery I've merged #38. I like this idea - we had also been using dependabot at ONF for some projects. It might require some additional configuration of GHA, but let's see.

There should not be any additional GHA configuration required, I've added this to both IPDK and ipdk-plugin without requiring changes.
I also rebased this and it's good to merge now!


chore: upgrade docusaurus to 2.3.1 & react 18.2

Upgrade these dependencies, so we can support deep-linking in Tabs.

Thanks for looking at Docusaurus upgrade! 👍
There are several thing which need to be done,, beside bumping the version in package.json:

we need to regenerate swizzled components, and re-apply our custom modifications, files are located in https://github.com/facebook/react-native-website/tree/main/website/src/theme, and there is a guide how to do this - https://docusaurus.io/docs/next/swizzling
we need to verify if custom classes used in our SCSS files still exist, there is no other way than going through various pages and looking for regressions in comparison to the currently deployed PROD instance

Broken thing I have spotted on quick check:

loading showcase page crashes Docusuarus - https://deploy-preview-3618--react-native.netlify.app/showcase
announcement bar (at top) looks unstyled now

Push for the Showcase issue coming soon.
I wrote some visual diff tooling to give me a better chance of catching these things.

@Simek I think this LGTM now.  When you have a moment would you mind looking over it again.  I've incorporated the swizzle changes and fixed the issue with showcase.

Thank you for all the work put into the upgrade! 🙏 Besides the comment about Showcase issue, this LGTM! 👍
Also, it looks like we have broken content container on versions page - https://deploy-preview-3618--react-native.netlify.app/versions - but surprisingly, this already seems to be an issue for PROD deployment, so we can address this in a separate PR.

I'm spamming netifly atm because the issue doesn't reproduce on my locally hosted version.  Apologies for the noise.


The endpoints is repeated in the case of the app is deployed to multiple clusters

Describe the bug
➜  catalog git:(main) ✗ vela status addon-traefik -n vela-system -p
E0427 19:41:24.165999   17465 handler.go:282] no matches for kind "Ingress" in version "networking.k8s.io/v1beta1"collect ingres by helm release failurehelmReleasetraefiknamespacevela-systemcluster
E0427 19:41:24.446231   17465 handler.go:282] no matches for kind "Ingress" in version "networking.k8s.io/v1beta1"collect ingres by helm release failurehelmReleasetraefiknamespacevela-systemclusterbj
+---------+-----------+-----------------------------+-----------------------+
| CLUSTER | COMPONENT |  REF(KIND/NAMESPACE/NAME)   |       ENDPOINT        |
+---------+-----------+-----------------------------+-----------------------+
|         | traefik   | Service/vela-system/traefik | http://*.*.*.*  |
|         | traefik   | Service/vela-system/traefik | https://*.*.*.*  |
| bj      | traefik   | Service/vela-system/traefik | http://*.*.*.*   |
| bj      | traefik   | Service/vela-system/traefik | https://*.*.*.*  |
+---------+-----------+-----------------------------+-----------------------+

The service type is Loadbanlacer.
KubeVela Version
1.3.2
Cluster information
v1.22.3-aliyun.1

What's your expected behavior? This should be normal as you deployed duplicated in these clusters.

I hid the IP address, the IP should be different but currently, it is same in two clusters .


Add in "function link" handler.

This will work in a similar way to the lambda handler, but instead of including the function code within the JSON, it will just reference a python function.
{   "New Header One": {     "old_column": "Old Header One",     "default": "DEFAULT FOR ONE",     "lambda": "lambda a, b: str(a.get(b.get('old_column'))).lower()"   },   "New Header Two": {     "old_column": "Old Header Two",     "default": "DEFAULT FOR TWO",     "funlink": "custom_python_file_or_module/custom_function"   } }
It would then split the string by the forward slash, and import it using:
https://docs.python.org/3/library/importlib.html
Finally, it would run the imported function, in a similar way to the lambda one, with the following variables:
input_line - the current INPUT line
previous_input_lines - all the previous input file lines
current_output_lines - the current output dictionary, not a copy, but the actual one, so these functions are able to view values from, and make changes to previous entries.
item - the current column item in the JSON config.
It would output both the individual item it is processing, as well as the current_output_lines, which is to be updated if there are any changes.

If you want the imported function to be run with previous_input_lines and current_output_lines, then I think it would change the way the code is organized. For example, I think it would be easier if the JSON config is structured this way:
{ 	"original header 1": { 		"new header": "NEW HEADER 1", 		"default": "A default value", 		"lambda": ... 		"funlink": ... 	} }
Also, does if a Lambda and Funlink are specified in the config, should both be ignored? I think the funlink should have a higher order of precedence.

It's structured this way as the output file can contain a column with multiple source columns, for example "full name": {"lambda": "...first_name + last_name..."}
There shouldn't actually be an "old column" mentioned at all in the examples above, that's my bad, though perhaps this would work:
Go through each possible type of value in this order, and if it returns None or an exception, try the next one.

funlink
Lambda
Old column
Default

Alright. I'm currently adding some tests for my funlinks implementation. Hopefully I'll be done tonight.


BLEDOM device no longer being detected

I have a cheap BLEDOM device that was working.
I though it may be someone the recent updates but I downgraded to version 0.0.4 which I was sure was working but the device was not detected by the plugin.
Was there a recent change in HA that limited this addon?
The specific device seems to be detailed on this Github page https://github.com/arduino12/ble_rgb_led_strip_controller

I think you need install new "bluetooth" integration :
https://www.home-assistant.io/integrations/bluetooth/
Yes HA have important changes in bluetooth devices
Luck!

After some new updates to HA I am able to complete the connection and use the device.
I now have a problem where every reboot I have to reconnect the Bluetooth device.
It was working just fine before.  Thank you for confirming that this is a HA issue.  I will have to find a way to report the bug to HA.


Changing URL programmatically

In a typical context of a CRUD, I would like, after a submit success of a form, to change the URL and navigate to page list.
Example:
<edit-attrezzo>
...
<form onsubmit={ submit }>
...
<button type='submit'>Save</button>
</form>
<script>
export default {
...
async submit(e) {
e.preventDefault()
const data = ...
await StoreService.saveAttrezzo(data)
route('/attrezzi')
}
...
}
</script>
</edit-attrezzo>
This does not work for me. In Riot v3, I got it in that way.
Is it possible to navigate to an URL programmatically? I can't get it.
Thanks in advance

The Riot.js router is built on the top of the browser standards. You should be able to change your routes simply calling history.pushState (in your case history.pushState({},'','/attrezzi'))

Hi, thanks for your help. I modified as you suggested:
<edit-attrezzo>
... html ...
<script>
...
         async submit(e) {
             e.preventDefault()
             await StoreService.saveAttrezzo(this.state.item_id, this.state.descrizione, this.state.icona)
             history.pushState({},'','/attrezzo')
}
</script>
<\edit-attrezzo>
In dashboard.riot (that is the container for main content) I have:
               <route path="/attrezzo">
                    <anagrafica-attrezzi></anagrafica-attrezzi>
                </route>
                <route path="/attrezzo/create">
                    <edit-attrezzi item=""></edit-attrezzi>
                </route>
                <route path="/attrezzo/edit/:item">
                    <edit-attrezzi item="{ route.params.item }"></edit-attrezzi>
                </route>
When I submit the form, the url in browser actually changes to /attrezzo, but the route is not activated and nothing happens to main content.
I can't understand the reason, since navigation by clicking on anchor tags works.
Thanks in advance

I am sorry you need to push the path to the router as well since the popstate event is not automatically triggered.
This code should do the trick
import { router } from '@riotjs/route'

const navigateTo = (path) => {
  // update the browser url
  history.pushState({}, '', path))
  // notify the path change to the riot router
  router.push(path)
}

Note that this code is code is only needed for the html5 history navigation
In future releases the above code will be no longer needed since we will rely on https://developer.mozilla.org/en-US/docs/Mozilla/Add-ons/WebExtensions/API/webNavigation/onHistoryStateUpdated to detect the navigation transitions

thank you, now it works!


Support non ansi character sets when converting hex codes to characters

The parser currently only supports the ansi charset, which leads to wrong hex code conversion when using the mac charset. For example the codes 0xD4 (212, left quote) and 0xD5 (213, right quote) actually correspond to ansi characters 145 and 146 which leads to text with wrong characters.

I know, code page support hasn't been implemented.  Do you have a patch and/or simple test document by chance?

I'm currently using http://www.scottlynch.us/files/rsurs-sample.rtf for testing my implementation and all characters are displayed correctly with the patch.


Using the dnn sample get a bug

when i use the sample caffe_googlenet.cpp I get a error,#136,the forward func,
when run the layerInstance = LayerFactory::createLayerInstance(type, params);i got a null ptr, the error is OpenCV Error: Unspecified error (Can't create layer "conv1/7x7_s2" of type "Convolution") in cv::dnn::LayerData::getLayerInstance, file E:\opencv_contrib-3.1.0\modules\dnn\src\dnn.cpp, line 122,so if who can give me some help, deeply grateful.
btw,I use vs2013 and win10.

add following line at the beginning of main function.
cv::dnn::initModule();


default output.log location should be /root

Summary and Scope
default output.log location should be /root
Issues and Related PRs
List and characterize relationship to Jira/Github issues and other pull requests. Be sure to list dependencies.

Resolves https://jira-pro.its.hpecorp.net:8443/browse/CASMINST-4622
Change will also be needed in <insert branch name here>
Future work required by [issue id](issue link)
Documentation changes required in [issue id](issue link)
Merge with/before/after <insert PR URL here>

Testing
List the environments in which these changes were tested.
Tested on:

<development system>
Local development environment
Virtual Shasta

Test description:
How were the changes tested and success verified? If schema changes were part of this change, how were those handled in your upgrade/downgrade testing?

Were the install/upgrade-based validation checks/tests run (goss tests/install-validation doc)?
Were continuous integration tests run? If not, why?
Was upgrade tested? If not, why?
Was downgrade tested? If not, why?
Were new tests (or test issues/Jiras) created for this change?

Risks and Mitigations
Are there known issues with these changes? Any other special considerations?
Pull Request Checklist

[ ] Version number(s) incremented, if applicable
[ ] Copyrights updated
[ ] License file intact
[ ] Target branch correct
[ ] CHANGELOG.md updated
[ ] Testing is appropriate and complete, if applicable
[ ] HPC Product Announcement prepared, if applicable

Need a PR to main as well.

Should this really be $HOME? Roots home is always /root, but would this ever needs to support a future use case of being ran as a different user?

Need a PR to main as well.

https://github.com/Cray-HPE/docs-csm/pull/1615


Tenancy for Laravel considers www. as a tenant

Bug description
The package considers any prefix before the main domain as a tenant and hence throws the error Tenant could not be identified on domain on TenantCouldNotBeIdentifiedOnDomainException.  This error doesn't appear on localhost till i deployed to production server.
Steps to reproduce
After creating tenants, I tried accessing tenants from the browser i observed it has a prefix of www.
This then throughs an error www.tenant not identified on domain
Expected behavior
It is expected that the package should ignore the www. prefix before the tenant or accommodate without through the tenant not found error.
Laravel version
8.2.0
stancl/tenancy version
3.1.0

This is meant to be handled at the webserver level by redirecting www. domains to the non-www version.
That said, in v4 we might add some behavior to ignore a configurable list of subdomains.

Alright, I was able to fix this from the web server by creating the correct DNS records A record for * and www and also forcing redirect from www. to non-www through the nginx server config.  I also installed an SSL certificate and the problem was solved.
But I still think this should be looked into to ignore the www.
This is an amazing package.

Will keep this open so that I remember to revisit this 👍🏻

I am facing same issue as well, can you please share a record details @Guzbyte-tech
i am using Cloudflare

I ended up creating a config file will 'blocked_subdomains' => [] and 'blocked_domains' => [] array. During registration, for instance, the function checks against the blocked_subdomains array in the config file and if there is a match, it fails.

nginx config file
`
server {
server_name example.com www.example .com;
root /var/www/example.com/public;
add_header X-Frame-Options "SAMEORIGIN";
add_header X_XSS-Protection "1; mode=block";
add_header X-Content-Type-Options "nosniff";
index index.html index.htm index.php;
charset utf-8;
location / {
try_files $uri $uri/ /index.php?$query_string;
}
location = /favicon.ico { access_log off; log_out_found off; }
location = /robots.txt { access_log off; log_out_found off; }
error_page 404 /index.php;
location ~ .php$ {
fastcgi_pass un ix:/var/run/php/php7.4-fpm.sock;
fastcgi_index index.php;
fastcgi_param SCRIPT_FILENAME $realpath_root$fastcgi_script_name;
include fastcgi_params;
}
location ~/.(?!well-known).* {
deny all;
}
}
`

i will try it..

any update about v4?

For updates about v4, follow our Discord.


Enable HybridApp test builds in App repo

Problem
During PR review we perform test only against ND while the users will use Hybrid App. It prevent us from finding possible bugs and problem users will face.
P/S from Jan here
Solution
Let's configure our CI/CD so it's possible to trigger AdHoc build of Hybrid App from ND repo.

Hello, I'm from SoftwareMansion and I'll work on this

@Julesssss I managed to successfully add new build type for Android and new scheme for iOS - here's draft PR.
The next step requires action from someone who has access to Firebase console and developer profile because I think it's time to generate new google-services.json, GoogleService-info.plist and create new provisioning profile.
I think that bundle id's are required for that configuration so I propose:
iOS:
com.expensify.expensifylite - Release
com.expensify.expensifylite.debug - Debug
com.expensify.expensifylite.adhoc - AdHoc
Android:
org.me.mobiexpensifyg - release
org.me.mobiexpensifyg.debug - debug
org.me.mobiexpensifyg.adhoc - adhoc
Please let me know if I should provide some additional info and when this can be accomplished

Hey, sorry for the delay! I will make time for this as soon as possible. No extra details required yet.

No problem, I've started to work on other things in the meantime :)

Sorry for the delay here, I'll have time to review tomorrow!

Hey @jacobkim9881, I can generate the Firebase apps. I'm thinking we should do .dev instead of .debug to align with App, and to make it clear that these are dev builds.
Let me know if you see any problems with this

Hey @jnowakow, I can generate the Firebase apps. I'm thinking we should do .dev instead of .debug to align with App, and to make it clear that these are dev builds.
Let me know if you see any problems with this

I have configured the iOS/Android apps in Firebase (using .dev). I haven't yet added the Android SHA certificate fingerprints, so Google SSO and other things won't work. But I think that is fine.
@jnowakow I will DM you the google services file.

@Julesssss I .dev instead of .debug is great for me. Thanks for taking care of it!

Hey @jnowakow, we've been chatting 1:1 but could you share an update on this issue please?

Sure!
Last update is that everything seems to work fine with provided firebase configurations. In order to proceed with iOS I'll need new provisioning profile, notification service profile and certificates. For Android I think I don't need any additional set up so I'll focus right now on creating GH workflow

@Julesssss here are the current changes:
old dot
new dot
I think everything is in place to test if android adhoc build is working fine

Nice, I will take a look.
I assume we need to merge Mobile-Expensify first? If so can we put a hold on the App PR? And mark Mobile-Expensify as ready for review. I can also try to run a test build against this branch if we are ready.

Thanks for review! I've answered the comments and addressed those two little TODOs.
In general yes, I think we would have to merge OD first but iOS is not 100% ready (I think nothing should break). I'm also afraid that we have to merge this PR in order to be able to run this workflow

btw I think there is a possible improvement to this workflow to enable people to specify old dot's commit they want to build app from. It would enable building hybrid app not only with ND changes that are not merged but also with OD changes which weren't yet merged to main

Okay, I will try running the action when I have a moment.
The improvement could be good, let's discuss that after the initial PRs.

Mobile PR merged.

Attempting to test the first of the workflows here.

Initial 2 PRs merged today. We'll need to await deployment to prod for testing...

^ waiting for prod deployment in order to test above PRs.

Build seemed to be successful, we has a post build permission error with build artifacts.

Yeah, app build correctly but there was problem with uploading. I'll take a look why this path is wrong

Second PR was merged and we're running another attempt here.

@Julesssss do you have some timeline when I can get provisioning profile, notification service profile and certificates in order to proceed with iOS?

Hey, I can get the profiles but can you just remind me whether you will need to register your phone and profile? We are currently limited and being ignore by Apple.

I hope registering my phone and profile will not be necessary

Great thanks. I sent you the provisioning profiles and I need to think about how we provided other certificates.

I added/updated the AdHoc iOS profiles in 1Password. Shared names with Jan.

Initial iOS PR was merged, once deployed we can kick off a test build.

Error with Android HybridApp build

Sounds similar to a fix we made here

Error with iOS HybridApp build

OldApp_AdHoc with 1Pass filename

Hi @jnowakow, yep all three of these work correctly for me. Modifying the names causes errors.

Hmm then I don't know where the problem is 🤔 I copied those commands from workflow so I don't know where I could do the typo

@Julesssss
I see plan for proceeding with this issue as follows:
Merge PR with changes for iOS in OD. I've merged newest main and adjusted it to new repo structure. I've tested it locally and all three schemas seems to work fine.
Figure out why 1Password doesn't see apple related files that you added recently. I'm afraid I need you help here because I don't have access to 1Pass. Can you check if those files are added in the same way as other files (eg. upload-key.keystore for Android)? Maybe they have to be exported or something like that?
Figure out why validateActor is failing. I suspect it was some random issue and if we rerun the workflow this step should pass. This step wasn't modified in the meantime and was working fine before. It's also the same as in other workflows (eg. here) and there it isn't failing. If we rerun the workflow and is keeps failing then I have to investigate it.
I think that on ND side almost everything is ready. There are small corrections to do like fixing the typo in workflow name or running dev scheme when launching from terminal.

Merge PR with changes for iOS in https://github.com/Expensify/Mobile-Expensify/pull/13297

Reviewing now 👍

Can you check if those files are added in the same way as other files (eg. upload-key.keystore for Android)? Maybe they have to be exported or something like that?

Sure, I am rushing to finish some things before going OOO but I can help with that in the new year

Figure out why validateActor is failing. I suspect it was some random issue and if we rerun the workflow this step should pass

Also sounds good


Updated validations.py python script.

Fixed the behavior of validate_user function in validations.py
Closes: #1
Fixes #1

It's a good idea to open an issue first for discussion.

[ ] Tests pass
[ ] Appropriate changes to README are included in PR

@googlebot I signed it!


Refactor/Saga to update modules and move delete modules of sem to backend

Updating modules seem to be a repetitive function (refactor to 1 saga)
Move deleting all modules of sem to backend - backend always have most updated information

#71


mitogen_sudo connection plugin and friends are the wrong plugin type for get_option('become_...')

ansible_mitogen.plugins.connection contains 3 "connections" that aren't connections in the Ansible world view

mitogen_doas
mitogen_sudo
mitogen_su

In Ansible land these would be become plugins. This wasn't a problem, but may become a problem now. I'm moving the PlaybookSpec to Ansible's plugin options API, but the options become_exe and become_flags are associated with become plugins and these plugins are connection plugins.
As a result the tests in tests/ansible/integration/stub_connections/all.yml fail with
PLAY [integration/stub_connections/mitogen_sudo.yml] ***************************

TASK [include_tasks _raw_params=../_mitogen_only.yml] **************************
Monday 28 October 2024  21:22:28 +0000 (0:00:00.372)       0:09:28.173 ********
included: /home/runner/work/mitogen/mitogen/tests/ansible/integration/_mitogen_only.yml for target-centos7-1, target-centos8-2, target-debian9-3, target-debian10-4, target-debian11-5, target-ubuntu1604-6, target-ubuntu1804-7, target-ubuntu2004-8

TASK [meta _raw_params=end_play] ***********************************************
Monday 28 October 2024  21:22:28 +0000 (0:00:00.159)       0:09:28.333 ********
skipping: [target-centos7-1]

TASK [custom_python_detect_environment ] ***************************************
Monday 28 October 2024  21:22:28 +0000 (0:00:00.012)       0:09:28.345 ********
An exception occurred during task execution. To see the full traceback, use -vvv. The error was: AttributeError: 'NoneType' object has no attribute 'get_option'
fatal: [target-centos7-1]: FAILED! =>
  msg: 'Unexpected failure during module execution: ''NoneType'' object has no attribute ''get_option'''
  stdout: ''
An exception occurred during task execution. To see the full traceback, use -vvv. The error was: AttributeError: 'NoneType' object has no attribute 'get_option'
fatal: [target-centos8-2]: FAILED! =>
  msg: 'Unexpected failure during module execution: ''NoneType'' object has no attribute ''get_option'''
  stdout: ''

see #1172


Add: theme.json to Theme Scaffold

Description of the Change

This PR introduces a barebones theme.json file to the theme scaffold.
Alternate Designs

none
Benefits

Controling aspects of a theme through the theme.json file will only become more prevelent in core. Therefore we should leverage it in our scaffold also. Since adding the theme.json has some sideefects:

There are some features that are enabled or disabled when a theme has support for a theme.json file:

The template editor is enabled.
The default layout and margin styles for themes are not enqueued and the new layout options are enabled instead (see related dev note for layout).
The inner div for the group block (wp-block-group__inner-container) is removed.
The default font-family styles for the editor are not enqueued.

https://make.wordpress.org/core/2021/06/25/introducing-theme-json-in-wordpress-5-8/

We've found that remediating these sideeffects when adding theme.json to an already existing project always is more of a pain. So we should start now to have a barebnes version of it in the scaffold to work with it from the beginning of a project.
Checklist:

[x] I have read the CONTRIBUTING document.
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my change.
[ ] All new and existing tests passed.

Applicable Issues

Closing: #57
Changelog Entry

Added theme.json to theme scaffold
Removed DropCap option form Core Paragraph Block
Removed Core Block Patterns by default
Removed Color options from Core Button Block
Removed Block Based Template Editor Support by default

@joesnellpdx as discussed in our 1on1 this is ready to be merged. I don't have the privileges to merge it myself so pinging you here.

@nicholasio
I want to get this into our scaffold. Adding theme.json is a breaking change in my mind. It doesn't look like we are using semantic versioning. Should we? Or should we just go ahead and get this merged into trunk?

@nicholasio
I want to get this into our scaffold. Adding theme.json is a breaking change in my mind. It doesn't look like we are using semantic versioning. Should we? Or should we just go ahead and get this merged into trunk?

We don't publish this anywhere, the way project teams have been using this is to just git clone. Only new projects will benefit wit this change so it should be fine to introduce breaking changes like this.


The pretrained weight

we want to built a custom detector  for our dataset.
we use the Mask-RCNN as the benchmark. we use GPU =1 、 batch_size= 6 、opetimizer = SGD、 learning rate = 0.02、max_epoch = 60 as the  training parameters.
we find that whether or not  the pre-trained weight on imagenet was used had a significant impact for the experimental results.
There are 20 percent gap on the map50  and AP50 values in bbox and segm task.
So  what we should do in the next step.  the custom models  do not have a proper pre-trained weights.

The default config of Mask-RCNN with ResNet50 freezes the BN layers' mean&std and the first backbone stage.
    backbone=dict(
        type='ResNet',
        depth=50,
        num_stages=4,
        out_indices=(0, 1, 2, 3),
        frozen_stages=1,  # freeze the first backbone stage, set to -1 means not freeze
        norm_cfg=dict(type='BN', requires_grad=True),
        norm_eval=True,  # freeze the BN layers' mean&std, set to False means not freeze
        style='pytorch',
        init_cfg=dict(type='Pretrained', checkpoint='torchvision://resnet50')),

If you want to train with your own model, you should not freeze the parameters.
Also, you need longer training epochs if you do not have a pretrained weight.
You can refer to configs/scratch/README.md and the paper Rethinking ImageNet Pre-training

i am very appreciate for you response,

The default config of Mask-RCNN with ResNet50 freezes the BN layers' mean&std and the first backbone stage.
    backbone=dict(
        type='ResNet',
        depth=50,
        num_stages=4,
        out_indices=(0, 1, 2, 3),
        frozen_stages=1,  # freeze the first backbone stage, set to -1 means not freeze
        norm_cfg=dict(type='BN', requires_grad=True),
        norm_eval=True,  # freeze the BN layers' mean&std, set to False means not freeze
        style='pytorch',
        init_cfg=dict(type='Pretrained', checkpoint='torchvision://resnet50')),

If you want to train with your own model, you should not freeze the parameters.
Also, you need longer training epochs if you do not have a pretrained weight. You can refer to configs/scratch/README.md and the paper Rethinking ImageNet Pre-training

i am very appreciate for you response,


testit package needs installing

when i type make in the analysis subdir i get an error that testit is not installed.
Probably should include a list of packages they need to install in the instructions.
(And minimize the number that are required...).
Quitting from lines 60-61 (single-cell-pca.Rmd)
Error in library("testit") : there is no package called 'testit'
(Maybe this example can be simplified so that testit is not required?)

The assert function in testit was originally used in the PCA functions to generate a message in case of an error. I removed from function.R any dependency on testit.


example folder needs clean up as there are scripts which do not follow the location guidelines.

I will create clean up commit later to move scripts which are supposed to be placed under ./examples/client with instructive README

I was also thinking of doing something similar but it wasn't always easy to draw the line between the two - IIRC some of the 'examples' don't use either of the interfaces.
Perhaps we should use something like /examples/scripts for all maintained scripts, /examples/client for demo/ true examples and /examples/deprecated-hubinstance for everything else?

Agree with the ideas for holding the scripts which are using Client class in /examples/client and other ones which are using HubInstance class in /examples/deprecated-hubinstance. What other scripts are suggested to move to /examples/scripts? Maybe scripts which are using neither Client nor HubInstance? Besides that, it sounds like difficult to me to draw the line between 'maintained' and 'not maintained' apart form the used hub classes (Client or HubInstance).

As you say, not maintained is essentially anything using HubInstance.
I think ideally the base /examples directory should only contain the folders:

/examples/client

/examples/offline-scripts

/examples/deprecated-hubinstance

Everything else that is currently there can be classified under one of these.


Provide actionable info on what is executing, and any stderr output

... from the child process. My intention is to make problems like https://github.com/cypress-io/cypress/issues/20774 easier to debug from the user's standpoint.
For example. with these changes under Gnome 3, I click on "View App Data", and I see this output:

[jon@archlinux cypress-repro-issue]$ ./node_modules/.bin/cypress open
[5401:0522/215836.661604:WARNING:vaapi_wrapper.cc(586)] VAAPI video acceleration not available for swiftshader
[5401:0522/215836.662568:ERROR:gpu_init.cc(453)] Passthrough is not supported, GL is swiftshader, ANGLE is
Spawning: /home/jon/.cache/Cypress/9.6.1/Cypress/resources/app/node_modules/opn/xdg-open /home/jon/.config/Cypress/cy/production
Error info: /home/jon/.cache/Cypress/9.6.1/Cypress/resources/app/node_modules/opn/xdg-open: line 540: gnome-open: command not found
Error: Exited with code 4
at ChildProcess. (/home/jon/.cache/Cypress/9.6.1/Cypress/resources/app/node_modules/opn/index.js:92:13)
at Object.onceWrapper (node:events:514:26)
at ChildProcess.emit (node:events:394:28)
at maybeClose (node:internal/child_process:1067:16)
at Socket. (node:internal/child_process:453:11)
at Socket.emit (node:events:394:28)
at Pipe. (node:net:672:12)
Error: Exited with code 4
at ChildProcess. (/home/jon/.cache/Cypress/9.6.1/Cypress/resources/app/node_modules/opn/index.js:92:13)
at Object.onceWrapper (node:events:514:26)
at ChildProcess.emit (node:events:394:28)
at maybeClose (node:internal/child_process:1067:16)
at Socket. (node:internal/child_process:453:11)
at Socket.emit (node:events:394:28)
at Pipe. (node:net:672:12)

Feel free to provide any suggestions on a better approach to resolving this issue, I am all ears. I left some additional background information here.

I'm looking at alternative approaches to fixing this, based on discussion over on the cypress repo. Closing this for now.


Edit folder name in admin

For example deleting the h in home makes the cursor to jump to the end of the field.

This issue was moved to getgrav/grav-plugin-admin#144


Composable envs on modern Nickel

This is a merge of #33 and #34 (since they were quite heavily conflicting). Could either merge this directly or cherry-pick this into one of them to make it possible to merge them sequentially

It's strange, NixString contract doesn't seem to be updated in this version.

(This part e.g.: https://github.com/nickel-lang/nickel-nix/pull/37/commits/78fe382e0cac3eacbc481e4d4ec9e8268f72d00f#diff-c0c73e6da72ae44caaf6f8fc765eb14c59fc836937c792188e59825282fd9b86R143)

Mh, I might have messed something up in the merge :thinking:

Dan's PR has been merged, and we still have to cleanly adapt the code for the new symbolic string structure. Will do that from a clean state.


DKPY2 fixes to getting started, running examples, and developer setup…

This basically is first draft of changes to getting started, along with all the other minor link fixes an fixes to docs that made sense following the changes and which fix any bugs that showed up in links and rendering during the build.
The getting started section is a lot simpler and I removed a lot of dependencies. Worth checking. I also removed the specific "running an example" section and instead now point to the examples section for that.
This does need a basic review.

Rebased to master.

looks good but has some git conflicts that need to be resolved, possibly squash this commit afterwards too

Yeah the conflicts were probably me :) the only diff should be <<<OLD STUFF WITH "droneapi" replaced with "dronekit" ===== HAMISH'S CHANGES >>>>

to be fair sphinx makes it really hard to understand what's going on

No guilty (me or sphinx). I'll try fix this up on Monday.

@tcr3dr @mrpollo I removed that rubbish. Admitting guilt  :-(  I made sure that this also refers to dronekit rather than droneapi, and removed the numpy dependency.
Ready to merge (though I will hopefully add further updates to examples, depending on how testing of other fixes goes).


Greatly improve styling

Greatly improves the visual design of all of the current pages by building off of the lightweight Wing CSS library. Here's a screenshot of the new application page in action:
This PR based off of PR #13 (OAuth authentication) and therefore #13's commits appear in the log below. #13 should be approved and merged before this is merged.
Closes #3.

please merge oauth first and then rebase, thanks!

Thanks! Merging this in now


[Feature-Request] Show Header in Column

Hi,
i'd like to visualize whether a mail comes from inside or outside our organization based on the headers our mail gateway adds to the mails. I think this would be a good way to prevent phising attacks.
One way to achive this is to show these specific headers as column in the mailbox.
It would be great if you could implement this.

Is there a particular header you're wanting to expose? and in which table do you want to see it?

Is there a particular header you're wanting to expose? and in which table do you want to see it?

I want to choose the header(s) Outlook should show. In my case there are special headers my mail gateway adds.
Here's an example:

It sounds like you're asking for MHA to reach into the Outlook UI and modify the columns Outlook shows. This would not be possible.


WIP: Added option to specify ADs in node pools

I haven't been able to test it yet, as I'm still working on getting a test environment up, but in the meantime the maintainers can tell if there are issues with the documentation or the commit message.

The node_pools argument gained a new optional key placement_ADs.
This key can be popolated by a list, like:
placement_ADs = ["sQTi:EU-FRANKFURT-1-AD-1"]
to choose the ADs the pool will deploy nodes to.
Default remains to deploy to all ADs of region.
Fixes: #515
Signed-off-by: Andrea Stacchiotti<EMAIL_ADDRESS>

@12345ieee Lets have a separate variable for this since the nodepool variable already have many keys and in future if choice is given for FD in single AD it can be added in the new variable and other options like capacity reservation etc. Another suggestion would be to use datasource to fetch the AD values instead of finding the value and hardcoding it  . For example: placementconfig = {np1 = [AD-1]} and filter the datasource to get the value . May i know what is the use-case for using single AD placement config.

@karthicgit thanks for the comments:
I disagree with having a different variable for ADs, as one could want to deploy node pool 1 on AD1, node pool 2 on AD2 and node pool 3 to all ADs. The ADs need to be a property of each pool.
The pool subobject is indeed getting pretty long, but that's better solved by recommending people to version a pretty printed json and then using jsondecode to fill the nested map.
My intention was for the module user to hold the AD datasource, not having it in the module, but didn't make it explicit in the docs. You'd normally use the module as such:
data "oci_identity_availability_domain" "AD1" {
  compartment_id = data.oci_identity_compartment.root.id
  ad_number      = "1"
}

module "oke" {
...
  {..., placement_ADs = [data.oci_identity_availability_domain.AD1.name], ... }
}

as I imagine any user doing nontrivial work on oci has the AD datasources laying around already.
My rationale for this feature is here: https://github.com/oracle-terraform-modules/terraform-oci-oke/issues/515#issuecomment-1114369120

@12345ieee  while using the same nodepool key it will be mapped . With a new variable you will map with the same np1 or other value used for the key in nodepool variable .

I see your point now @karthicgit .
If we're splitting the map in 2 it may make sense to move the content of the whole node_config_details block into the new map, resulting in something like:
node_pools = {
  np1 = {shape="VM.Standard.E3.Flex",ocpus=2,boot_volume_size=150}
}

node_pools_config_details = {
  np1 = {node_pool_size=2,placement_ADs=["sQTi:EU-FRANKFURT-1-AD-1"]}
}

this also gives us a path forward into adding more options, by splitting each nodepool configuration block into its argument map.
If you like this option better I can implement it.

So, what would be the preferred solution @karthicgit ?

@12345ieee i would prefer not changing the existing keys and add only the new ones in new variable.
@hyder  could you please share your inputs.

I think for now the easiest way is to proceed with the simple change, deeper restructuring can be done on top.

@12345ieee As discussed please create a new variable and also don't hardcode the AD values .we have to fetch it from input like AD1,AD2 etc..  Fault domain support in placement config is also available now . https://docs.oracle.com/en-us/iaas/releasenotes/changes/d13cef7f-42fa-4313-be4d-b6ff117f7997/ .

@12345ieee As discussed please create a new variable and also don't hardcode the AD values .we have to fetch it from input like AD1,AD2 etc..  Fault domain support in placement config is also available now . https://docs.oracle.com/en-us/iaas/releasenotes/changes/d13cef7f-42fa-4313-be4d-b6ff117f7997

hey @12345ieee,
Where are you with this? Another user just reached out to me about the possibility of pinning a node pool to an AD.
I note that it's been quite a while so your branch is very likely well out of date considering the recent changes we did to nodepools. Let us know if you need any help. I would like to move this forward.

Hi @hyder ,
the code has been ready for a while, but I haven't been able to test it in action.
I have also not implemented the additional variable that was requested, but I've added the key to the np map.
I've just pushed what I have, the intent should be clear and you should be able to build on it if needed, I hope to be able to test it if still needed in the following weeks.

@12345ieee: We also need you to sign the OCA.

Yeah, this is what happens when I push untested code.
I'm not super happy about what I did for ad numbers, but that's what the API offers.

Thanks @12345ieee for your patience.


Jit compile x3 bootstrap

... else it is rather slow.

It's already jit compiled. I forgot to tag the commit. But code could be further optimized. Closing for now.


test-branch/execute-job/wrong-commit-sha-on-pr/5b4fb419-a67b-457b-ac73-4fa0ee4f40ce

PR for testing 2ae01d380b1f700cacf7c8a905b532a79e17193b

/canonical/self-hosted-runners/run-workflows 931ad21dd9ace68aaa0a7d341e94de96ad93fa6b


Drop unknown entity resolutions and test roles

This change will drop any entity resolutions that are unknown and don't match the standard pattern.  For example, it would have dropped the new ordinal changes.  (Although in this case we probably should have established a new ordinal prebuilt.)
Update LUIS tests to reflect the latest server results including roles.  Also fixed some of the issues caused by moving to functional tests.  (And why this was a functional test is a good question...)

No issues found in Microsoft.Bot.Builder.dll
No issues found in Microsoft.Bot.Builder.AI.Luis.dll
No issues found in Microsoft.Bot.Builder.AI.QnA.dll
No issues found in Microsoft.Bot.Builder.ApplicationInsights.dll
No issues found in Microsoft.Bot.Builder.Azure.dll
No issues found in Microsoft.Bot.Builder.Dialogs.dll
No issues found in Microsoft.Bot.Builder.TemplateManager.dll
No issues found in Microsoft.Bot.Configuration.dll
No issues found in Microsoft.Bot.Connector.dll
No issues found in Microsoft.Bot.Schema.dll

No issues found in Microsoft.Bot.Builder.dll
No issues found in Microsoft.Bot.Builder.AI.Luis.dll
No issues found in Microsoft.Bot.Builder.AI.QnA.dll
No issues found in Microsoft.Bot.Builder.ApplicationInsights.dll
No issues found in Microsoft.Bot.Builder.Azure.dll
No issues found in Microsoft.Bot.Builder.Dialogs.dll
No issues found in Microsoft.Bot.Builder.TemplateManager.dll
No issues found in Microsoft.Bot.Configuration.dll
No issues found in Microsoft.Bot.Connector.dll
No issues found in Microsoft.Bot.Schema.dll


Does Microsoft allow to copy fonts "Segoe UI" into Server Core image for production environment?

We are using SQL Server Reporting Service in our production, font "Segoe UI" is used for many of our reports.
Now we need deploy our product into cloud, we selected Windows Server Core image as base image for our product.
The problem there is no TrueType font in Windows Server Core image, this caused our reports use lucon.ttf as fallback font for "Segoe UI", the style of reports is not so good.
I'm wondering if we can copy Segoe UI fonts from Windows image into our product image, is that allowed by Microsoft? or do we need a license to do so?
Thanks!

There is guide from MS to install fonts to container, link is https://techcommunity.microsoft.com/t5/itops-talk-blog/adding-optional-font-packages-to-windows-containers/ba-p/3559761. Does this help?

@zhiweiv Thanks
I tried it on my local computer, but unfortunately I failed to do so, it always failed to access the network path.
But does it mean Microsoft officially allowed to copy font files from a Windows product to a Windows Server Core image?
Technically I can copy the font files from a Windows image into our product image based on Windows Server Core image, I'm worried about the license.

@akarshm any thoughts on this?

Or if it's OK to copy the fonts from windows server image to windows server core image? thanks!
FROM mcr.microsoft.com/windows:ltsc2019 as file-source
FROM mcr.microsoft.com/windows/servercore/iis:windowsservercore-ltsc2019

COPY --from=file-source ["/Windows/Fonts/segoe*.ttf", "/Windows/Fonts/"]

# ...

Can you please let us know what is the issue with accessing the networking path? is there an error you see or is it a different networking setup? Accessing the network path should be simple as this is local access.

Sorry to reply you late, I tried again, it returned such error:
C:\install>for /F "tokens=3 delims=: " %g in ('netsh interface ip show address | findstr /c:"Default Gateway"') do set GATEWAY=%g

C:\install>set GATEWAY=<IP_ADDRESS>

C:\install>net use o: \\<IP_ADDRESS>\WinSxS /user:ShareUser Abc_Abc.123
System error 53 has occurred.

The network path was not found.

Any update on this?
Is it legal to copy fonts from an mcr.microsoft.com/windows image to an image based on a windowsservercore image?
If not then is it legal to temporary copy the WinSxS folder, enable font features, and then remove the temporary WinSxS folder?

Unfortunately, I personally can't comment on the legality of what is okay and what isn't. Please have a look at our licensing terms.


Write (more) tests

Reopening, as while there are tests, there is also still room for improvement.


Inference cost code search task.

Hello I saw that the codesearch task had an inference cost of about 7 min for the AdvTest and I was wondering how do you compute that ?
Is it the time you got using a single query on 2x P100 or on multiples queries ?
And is the inference cost similar in https://github.com/microsoft/CodeBERT ? If not why does it take so much time and is there a way to reduce it ?

Because on your table

It is unclear whether the inference time are for practising the inference on the whole dataset or just to make one query.
But I guess 7 min is the inference for a whole dataset isn't it ?

7 min is the inference for a whole dataset.


Fixed hashrate printout when nonce found

Fixes an issue that causes reported hashrate to be much higher than actual when mining on rinkeby.

This also now fixes integer overflow in pool.go


SyncBot: Support a directed graph of flows.

This to enable the merging of two discourse categories down to one flowdock flow for support.

Router function supports configuration of which events to route.
SyncBot configures routers to synchronise threads in particular directions.
SyncBot reverse-maps the synchronisation of messages along mirrored threads.

Change-Type: patch
Connects-To: #297

Contributor checklist

[x] Build output been rebuilt and tested
[ ] Introduces security considerations
[ ] Tests are included
[x] Documentation is added or changed
[ ] Affects the development, build or deployment processes of the component

Reviewer Guidelines

When submitting a review, please pick:

'Approve' if this change would be acceptable in the codebase (even if there are minor or cosmetic tweaks that could be improved).
'Request Changes' if this change would not be acceptable in our codebase (e.g. bugs, changes that will make development harder in future, security/performance issues, etc).
'Comment' if you don't feel you have enough information to decide either way (e.g. if you have major questions, or you don't understand the context of the change sufficiently to fully review yourself, but want to make a comment)

---- Autogenerated Waffleboard Connection: Connects to #297

Parking this one for a bit, a tweak that's happened in log output has revealed a bug, that is being saved by the API goalkeeper, where it should be saved by the defenders.


Add icon to labels

Greetings
This is indeed a great library. I would like to request setting icons for every IPieInfo optionally. Ideally, it could accept a drawable or a vector. Apps supporting <21 must pass a .png while apps where min is 21 can pass a vector. This way the library doesn't have to deal with supporting vectors on older versions.
The icon would be on the side of the label, depending on the positioning and as the label the color should be the same than in the chart.

Thanks for the awesome work :)

let's do this~
Hope you will be well soon.

Now I have time to do this...

please update to 1.2.4

So icon is a setter setLabel(Bitmap bm) thanks! I'm gonna added to the development queue in my project

yep.
setLabelIcon(Bitmap) in PieOption.
and you can also decide label's relative position by setLabelPosition().


[Pendragon] Quality of life changes.

Changes to dice rolls with the skill surplus beyond 20 added to the die roll.
Lines: All roll_ / roll_test lines apart from attacks.
-Additional fixes
Squire horsemanship fixed to being horsemanship and not battle.
Line:584
Squire skills are called their name now instead of the skill values being put in. i.e: @{squire_skill_value} is now 'squire skill roll
Lines:566 572 578 584 590
'Repeating skill' line changed. @Cskillvalue changed to @Cskillname in the roll text at the end.
Line:557
Typo fixed. Ennemy changed to Enemy
Line:182
-Revised-
Minor errors found and corrected.
Distaff being Courtesy in roll text
Line:441
Fashion being Falconry in roll text
Line:461
TPio changed to TSpi to reflect Spiritual instead of old Pious.
Line:326-335 and 392
OCD fixed, Chirurgery > chirurgery
Line:426

Working on personality traits being able to be over 20 on either side and the opposite staying at 0

Tried to make personality traits go above 21 on both sides as the rule states it can happen. And as it happens the surplus beyond 20 gets added to the die roll.
I was not able to make this happen.
The positive side of the personalities somewhat works as the surplus gets added. But the negative side becomes -1 and decreases as the positive one increases while it should stay at 0.
The negitive traits can't go higher than 20 as the traits would need to be editable (becomes 20--1 if you go below 0 on the positive trait which you shouldn't be able to do. And breaks the surplus roll).
What I figured was, the easiest fix would be to make people set and keep track of these themselves.
If it was something people wanted.

You've got some merge conflicts. There was another change to Pendragon that was submitted before yours, and I'm guessing that was the issue. You'll need to resolve the conflicts before I can accept the Pull Request.

Took some time to Ctrl C/V the whole thing I did just becouse the order of two things was changed. Still found some more errors though..


Significance vs Security configuration apps

What's the difference between this one and the Microsoft Azure that is in the [Security configuration apps] tab?

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

ID: da9a9fad-ff34-4689-c30e-404a774f9bd3
Version Independent ID: 351b6aa7-da8e-350d-c002-6e0689b350cc
Content: Connect Azure to Cloud App Security
Content Source: CloudAppSecurityDocs/connect-azure-to-microsoft-cloud-app-security.md
Service: cloud-app-security
GitHub Login: @shsagir
Microsoft Alias: shsagir

@icelava Thanks for contacting Microsoft about your issue.
As we understand it, you're asking about a feature comparison between products and not a problem with the product documentation. Please open a support ticket or Send feedback so that we can investigate.


Frontend doesn't answer over IPv6

Home Assistant release with the issue:

0.67.0
Last working Home Assistant release (if known):
It never worked
Operating environment (Hass.io/Docker/Windows/etc.):

Raspbian 9.4
Component/platform:

frontend
Description of problem:
The frontend only listens on legacy IPv4, not IPv6
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):

frontend:

# Enables configuration UI
config:

http:
  # Uncomment this to add a password (recommended!)
  api_password: bla
  # Uncomment this if you are using SSL/TLS, running in Docker container, etc.
  base_url: https://my.ipv6.host:8123

Traceback (if applicable):
n/a

Additional information:
See #1442 for an old discussion on the same issue and a workaround.

Please post feature requests via the forum
Ipv4 might be seen as legacy, but quite alive in most places  :smile:


CppParameter doesn't expose default value

A small feature request:
#define DEFAULTMAXDEPTH 1024
bool allocateCapacity(size_t len, size_t maxdepth = DEFAULTMAXDEPTH);

Currently it's not possible to extract that default value for maxdepth = 1024

Oh right, actually working on it

Should be fixed via ba670ef
Available shortly in 0.2.0


Update Vue example in README.md

In Vue documentation mostly used PascalCase

I read this as
kebab-case: compatible with everything, including DOM-authored Vue apps
PascalCase: only compatible with precompiled Vue apps
I'm not a big Vue user, so I am happy to defer to your judgement if you feel like every Vue dev knows these caveats, and it's not necessary to write docs in the most compatible style.

Yea I think most of Vue devs understand this examples


Update ycmd fork for Rust support

Rust support is now merged into the upstream ycmd so for Atom to support it, the fork needs to merge those commits in. ycmd seems to have better support for Rust now than the existing Atom-specific plugins that call into cargo and racer directly.

The upstream of ycmd has fixed the issues that I worked around in the fork and the fork is not needed any more. My winter vacation is coming soon and I will be able to work on it one or two weeks later.

The fork is not needed any more? Do you think we could compile it ourselves and it will work or you have to do some additional work?

For rust support there's some additional work needed, but it also seems like some API changes have happened underneath. I can't get the current version of the extension working with the latest ycmd, the daemon won't start.

Now this package uses the original ycmd instead of my fork and it should work for all ycmd-supported languages.

Thanks!

Thank you!


How do we handle injection attacks?

How do we handle injection attacks? In SQL, you can inject something into a query from a dynamically provided value if it's concatenated into the query.
Does Ftrack support a way to do this safely?

The API query language is not SQL, but a custom language that is parsed into a internal representation on the backend.
On the backend we apply permission checks on the query and returned entities, to guarantee that the data returned only contains information that the API user is allowed to see. If the query run as the user then they are not able to modify it to get access to more data than what their permissions allow.
If you are not using the users credentials, then the user will be able to access more data because it is bypassing ftrack's permissions.
The ftrack-javascript client does not have any method of preventing injections as we haven't found it necessary. If you build UI’s which have free text search for example, I recommend that you escape quotes within the value and surround the value with quotes in the query to avoid issues with malformed queries. As if this is enough to prevent an injection, I can not say.

The problem is that much of our system consists of actions. A user may invoke an action, but the system that performs the mutations in Ftrack has higher access than the user.
That's intentional. We want to lock down certain actions to be system-only. How can we retain this while still allowing the system to be safe?
For instance, it would be less than ideal if user A would be able to gain insight into something from user B.
Some examples I can think of:

Doing a select secret_stuff from SomeType where stuff='<dynamic value>' and then providing the dynamic value ' or __entity_type__ not in ', leading to a resulting query of select secret_stuff from SomeType where stuff='' or __entity_type__ not in ''. This makes it select the first record instead of the relevant one.
Mutating some kind of value, but then using the same trick to do it on behalf of the first match instead.

As long as you can confirm that it is enough to escape the quotes, then that's good enough for us.


Building crashes on "Building routes manifest file"

Describe the bug
I have created an empty project and added ssr-boost there, but when building it crashes on Building routes manifest file... with no errors, except Before starting the server, you need to create a build: ssr-boost build
Development server and this template seem to be working fine.
To Reproduce
Steps to reproduce the behavior:

Clone this repository
npm i
npm run build
Build crashed with no errors.

Additional Info

OS: MacOS 14.2.1
Node: 20.8.1
Vite: 5.0.11
SSR Boost: 2.0.3

@FullStack0verfl0w thanks for reporting. Try 2.1.0 version. Also as I can see, you changed outDir and in this case use follow commands (or change outDir to default ../build):
    "start:ssr": "ssr-boost start --build-dir dist",
    "start:spa": "ssr-boost start --only-client --build-dir dist",
    "preview": "ssr-boost preview --build-dir dist"


DataLakeRestClient AppendData exception for zero-length files

Describe the bug
AppendData using a zero length stream throws an exception.
Exception or Stack Trace
Azure.RequestFailedException: The value for one of the HTTP headers is not in the correct format.
RequestId:d3d0ae4c-501f-00e4-3f92-cbcbec000000
Time:2020-01-15T10:58:39.2410781Z
 ---> System.Exception: Exception of type 'System.Exception' was thrown.
   --- End of inner exception stack trace ---
   at Azure.Storage.Files.DataLake.DataLakeRestClient.Path.AppendDataAsync_CreateResponse(Response response)
   at Azure.Storage.Files.DataLake.DataLakeRestClient.Path.AppendDataAsync(ClientDiagnostics clientDiagnostics, HttpPipeline pipeline, Uri resourceUri, Stream body, String version, Nullable`1 position, Nullable`1 timeout, Nullable`1 contentLength, Byte[] transactionalContentHash, String leaseId, String requestId, Boolean async, String operationName, CancellationToken cancellationToken)
   at Azure.Storage.Files.DataLake.DataLakeFileClient.AppendInternal(Stream content, Nullable`1 offset, Byte[] contentHash, String leaseId, Boolean async, CancellationToken cancellationToken)
   at Azure.Storage.Files.DataLake.DataLakeFileClient.AppendAsync(Stream content, Int64 offset, Byte[] contentHash, String leaseId, CancellationToken cancellationToken)

To Reproduce
using System;
using System.IO;
using System.Threading.Tasks;
using Azure.Storage;
using Azure.Storage.Files.DataLake;

namespace ZeroLengthRepro
{
    class Program
    {
        static async Task Main(string[] args)
        {
            var storageAccountName = "<Account>";

            string storageAccountKey = "<Key>";
            StorageSharedKeyCredential sharedKeyCredential = new StorageSharedKeyCredential(storageAccountName, storageAccountKey);
            // Create DataLakeServiceClient using StorageSharedKeyCredentials
            Uri serviceUri = new Uri("<URL>");
            DataLakeServiceClient serviceClient = new DataLakeServiceClient(serviceUri, sharedKeyCredential);
            // Create a DataLake Filesystem
            DataLakeFileSystemClient filesystem = serviceClient.GetFileSystemClient(Guid.NewGuid().ToString().ToLowerInvariant());
            await filesystem.CreateAsync().ConfigureAwait(false);
            var response = await filesystem.CreateFileAsync("test.dat").ConfigureAwait(false);
            var fileClient = response.Value;
            await using var fs = new MemoryStream();
            var length = fs.Length;
            await fileClient.AppendAsync(fs, 0).ConfigureAwait(false);
            await fileClient.FlushAsync(length).ConfigureAwait(false);
        }
    }
}

Code Snippet
await fileClient.AppendAsync(fs, 0).ConfigureAwait(false);

Expected behavior
No exception or an error message with a useful error message.
Workaround: Check for length > 0 prior to Append.
Screenshots
If applicable, add screenshots to help explain your problem.
Setup (please complete the following information):

OS: [e.g. iOS] Windows 10
IDE : [e.g. IntelliJ] Visual Studio 16.4
Version of the Library used 12.0.0-preview.8

Additional context
Add any other context about the problem here.
I noticed that AppendAsync closes the stream, you might want to put that into the intellisense comments as it's a bit unusual (some don't close it if you pass it in, some provide a close bool etc.).
Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[x] Bug Description Added
[x] Repro Steps Added
[x] Setup information Added

@Tornhoof, in uploading a file with a 0 length stream, is your goal to achieve the functionality of  file.OpenWrite()?

is your goal to achieve the functionality of file.OpenWrite()?

I'm not sure that I follow. Do you mean that I want to create a remote file which then returns a stream I can use for writing to azdl? Then the answer is no.
I simply have a 0 byte length file on my filesystem (marker file for example) and this needs to be uploaded to azdl too (and exists there as a 0 byte length file).
It's just a bug which is not very visible, occurs only with specific files, making it hard to debug, because who tests their upload logic with zero length files. I certainly did not, a customer noticed the problem, he had an empty "New Text document.txt" in a directory. It took me awhile to find out that the file was empty.
From my POV just short-circuit the append operation if the stream is empty.

@tg-msft, would you be ok if we checked the stream length in DataLakeFileClient.Append(), and not perform the request if content.Length == 0?  I don't think we currently support non-seekable streams, and I'm not sure if we could/plan to.

I don't know if we should just return if the stream length == 0, that may introduce some ambiguity - the Data Lake service doesn't accept Appends of length 0.
We could manually check the stream length (assuming it is seekable) and throw an ArgumentException with a more specific message.  In this case, the customer would need to check the file length before Append, but it would be more clear what is happening.

Did we still want to go through with a manual check of the stream length that @seanmcc-msft suggested?
@tg-msft would you be okay with that?

We're currently looking into resolving this offline and seeing if the service can produce a more helpful error.

@amnguye any updates?

@Tornhoof Apologies for the late reply. We will update this thread once we have more details.
@amnguye @seanmcc-msft Could you please provide an update on this github thread once you get a chance ? Awaiting your reply.


Terminal not working right

Terminal doesnt log messages sent from discord bot, also when i run nodemon it wont let me input anymore prompts i have to kill the terminal and use another one to use prompts

It looks like your report is missing some important information, See this wiki page for more info on how to report actionable terminal issues.
/label info-needed terminal

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Support for Gitlab

I have found this project, and I love it. It is simple, requires almost no setup, and it just works. However, I think somthing that would make this project better is support for gitlab for people using gitlab pages.
Perhaps we could make this part of utterance a module that could be easily replaced?
I don't know any typescript or I would look into adding this feature.

Dupe of a few other issues https://github.com/utterance/utterances/search?q=Gitlab&type=issues

I see almost all of them were closed without reason, and the one that did have a reason, was updated with new information and was ignored.

Yeah, integration with gitlab would be awesome. I have moved most of stuff on there as $M S has now taken over github.

from above dupe list #240 has some useful / worth reading info imho. but as I see Jeremy is not really a fun of these ideas see #508

bumping this for gitlab support

Any life left in this idea?
Integration of this with gitlab pages would indeed be pretty sweet.


gradle appRun raises ClassNotFoundException for HikariCP

I just created a StackOverflow post hopping someone help me solve this problem. I'm not sure if I'm missing any configuration or if it's a bug.
You can get the source code on my proof of concept project and see what is happening.
Thanks,
Wellington

All done, it was a misconfiguration. just add the hikaricp as a gretty dependency


Make Red Hat manifests available in policy input

https://issues.redhat.com/browse/HACBS-2369

PaC failure addressed here: https://github.com/enterprise-contract/ec-cli/pull/895


RuntimeError: zero-dimensional tensor (at position 0) cannot be concatenated

Hello @rwightman, Thanks for your great repo.
However, I was trying to run this notebook which had used over here by its previous version of this repo. In the inference time, it called the model like following:
def make_predictions(images, score_threshold=0.22):
    images = torch.stack(images).cuda().float()
    predictions = []
    with torch.no_grad():
        det = net(images, torch.tensor([1]*images.shape[0]).float().cuda())
        for i in range(images.shape[0]):
            boxes = det[i].detach().cpu().numpy()[:,:4]
            scores = det[i].detach().cpu().numpy()[:,4]
            indexes = np.where(scores > score_threshold)[0]
...
...

I've found that the DetBenchEval method has renamed with DetBenchPredict, and defined here. And now the forward function is taking additionally img_size argument (forward(self, x, img_scales, img_size):. And so, I've tried to change accordingly:
    images = torch.stack(images).cuda().float()
    predictions = []
    with torch.no_grad():
        print(images.shape[0])

        det = net(images,
                  torch.tensor([1]*images.shape[0]).float().cuda(),
                  torch.tensor([512]*images.shape[0])) # 512 for img_size

        for i in range(images.shape[0]):
            boxes = det[i].detach().cpu().numpy()[:,:4]
            scores = det[i].detach().cpu().numpy()[:,4]
            indexes = np.where(scores > score_threshold)[0]
...
...

And it gave me RuntimeError and points over here
RuntimeError: The following operation failed in the TorchScript interpreter.
Traceback of TorchScript (most recent call last):
  File "c:\users\innat\anaconda3\envs\alaska\lib\site-packages\effdet\anchors.py", line 214, in _batch_detection
    boxes = decode_box_outputs(box_outputs.float(), anchor_boxes, output_xyxy=True)
    if img_scale is not None and img_size is not None:
        boxes = clip_boxes_xyxy(boxes, img_size / img_scale)  # clip before NMS better?
                ~~~~~~~~~~~~~~~ <--- HERE

    scores = cls_outputs.sigmoid().squeeze(1).float()
  File "c:\users\innat\anaconda3\envs\alaska\lib\site-packages\effdet\anchors.py", line 167, in clip_boxes_xyxy
def clip_boxes_xyxy(boxes: torch.Tensor, size: torch.Tensor):
    boxes = boxes.clamp(min=0)
    size = torch.cat([size, size], dim=0)
           ~~~~~~~~~ <--- HERE
    boxes = boxes.min(size)
    return boxes
RuntimeError: zero-dimensional tensor (at position 0) cannot be concatenated

Any catch? Anyway, I'm sorry if it looks dumb question, not much familiar with the torch ecosystem. If the above information isn't enough, I can explain in many details.

@innat those changes did make things a bit more complicated when you don't use/need them.. tried to make them optional but torchscript complained so had to keep them as mandatory for now, when you use them with variable sized images being predicted on the square canvas, it allows clipping of the detection results to clip boxes that are in the letter box region of the images. Doing that before nms seemed to improve speed slightly and more importantly the end mAP results.
The scales needs to be a [batch_size, 1] float tensor, of ones if you aren't trying to restore the scale of your bbox to a rescaled input, and the img_size needs to be a [batch_size, 2] tensor of form width, height for the size of the original image in original aspect ratio. Again, if you aren't caring about rescaling or clipping, you should set it to the img_size.
torch.ones((images.shape[0], 2)) * 512 should work for img_size ... and just torch.ones(images.shape[0]) for scale

Oh, I see. Thanks. @rwightman


zsh: fix nixUnstable completions

Motivation for this change
The unstable nix completions work very well. Unfortunately, zsh users have so far been left to fend for themselves when it comes to getting them working. This fixes the situation by bumping nixUnstable to a commit with the working zsh completion script, and cleaning up a conflict with nixStable and nix-zsh-completions.
Things done
Tested on my local NixOS machine and it works
Changes:
0da6499575a (Timothy DeHerrera, 19 minutes ago)
zsh: remove conflicting nixUnstable completions
ab81e523441 (Timothy DeHerrera, 19 minutes ago)
nixUnstable: bump to include zsh completions

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/553

Result of nixpkgs-review pr 128649 at c49209fe run on aarch64-linux 1

4 packages built successfully:

hydra-unstable
nix-update
nixUnstable
nixpkgs-review

Would it make sense to still install nix-zsh-completions for Nix 2.4, but to remove the completions for the nix-cmd from the derivation? Because it seems still useful for e.g. nix-build etc.

is there a simple filter function for derivations, or would I need to do a simple mkDerivation?

nvm, I'll just overrideAttrs and add a postInstall phase to remove it

Yup, that's what I've done in an overlay locally as well :)
Btw as you seem to be using Nix Unstable completions as well, you may want to try out https://github.com/NixOS/nix/pull/4980 (basically support for completing nix build -f . foo<tab>). It shouldn't be added to this PR yet and I won't give safety guarantees for now as the patch is only a few hours old, but I'd be happy to get some feedback on this :)

Btw as you seem to be using Nix Unstable completions as well, you may want to try out NixOS/nix#4980 (basically support for completing nix build -f . foo<tab>).

I'll be glad to make a patch and see if I can get it working on my local machine. I am also currently using nix-dram so hopefully they don't conflict. Thanks for the tip!
Also, I just submitted a commit to implement your suggestion :+1:

Now that #129574 has been merged, I went ahead a removed the commit to bump the nixUnstable version, and this should be a lot less scary to merge :smile:

@nrdxp do you think it'd make sense to backport this? :)

@Ma27, I think so, assuming the above mentioned nixUnstable bump on which this depends is also backported.

It is :)


Localization of the bot

At the moment, there are still some shortcomings, but I will try to fix them soon.
I am waiting for your comments on this idea.

Wow, thanks so much for the contribution!
I was definitely hoping to add proper localization at some point, and you seem to have made a big contribution in that respect, so thanks for taking that off my plate ;)
Unfortunately, I have a long-standing branch here: https://github.com/denverquane/amongusdiscord/pull/177 that I'm hoping to merge ASAP, and it does introduce new formatting for a lot of the bot output (for the better; the help and settings output are really ugly right now).
I will probably focus merging that branch first, so I apologize for the conflicts it will create with your PR, but overall I really really appreciate the contribution, and would love to merge yours in promptly after that merge. It will also help guide me towards using those localization formats for future additions...

Let me know how I can help you after I merge that PR in; localization is definitely a big priority, it's just rather fragile right now as a lot of the commands/output/formatting changes so regularly in these early development stages.
We plan on focusing towards a single hosted bot in the future for a variety of reasons, so bonus points for any approaches that would allow users to change the language on the fly, while the bot is running (per server, probably). We will always support self-hosting the bot, but the process might become a bit more complex (for good reasons)


UTM issue with MySQL

About the added UTM feature (@cywio 👍)
URLs are not filtered correctly with MySQL, and all end up in umami UTM views.
substring_index in MySQL doesn't have the same behaviour as PostgreSQL split_part

last arg of split_part is the part position to return, with an empty string returned if it's greater than number of parts after splitting
whereas it's an occurrence count of the delimiter in substring_index, the returned value corresponding to the substring before that count. So the all string is returned if delimiter isn't there.

Thought it could be filtered with a where like clause (with the utm param),
and the extraction handled with eventual various db specific functions.
It would probably be easier to have multiple db support.
What do you think?

We should move all this logic to the UI and use table filters. The database query should simply do a search for LIKE "%?" which will pick up all urls with query strings, then we can filter them down on the frontend.

Fixed in v1.37.0.


Kubernetes services aren't cleaned up

What happened?:
After running pipelines for a while, there are a considerable number of kubernetes services defined, that are pointing towards objects that have been cleaned up.
kubectl get svc -l suite=pachyderm,component=worker | wc -l
     630

The RCs, however, have been properly cleaned up:
kubectl get rc -l suite=pachyderm,component=worker | wc -l
      19

pachctl list pipeline | wc -l
      19

What you expected to happen?:
The services are deleted at the same time as the RCs whenever a pipeline is updated.
Environment?:

Kubernetes version (use kubectl version): 1.12-gke
Pachyderm CLI and pachd server version (use pachctl version): 1.9.3

Just want to drop this error here in case anyone encounters it. We had most of the kubernetes cluster fail to start with the following error:
standard_init_linux.go:178: exec user process caused "argument list too long"

In our case, there were so many created services that the space reserved for environment variables was exhausted, and pods could effectively not start. All services can then be deleted with kubectl delete svc -l suite=pachyderm,component=worker (in our case we didn't use services as part of pipelines).
This was partially caused by one of internal component updating a lot of pipelines automatically.

I have the same issue: old services are not deleted after pachctl update. This is an important issue as my k8s cluster has quotas limitting the number of created services.

@marcadella everything worked fine for me after deleting them all, I suspect it recreates objects that it needs

I believe this was fixed by https://github.com/pachyderm/pachyderm/pull/4782. If you see this issue post Pachyderm 1.10.3, please re-open and we'll take another look.


WIP Session split

Codecov Report

Merging #649 into master will increase coverage by 0.02%.
The diff coverage is 80.43%.

@@            Coverage Diff             @@
##           master     #649      +/-   ##
==========================================
+ Coverage   91.28%   91.29%   +0.02%
==========================================
  Files          80       82       +2
  Lines        5527     5560      +33
==========================================
+ Hits         5045     5076      +31
- Misses        303      305       +2
  Partials      179      179

Impacted Files
Coverage Δ

session_sender.go
78.64% <78.64%> (ø)

session_receiver.go
79.65% <79.65%> (ø)

session.go
96.53% <94.74%> (+8.29%)
:arrow_up:

Continue to review full report at Codecov.

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


🍒 Cherry pick PR #14921 to staging 🍒

🍒 Cherry pick https://github.com/Expensify/App/pull/14921 to staging 🍒

Hey! I see that you made changes to our Form component. Make sure to update the docs in FORMS.md accordingly. Cheers!

🚀 Cherry-picked to staging by https://github.com/OSBotify in version: 1.2.67-5 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

🚀 Deployed to production by https://github.com/mountiny in version: 1.2.67-7 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Cherry-picked to staging by https://github.com/AndrewGable in version: 1.3.28-2 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

🚀 Deployed to production by https://github.com/AndrewGable in version: 1.3.28-5 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


JVM-Runtime

Description:
Allows to deploy jar files, to support all jvm-based languages.
TODOs:

[x] Ready to review
[x] Automated Tests
[x] Docs

Could you add some basic docs, I am curious about the exact UX

@sebgoa Done

nice
let's add @murali-reddy review

LGTM.
@caraboides could you please hook your example tests into CI like https://github.com/kubeless/kubeless/blob/master/tests/integration-tests.bats#L202-L212

@murali-reddy done

fine with me as is. We can improve on it a bitlater


WARNING when webpack loading all *.js.map

Hi,
After upgrading ng2-bootstrap from 1.0.17 to 1.0.20, a lot of warning appear during the TypeScript compilation :
[...]
WARNING in ./~/ng2-bootstrap/components/rating.js
Cannot find SourceMap 'rating.js.map': Error: Cannot resolve 'file' or 'directory' ./rating.js.map in /var/www/frontend-admin/node_modules/ng2-bootstrap/components

WARNING in ./~/ng2-bootstrap/components/tabs.js
Cannot find SourceMap 'tabs.js.map': Error: Cannot resolve 'file' or 'directory' ./tabs.js.map in /var/www/frontend-admin/node_modules/ng2-bootstrap/components

WARNING in ./~/ng2-bootstrap/components/timepicker.js
Cannot find SourceMap 'timepicker.js.map': Error: Cannot resolve 'file' or 'directory' ./timepicker.js.map in /var/www/frontend-admin/node_modules/ng2-bootstrap/components
[...]

Full version here : warning-bg2-bootstrap.txt
I'm using webpack (from https://github.com/AngularClass/angular2-webpack-starter).
It seems to work but this is very annoying.

@antitoine We don't add sourceMaps and webpack seems to require them with your configuration.

Having the same problem.
I don't use bundles, my app is building from required components and ng2-bootstrap components has refs for map files.
'Alert' component for example:
...
exports.AlertComponent = AlertComponent;
//# sourceMappingURL=alert.component.js.map

@nick1247 Thanks for this information. Unluckily the sourcemaps get excluded by the .npmignore.
@valorkin Either we set "sourceMap": true in the tsconfig.json or we remove the exclusion in the .npmignore

Thanks

Same problem with SystemJS

there are 2 possible options,

disable source maps for common.js build
remove source map ignore from npmignore

@antitoine please join https://www.hamsterpad.com/chat/ng2
and drop me a message
so we can try both options fast

source maps removed because they produce warning hell
WARNING in ./~/ng2-bootstrap/ng2-bootstrap.js
Cannot find source file 'ng2-bootstrap.ts': Error: Cannot resolve 'file' or 'directory' ./ng2-bootstrap.ts in /home/valorkin/work/open-source/angular2-webpack-starter/node_modules/ng2-bootstrap

Fixed with the 1.0.22 npm package ! Thank


Fix casting bug when backfilling work histories

Context
This PR attempts to fix a bug from the previous PR https://github.com/DFE-Digital/apply-for-teacher-training/pull/9853
There are two enums on the ApplicationWorkHistory record. part_time and full_time. ActiveRecord returns these enums in the console like so part_time but in the DB it's saved Part time.
Previously because we called record.attributes it returned 'part_time' and the value in the DB would be saved as 'part_time'. Not 'Part time'.
This would break some views as ActiveRecord expects the value in the DB to be 'Part time'. This commit fixes this issue.
Changes proposed in this pull request
Cast part_time and full_time to Part time and Full time before saving work experiences to DB in the MigrateApplicationChoicesWorker worker
Guidance to review

Things to check

[ ] If the code removes any existing feature flags, a data migration has also been added to delete the entry from the database
[ ] This code does not rely on migrations in the same Pull Request
[ ] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[ ] If this code adds a column to the DB, decide whether it needs to be in analytics yml file or analytics blocklist
[ ] API release notes have been updated if necessary
[ ] If it adds a significant user-facing change, is it documented in the CHANGELOG?
[ ] Required environment variables have been updated added to the Azure KeyVault
[ ] Inform data insights team due to database changes
[x] Make sure all information from the Trello card is in here
[x] Rebased main
[x] Cleaned commit history
[x] Tested by running locally
[x] Add PR link to Trello card

Tested this locally and all new records have the correct commitment enum value Full time or Part time

 Edit work experiences - Enable feature


Groovy script to configure CAS

This is a bit of an support request, but I'm hoping you could assist.
At the moment,in another Jenkins instance I'm using a groovy script similar to  this to configure LDAP authentication. I was wondering if you could provide something similar for configuring CAS as well.
If not, no worries, I'll make-do with inserting the XML directly into the jenkins config, but I think a groovy script would be more appropriate.
Thanks in advance

I've never tried configuring Jenkins with a Groovy script, but judging by the example you linked to, you should be able to create a new CasSecurityRealm(params...) instead of the LDAPSecurityRealm. Refer to the CasSecurityRealm.java file and protocols (e.g. Cas30Protocol.java to see which parameters to use.

Perfect. That will help. I didnt know what to look for. I should be able to figure the rest on my own. Thanks!


chore(CI): enforce yarn version 1.6.0

This is considered best practice to ensure builds are working across the
board. In fact, right now we have the problem that there's a bug in 1.7.0
which prevents our builds to pass. Until 1.8.0 is released, we have to nail
the version to 1.6.0 as CI uses the latest available version by default.

Okay, turns out this syntax doesn't really do it for matrix based travis configurations. Travis seems to install the yarn version of every dedicated package. In order to land #819 we just need to roll back the version numbers from 1.7.0 to 1.6.0 in very upgrade commit.
Closing this one.


Removing admin group from root user?

via https://twitter.com/Tigersharke/status/580973057281409024

What I've seen is that when the admins group is stripped, the login stops working.

Can't reproduce this anymore.


Sum Type Variants as Types

One feature that I wish rust had was the ability to specify an enum's variant as a distinct type. This is particularly useful if you want to indicate in your type system that a function returns an enum Foo, but can only return a Foo::Bar. If we introduce #147 Sum Types, then this is a feature I'd like Haystack to support.

closed with #177.
You can now express variants as distinct types. This probably has some bugs in it, but those will come up as needed.
enum struct Foo {
    u64: Bar
    Str: Baz
}

fn always_foo_bar(u64) -> [Foo::Bar] {
    cast(Foo::Bar)
}


Burlap as a viable book-binding material in Mystcraft

I'm not sure how it's added but it'd be cool if burlap could be used in place of leather in the book binder and link book recipes.

Good idea, will look into it.

Quite a bit more difficult than I expected. Kept finding more and more places where itemStack == Items.leather was hardcoded in the Mystcraft code :disappointed:

@erima here's a build with this code in it if you want to test it out: VeganOption-mc1.7.10-0.1.0-mystcraft-test.jar
Let me know if there's anything else for which the VO alternatives aren't working in Mystcraft.

Ah, thanks for looking into it! I just tried out the test.jar and it seems to work great. The other item that stuck out to me was black ink.  It's kind of an odd combo - I like your implementation of black ink but it also can be used to make ink vials, which is in the Fluid Registry as black ink. Perhaps a convenient way to approach it would be to consider VO black ink to be a smaller mb of ink than a vial (though the name implies the opposite).
I'm not implying that you should change yours or not, just pointing out that it may be confusing, and that I'd also +1 any kind of fluid registry additions for VO liquids. Soap ocean worlds!


Allow to use PyCharm built-in local inline code completion together with Continue

Validations

[X] I believe this is a way to improve. I'll try to join the Continue Discord for questions
[X] I'm not able to find an open issue that requests the same enhancement

Problem
When using Continue it is not possible to use the inline code completion from PyCharm together with Continue Chat and Edit functions.
During working the inline code suggestions are sometimes great but also often annoying as they interfere with normal editing (e.g. just change a word or number) and cursor navigation with the keyboard.

Solution
Just have a configuration option to allow PyCharm to activate the built-in inline code completions and deactivate the ones from Continue.

I believe this is a duplicate of: https://github.com/continuedev/continue/issues/2061
For this to work, Continue has to switch to jetbrains API instead of how they are currently doing it.

^Correct, but appreciate the +1 here.

You are right. Sorry for the duplicate.
Closing since duplicate of #2061