debugger connection stopped working with on prem

Please include the following with each issue:
1. Describe the bug
I used to be able to connect with debug from VS Code at a specific on prem customer. Now it just hangs and does not connect.
They are using Edge as the browser (company requirement). Operating system is Windows 2019 server standard.
2. To Reproduce
Bring up project in VS Code. Select debug without publishing.

Go to '...'

N/A

Note: Because the developers need to copy and paste the code snippet, including a code snippet as a media file (i.e. .gif) is not sufficient.
3. Expected behavior
A clear and concise description of what you expected to happen.
4. Actual behavior
A clear and concise description of what happened accompanied by images, animations, or a link to a video showing the issue occurring
5. Versions:
13.0.971907

AL Language: 9.0
Visual Studio Code:
Business Central: BC22.5
List of Visual Studio Code extensions that you have installed:
AL Extension Pack
AL Formatter
AL Object Ninja

Final Checklist
Please remember to do the following:

[ ] Search the issue repository to ensure you are reporting a new issue

[ ] Reproduce the issue after disabling all extensions except the AL Language extension

[ ] Simplify your code around the issue to better isolate the problem

Thanks for the logfiles, you are right in that there seems to be nothing much in them. That is very weird. What is in the debug output in VSCode when it hangs? I assume general connectivity from VSCode is fine: i.e. you can download symbols and publish to the server?

I see, it's failing on the very first request to the server (dev/metadata) so it can't reach the server at all. Does the request eventually time out and give you something like:
[2024-04-10 10:36:45.74] Error: No such host is known. (bcserver:7049)
No such host is known.

[2024-04-10 10:36:45.74] Error: An error occured while processing the request.
Request ID: 344da29c-81b0-4f69-a899-54c7ec4358b7
Session ID: 34ea89c7-ff95-44b6-a23e-0f2a0f6137cf

Or does it hang indefinitely and stay stuck?
Checking the obvious, the developer port on the server (default 7049) is enabled right?

Hangs forever
Dev port is enabled
It is late here
I will be back online in the morning
I am USA central time
On Tue, Apr 9, 2024 at 8:55 PM Thaddeus Loke @.***>
wrote:

I see, it's failing on the very first request to the server (dev/metadata)
so it can't reach the server at all. Does the request eventually time out
and give you something like:
[2024-04-10 10:36:45.74] Error: No such host is known. (bcserver:7049)
No such host is known.
[2024-04-10 10:36:45.74] Error: An error occured while processing the request.
Request ID: 344da29c-81b0-4f69-a899-54c7ec4358b7
Session ID: 34ea89c7-ff95-44b6-a23e-0f2a0f6137cf
Or does it hang indefinitely and stay stuck?
Checking the obvious, the developer port on the server (default 7049) is
enabled right?
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/AL/issues/7709#issuecomment-2046317553, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AYZXW7PTTNZHKXRPT2QD4MDY4SLYFAVCNFSM6AAAAABF5REBCCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANBWGMYTONJVGM
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
This email may contain confidential or privileged information.  If you
believe you have received this email in error,
please notify the sender
by reply email and then delete this email immediately.

That's very weird, in which case it might be the server that's causing the hanging and not the VSCode extension. Can you check the Event Viewer of the server host for entries under MicrosoftDynamicsNavServer to see if there's any errors server-side?
If it's a problem with this specific server, it's very likely a server side configuration issue.
Reply when convenient, there's no rush.

No new events on event viewer. I exited VS Code and brought it back up,
which restored my symbols.
IIS has a list of allowed apps that can communicate through Windows
Defender Firewall.
Do you know if VS Code  debugging needs to be allowed - and if yes, what
has to be allowed?
On Tue, Apr 9, 2024 at 10:09 PM Thaddeus Loke @.***>
wrote:

That's very weird, in which case it might be the server that's causing the
hanging and not the VSCode extension. Can you check the Event Viewer of the
server host for entries under MicrosoftDynamicsNavServer to see if there's
any errors server-side?
If it's a problem with this specific server, it's very likely a server
side configuration issue.
Reply when convenient, there's no rush.
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/AL/issues/7709#issuecomment-2046466902, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AYZXW7KN7A6R6CMAMQ6GR2DY4SUM5AVCNFSM6AAAAABF5REBCCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANBWGQ3DMOJQGI
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
This email may contain confidential or privileged information.  If you
believe you have received this email in error,
please notify the sender
by reply email and then delete this email immediately.

When I follow the link, I see this text:
{
"runtimeVersion": "11.1",
"webApiVersion": "6.0",
"debuggerVersion": "6.0",
"webEndpoint": "http:////Webclient",
"extensionAllowedTargetLevel": "Internal"
}
The web site it takes me to:
http://localhost:7049/BC220/dev/metadata?tenant=default
Does this mean anything to you?
On Wed, Apr 10, 2024 at 9:10 AM David Machanick @.***>
wrote:

No new events on event viewer. I exited VS Code and brought it back up,
which restored my symbols.
IIS has a list of allowed apps that can communicate through Windows
Defender Firewall.
Do you know if VS Code  debugging needs to be allowed - and if yes, what
has to be allowed?
On Tue, Apr 9, 2024 at 10:09 PM Thaddeus Loke @.***>
wrote:

That's very weird, in which case it might be the server that's causing
the hanging and not the VSCode extension. Can you check the Event Viewer of
the server host for entries under MicrosoftDynamicsNavServer to see if
there's any errors server-side?
If it's a problem with this specific server, it's very likely a server
side configuration issue.
Reply when convenient, there's no rush.
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/AL/issues/7709#issuecomment-2046466902,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYZXW7KN7A6R6CMAMQ6GR2DY4SUM5AVCNFSM6AAAAABF5REBCCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANBWGQ3DMOJQGI
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
This email may contain confidential or privileged information.  If you
believe you have received this email in error,
please notify the sender
by reply email and then delete this email immediately.

There shouldn't be anything special about VSCode debugging that needs to be enabled beyond making sure that port 7049 (or whichever is configured on your server as the developer port) is open. Unfortunately since we're getting into the weeds of the server configuration, I don't think I can help much more through the github repo. I'll need someone with expertise on the server side, so this is at the point where we should move this to a customer support case. From there, it will be easier to bring additional people in to look at this. It will be much faster to get a resolution that way too.
Can you do one of the following?

Open a support request to CSS through the PartnerSource portal
Open a bug in Collaborate
Contact your Service Account Manager (SAM) in your local subsidiary to understand what is included in your contract as of support incident and PAH (Partner Advisory Hours). Your SAM might also direct you "step by step" how to open a support request or how to get credentials, if this is the first time for you or your company

If it helps make some progress, that URL that you shared just means that you're targeting a local server on your machine (docker probably). I assume you can contact the server through other means (webclient, web api calls, etc..) and it's just a problem with the developer endpoint (port 7049)? If so, you can try to verify that traffic through that port is actually reaching the server.

I'm going to raise this internally to see why support is pushing you to here, since I cannot handle what should be a support case through github.

I think I finally convinced support to take it further, but I think it
would still be good to raise this internally.
Otherwise you become an adjunct of the support department.
Dave
On Thu, Apr 11, 2024 at 8:23 PM Thaddeus Loke @.***>
wrote:

I'm going to raise this internally to see why support is pushing you to
here, since I cannot handle what should be a support case through github.
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/AL/issues/7709#issuecomment-2050801708, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AYZXW7PMO4RKQ37XX4UHWW3Y44ZRLAVCNFSM6AAAAABF5REBCCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANJQHAYDCNZQHA
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
This email may contain confidential or privileged information.  If you
believe you have received this email in error,
please notify the sender
by reply email and then delete this email immediately.

There is a setting in the instance configuration for PublicWebBaseURL
that does not look like it is used for user connections.
I set this:
Set-NAVServerConfiguration BC220 -keyname PublicWebBaseUrl -keyvalue "
http://localhost:8080/BC220/?company=US&dc=0"
and now the debug works.
It is fussier though - it hangs if it thinks breakpoints are in other apps
w2here I don't have access. Once I clear all breakpoints it works.
The other app has these settings
"allowDebugging": true,
"allowDownloadingSource": false,
"includeSourceInSymbolFile": true
This used to be okay to set breakpoints - did this change?
Dave
On Fri, Apr 12, 2024 at 8:53 AM David Machanick @.***>
wrote:

I think I finally convinced support to take it further, but I think it
would still be good to raise this internally.
Otherwise you become an adjunct of the support department.
Dave
On Thu, Apr 11, 2024 at 8:23 PM Thaddeus Loke @.***>
wrote:

I'm going to raise this internally to see why support is pushing you to
here, since I cannot handle what should be a support case through github.
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/AL/issues/7709#issuecomment-2050801708,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYZXW7PMO4RKQ37XX4UHWW3Y44ZRLAVCNFSM6AAAAABF5REBCCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANJQHAYDCNZQHA
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
This email may contain confidential or privileged information.  If you
believe you have received this email in error,
please notify the sender
by reply email and then delete this email immediately.

That's interesting because the PublicWebBaseUrl is used to determine the URI to handle web client requests, so if that was set wrongly you shouldn't have been able to even access BC through the browser. It's also a component of where we send our VSCode requests too, so it will have an effect on debugging.
It shouldn't hang on breakpoints in other apps though, it should work given the settings you mentioned.


Fix slash, and call cmd args in test_runner.sh.

Fixes the incomplete #117 implementation.

I'm skipping the OSX testing, as the Linux passes and I want to get this moving.


[Feature Request] Line break (\n)  support in table cell

Existing Component
Yes
Component Name
table
Description
There is no way to override table cell style to enable line break in table cell.
Please add line break support in table cell or a way to override cell styles.

I think it can be overridden. Please write a demo so that I'll know what goes wrong.

Here is the demo
https://jsfiddle.net/gk7b0Lwb/
documentation says "class-name" is class name of cells in the column but ".el-table .cell" styles in theme overriding given styles.

How about this?
@nidin
.el-table .line-break .cell {
  white-space: pre;
}

@wacky6 Perfect, Issue solved.


Feature Request - Canvas membership API  max page size =50

Is your feature request related to a problem? Please describe.
At present there may be a issue if a Canvas course has over 50 registered students. based on the Canvas Membership API which has a default limit of 50
module Lti::Ims::Providers
class MembershipsProvider
include Api::V1::User
MAX_PAGE_SIZE = 50

See https://github.com/instructure/canvas-lms/blob/master/app/controllers/lti/ims/providers/memberships_provider.rb & https://github.com/instructure/canvas-lms/wiki/Settings-(customization)
Describe the solution you'd like
Update the Learn LTI application to retrieve the full class membership
Code https://github.com/microsoft/Learn-LTI/blob/main/backend/ThirdParty/LtiAdvantage/LtiAdvantage/NamesRoleProvisioningService/GetMembershipRequest.cs
Describe alternatives you've considered
For self hosted solution the Canvas source code at See https://github.com/instructure/canvas-lms/blob/master/app/controllers/lti/ims/providers/memberships_provider.rb & https://github.com/instructure/canvas-lms/wiki/Settings-(customization)  can be modified to return more than the default 50 members of the course.
Additional context
This issue seems to be only effecting Canvas SaaS based solutions hosted by Canvas Infrastructure

PR to fix this issue https://github.com/microsoft/Learn-LTI/pull/171


Integration with CI is not working on zafira

Integration with CI is not working on zafira
Impossible to Rerun-test from Jenkins

it should be fixed in scope of the integrations fixes

Tested on local env. Fixed.


SCIM Group Lookup Failures

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or other comments that do not add relevant new information or questions, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Terraform Version
1.5
Affected Resource(s)
zpa_policy_access_rule
Terraform Configuration Files

data "zpa_scim_groups" "group_abc" {
  name     = "abc"
  idp_name = "idp-name"
}

Debug Output
N/A
Panic Output
N/A
Expected Behavior
Group lookup successful
Actual Behavior
Group lookup failing
Error: FAILED: GET, https://config.private.zscaler.com/userconfig/v1/customers/***/scimgroup/idpId/<customernumber>?page=1&pagesize=500&search=abc&sortBy=name&sortOrder=ASC, 400, 400 Bad Request, {
  "params" : [ "idpId" ],
  "id" : "invalid.resource.argument",
  "reason" : "One or more input fields idpId are not valid."
}
  with data.zpa_scim_groups.abc,
  on user_groups.tf line 38, in data "zpa_scim_groups" "group_abc":
  38: data "zpa_scim_groups" "group_abc" {

Steps to Reproduce
data lookup
Important Factoids
Starting the morning of 7/12, I have had several group start to fail their data lookup when drift ran.  Not all of my group updates are failing though.  I did try and run the API manually against our environment and I get data back.  For example, I ran:
/bin/curl -s -x $PXY:80 -H "Content-Type: application/json" -H "Authorization: Bearer $TOKEN" -X GET "https://$ZPAAPI/userconfig/v1/customers/$ZPA_CST_ID/scimgroup/idpId/$IDP_ID?page=2&pagesize=500&search=abc&sortBy=name&sortOrder=ASC"

I also confirmed that I see the same groups that are erroring out, in the ZPA admin console, so I know they exist.
No changes that I'm aware of were made to the repo, IDP, or ZPA tenant.  I did try and update the module to 3.31.0, and it didn't make any difference.  I did also open a ticket with Zscaler support, but figured I'd add an issue here too.
References
N/A

#0000

@watnenatePFG We recommend to raise a case with the Zscaler support team, as the issue is unrelated to the Terraform Provider or the underlying constructs.
Zscaler DevRel

Thanks, William.  Case 05013330 was opened on Friday.  Based off my limited testing it appears there is a bad server answer API calls.  I run the same group lookup call several times in a row and I get a periodic parameter error back.  Waiting on a response from the backend team.

Issue has been resolved. Please check with Zscaler support for further updates.
Zscaler DevRel


how to use with Reagent

I would like to use infamous as a component in my reagent application, I don't want to write the entire web-app in famous.
I've tried unsuccessfully to integrate infamous as a component, by taking the carousel demo and trying to render it to a [:div#carousel] using the reagent component-did-mount lifecycle method.
I can't get it working properly. (it briefly flashes an image up, but then disappears and does odd things)
Would you be able to explain the best pattern for doing this, that works with both the reagent and infamous paradigms?
I'm mainly wanting to use infamous as a wrapper around my pages's main content, so that I can apply nice transitions to page changes, rather than have to run the entire application as a famous app.

famous.js has been discontinued so I stopped developing this library. I then looged into using three.js and created https://github.com/sonwh98/four.  Here's an example of the famous periodical table implement with clojurescript using three.js http://shiva.kaicode.com/elements.html
However, three.js does not play well with reagent.  I am working on two independent clojure/clojurescript libraries to allow me do what famous.js and three.js allows me to do.

https://github.com/sonwh98/morpheus matrix manipulation
https://github.com/sonwh98/tweenie tweening animation

I will put up an example project in the next couple of weeks

Wow wasn't aware that Famous had been discontinued!
Thought they just moved everything to Famous Engine.
Your elements demo is amazing ... Can't wait to see where you get with this.
Thanks for all the info, will take a proper look tomorrow.

Here is sample project https://github.com/sonwh98/food-truck
Look at layout.cljs for on-screen and off-screen functions to see how to tween animation

it won't boot for me
java.io.FileNotFoundException: resources/schema.edn (No such file or directory)

sorry for got to commit the files. try it now


feature request: a simpler logging service accepting log message and log payload

Consider adding a simpler logging service for users who prefer calling a method such as logCritical('log message', { thisIs: 'my log payload' }) rather than passing full log objects or using declarative logger services.

Do we need to create a new service, or can we expose the functions?
WDYT @LayZeeDK @NachoVazquez
we can do logCritical(params) {
const service = inject(LoggerService).log()
}

I don't completely follow. Could you describe the usage API?

The API will look like
export function logCritical(message: string, payload: LumberjackLogPayload) {   return inject(LumberjackService).log({     message: message,     payload: payload,     level: LumberjackLevel.Critical,     createdAt: Date.now(),     scope: 'Forest App',   }); }
And we can use it like below.
 logCritical('The forest is on fire', { angularVersion: VERSION.full })

I think that's interesting. We could definitely iterate over the idea.
Good thoughts

Great, should I try to create a POC and open a PR with this idea? Or do we want to go with Services in the first iteration?

Sorry, @santoshyadavdev I missed this notification. Go ahead with this idea POC. :)

Hey! Is this issue still open? I'd like to help and looking for a starting point :smile:

Hi @pBouillon
Yes, this is still open and relevant. We are currently working on a major rework of Lumberjack to make it framework-agnostic (see #166) so we won't accept any features or publish new feature releases before that work is done. But please feel free to draft design documents, implementation, and tests for this feature.

I'm still new to the process, what do you mean by drafting and testing this? Should I wait for the v16 and suggest changes then, based on the rework or write what the API might look like, if not what @santoshyadavdev already suggested but without referencing inject since it is Angular-related?
On a side note, this is a huge refacto, good luck! 🙌

@pBouillon
The partial implementation suggested doesn't quite make sense since scope isn't known in this example.

I'm still new to the process, what do you mean by drafting and testing this?
I mean feel free to start writing code, tests, and documentation and we will help you rework your contribution after the framework-agnostic release of Lumberjack.

without referencing inject since it is Angular-related?
We will still have Angular-specific packages:

@lumberjackjs/angular
@lumberjackjs/angular-http-driver


Fix document.documentElement should return HtmlHtmlElement #214

Fix #214

These APIs are generated. We don't want to patch things in this way.
CC @srujzs

I think you're right; this issue seems a bit strange.

You're saying

The MDN notes also mention that it can be other types of Elements: https://developer.mozilla.org/en-US/docs/Web/API/Document/documentElement#notes.

at link^

Document.documentElement returns the [Element]

...which it is an Element. But I'd say a super type, not an other type.
Especially since https://developer.mozilla.org/en-US/docs/Web/API/HTMLHtmlElement says

You can retrieve the HTMLHtmlElement object for a given document by reading the value of the document.documentElement property.

Sorry, I wasn't clear what I was referring to there. Specifically this sentence:

For any non-empty XML document, documentElement will always be whatever element is the root element of the document.

This sentence leads me to believe this getter can return other types of Elements beyond just html elements, so if we made this HTMLHtmlElement, it might be wrong in some cases.


MistralForCasualLM object has no attribute merge_and_upload

System Info
petf 0.7.1 accellerate transfomers
Who can help?
No response
Information

[ ] The official example scripts
[x] My own modified scripts

Tasks

[ ] An officially supported task in the examples folder
[ ] My own task or dataset (give details below)

Reproduction
I have trained a fine tuned version of Mistral-7B-v0.1 using accellerate and saved the adapters to huggingface at https://huggingface.co/giux78/zefiro-ita-7b-sft-lora/tree/main. Once I try to merge adapters using
model_id = "mistralai/Mistral-7B-v0.1"
peft_model_id = "giux78/zefiro-ita-7b-sft-lora"

tokenizer = AutoTokenizer.from_pretrained("giux78/zefiro-ita-7b-sft-lora", padding_side="left")

#model = AutoModelForCausalLM.from_pretrained(model_id)

model = AutoModelForCausalLM.from_pretrained(
        model_id)

model_fin = PeftModel.from_pretrained(model, peft_model_id)
model_fin.merge_and_upload()

it gave this error
MistralForCasualLM object has no attribute merge_and_upload
seems the base model wrapper  MistralForCasualLM does not have this method. Is that correct?
Expected behavior
merge_and_upload should give in return the merged model.

sorry it was a typo but I have the error with merge_and_unload


Add google/bundletool

Tool name
google/bundletool
Tool license
Apache License 2.0
Add or update?

[X] Add
[ ] Update

Desired version
1.10.0
Approximate size
No response
Brief description of tool
Bundletool is a tool to manipulate Android App Bundles. The Android App Bundle is a new format for publishing Android apps in app distribution stores such as Google Play.
Bundletool has a few different responsibilities:
Build an Android App Bundle from pre-compiled modules of a project.
Generate an APK Set archive containing APKs for all possible devices.
Extract APK(s) from the APK Set compatible with a given device.
Install APK(s) from the APK Set compatible with a connected device.
Extract device spec from a device as a JSON file.
Add code transparency to an Android App Bundle.
Verify code transparency inside an Android App Bundle, APK files or an application installed on a connected device.
URL for tool's homepage
https://github.com/google/bundletool
Provide a basic test case to validate the tool's functionality.
bundletool version

Platforms where you need the tool

[X] Azure DevOps
[X] GitHub Actions

Virtual environments where you need the tool

[X] Ubuntu 18.04
[X] Ubuntu 20.04
[X] Ubuntu 22.04
[X] macOS 10.15
[X] macOS 11
[X] macOS 12
[X] Windows Server 2019
[X] Windows Server 2022

Can this tool be installed during the build?
brew install bundletool

Tool installation time in runtime
3m 30s
Are you willing to submit a PR?
No response

Hello @nevardreik , we will take a look

@nevardreik we wouldn't like to use homebrew bundletool package as it would bring OpenJDK as well.
Please download and use the jar file directly as it's pretty simple and quick:

windows:

$json = Invoke-RestMethod -Uri "https://api.github.com/repos/google/bundletool/releases/latest"
$downloadUrl = $json.assets.browser_download_url
$filePath = ".\bundletool.jar"
(New-Object System.Net.WebClient).DownloadFile($downloadUrl, $filePath)
java -jar bundletool.jar

macos

json=$(curl -s "https://api.github.com/repos/google/bundletool/releases/latest")
downloadUrl=$(echo $json | jq -r ".assets | .[].browser_download_url")
curl $downloadUrl -4 -sL -o 'bundletool.jar'
java -jar bundletool.jar version

ubuntu:

json=$(curl -s https://api.github.com/repos/google/bundletool/releases/latest)
downloadUrl=$(echo $json | jq -r ".assets | .[].browser_download_url")
curl $downloadUrl -4 -sL -o 'bundletool.jar'
java -jar bundletool.jar version

I'm going to close the issue. Please feel free to contact us if you have any concerns.
Thank you!


Change TileOverlay cause OOM

mapView.getOverlayManager().setTilesOverlay(tilesOverlay);
ZoomIn zoomOut and scroll the map,   Memory usage gets bigger and bigger.
Version of osmdroid the issue relates to:
6.0.1

@qq86262363 There were recent changes in the code regarding memory cache.
Now we specifically keep in memory the tiles that are displayed, and also the corresponding tiles for zoom +-1 (cf. the constructor and the garbageCollection method in MapTileCache).
The cache capacity is no longer a relevant value regarding UX. Could you try with a low value for the capacity, something like Configuration.getInstance().setCacheMapTileCount(0)?

It cant solve the porblem .
If I havent call the method 'mapView.getOverlayManager().setTilesOverlay(tilesOverlay);', its no problem .
Has I change the config after call the method ?

mapNikTileOverlay = new TilesOverlay(new MapTileProviderBasic(this, TileSourceFactory.MAPNIK), this);         mapView.getOverlayManager().setTilesOverlay(mapNikTileOverlay);
I just do like that and it went wrong. What should I do if I want to change the base tileOverlay?

i'm looking into it now

what exactly is in your tile chain?

Just use the default tile chain TileSourceFactory.MAPNIK ,it went wrong .
I think i find the problem , if I call the method mapView.setTileSource(TileSourceFactory.MAPNIK),its no problem .


ActiveWindowSensor shows COD title wrong

ActiveWindowSensor shows Call Of Duty Modern Warfare / Warzone as
?C?a?l?l? ?o?f? ?D?u?t?y?®?:? ?M?o?d?e?r?n? ?W?a?r?f?a?r?e?®???????????????????????????????????????????????????????????????????
And I copied this exact string into HA but it won't work for some reason.
It used to work just fine yesterday and it still does for say Apex Legends and even after re-copying the string from HA dev tools it doesn't work.

I'm copying your own workaround from Discord here:
template:
  sensor:
    - name: Current Window Fixed
      state: '{{ states("sensor.focussedwindow").replace("?" , "")|replace("®", "") }}'

I believe this will be good enough for now. I may reopen this later if more application titles cause the same problem.


iPad DrawerController's bottom corner regression

Environment Information

Platform:

[x] iOS
[ ] macOS

Package version(s): 0.1.17
Xcode and OS versions: (fill this out if relevant)

Please provide a reproduction of the bug:

Actual behavior:

Expected behavior:

Priorities and help requested:
Are you willing to submit a PR to fix? (Yes, No)
Requested priority: (Blocking, High, Normal, Low)
Products/applications affected: (if applicable)

Fixed by #291


[ARP] working agreement: make reviewer workflow

Please leave this PR open until the ARP working group discusses and votes.
This PR is built on top of https://github.com/cloudfoundry/community/pull/155

+1

+1

+1

+1


fix button

fix button css

@cnrdgrgry You snooze you lose 😆

Wtf?! 😬😂

I was on the beers already.
Skool'z out for Summer!

@cnrdgrgry We accidentally assigned you


Fix for small documentation error for save-data-file argument

Hi,
I was flustered by the documentation of the save-data-file argument. It states that if save-data-file is true, the current benchmark will not be saved to the external data file, which doesn't make much sense given the name of the argument. After running the action in one of my repos, I realized that this is a mistake in the documentation.
Because this is such a small fix, I didn't bother raising an issue beforehand, hope that's alright.

Thank you @jofas for this doc fix! Much appreciated!

The CI failure is related to some underlining GH action runner changes. I'll fix that in a sec 😅


Refuses the request

Type: Bug
make an empty  js programing file
then ask it to genrate some code
then it will respond with Sorry, I can't assist with that.
Extension version: 0.23.1
VS Code version: Code 1.96.0 (138f619c86f1199955d53b4166bef66ef252935c, 2024-12-11T02:29:09.626Z)
OS version: Windows_NT x64 10.0.22631
Modes:

https://github.com/microsoft/vscode-copilot-release/issues/2625


Sub-intents

Is your feature request related to a problem? Please describe.
I'm not sure how I can have intents work for sub-dialogs (sub-intents). I can only add intent to the main dialog and have it work just from there. When I move from the main dialog to another and try to add intents there, it will process the user input according to the main dialog.
Describe the solution you'd like
Be able to have intents working in sub-dialogs.
Describe alternatives you've considered
I'm not really sure if this feature is supported at the moment.
Additional context

@0x454447415244 you need to enable luis or regex recognizer in dialog setting. then you can create trigger for sub dialog.

@hibrenda I'm already creating triggers for sub-dialogs. But how do I recognize intents in sub-dialogs? It doesn't work. It falls back to the main dialog.

I have room cleaning and room reservation intents. They work fine.

Intents placed in reservation sub-dialog do not work.

@0x454447415244  this is because in  RoomReservation , all actions have been executed in 'BeginDialog' and then return back to main. There is no input prompts inside the sub dialog to  receive utterance from user. eg. you can add an input prompt in 'BeginDialog' for "what kind of room do you want to book?". In that case, it will wait for user's input. If user say "big", it will trigger the "big" flow. and after "big" flow handled,  it will return to main dialog as well. If you still want to keep in this sub dialog, you can design your dialog with more prompts.

@hibrenda How do you use RegEx or LUIS with User Input? Below is an image where for example I get the room number. But in general, how do I use RegEx with user input? I only know how to use them with trigger in the main dialog.

Your help is really appreciated. Thanks!

@0x454447415244  you can reference https://docs.microsoft.com/en-us/composer/tutorial/tutorial-get-weather#get-weather-report or create a sample based on the template.

@hibrenda In both examples, I can't see where they have sub-intents (intents in sub-dialogs). All I see in sub-dialogs is input text processing and validation.

you can reference the tutorial to define the regex as the recognizer type. and then create a trigger ,and then add the action (input prompts) to hook with the trigger. (you have already done that part) so this is the common steps for each dialog. no mater it is main dialog or sub dialog. above samples showed how you define the input prompts.

No, I haven't done that part. What I did is trigger a new dialog based on an intent captured via regex.
The main problem is that I can't capture intents in the sub dialog that I opened.
I can assign a variable to the input prompt and do basic input check, but I can't use LUIS or Regex on this variable to trigger anything.
Am I missing something?

@0x454447415244 sorry for the late response.  You can edit your input prompts with "Allow Interruptions" to true as below. This option is in "Other" tab.

@hibrenda  allowing interruption for the input prompt will allow triggering intents from the main dialog. This is not what I'm asking for.
I need the ability to have child intents. For example in a chat bot designed for a hotel you could have 2 intents: Reservation and Room Service. Under Room Service intent I should be able to define sub/child intents, like for example: Cleaning, Repair, Food, etc...
How to do that? I hope my explanation is clear enough and thanks for your help.

EchoBot-1.zip
No. The trigger intent in child dialog can also be triggered.  see above sample code.

@hibrenda Thanks for sharing this. How can I try this project? I tried to load it with Bot Framework Composer but it doesn't work.

you can load it with in master branch.

@hibrenda Can you please elaborate?

@0x454447415244 You should be able to extract the files from the zipped file, and then open the bot assets in Composer by clicking the 'Open' menu option on the home page and navigating to the file you unzipped. If you are still having an issue with this, please let us know.

@0x454447415244 Does @tdurnford 's answer resolve your issue?

Closing due to inactivity.


Add download route to CDN

Codecov Report

Merging #169 into develop will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #169   +/-   ##
========================================
  Coverage    41.77%   41.77%
========================================
  Files           11       11
  Lines           79       79
  Branches         7        7
========================================
  Hits            33       33
  Misses          45       45
  Partials         1        1

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 ec20811...20ef959. Read the comment docs.


Abd parameter gui feature

Added a feature to allow the gui to override parameters in the simulation TB command

See screenshot:

Impresive. I will wait till the end of maifa_sanity and close that PR!


minify support

Maybe we can support min flag for unbuild and unify tooling?

see https://github.com/nuxt-community/color-mode-module/pull/118#discussion_r796728630

Is there any workaround?

@seanghay I found this config seemed to work:
  rollup: {
    esbuild: {
      minify: true,
    },
  },

I also have this problem, and my unbuild config this:
import { defineBuildConfig } from "unbuild";

export default defineBuildConfig({
  entries: ["./index"],
  declaration: true,
  clean: true,
  failOnWarn: false,
  rollup: {
    emitCJS: true,
    esbuild: {
      minify: true
    }
  }
});

But After I pnpm build , I get this error:
Error building E:/project/changan-incall/packages/microApp: Error: Transform failed with 1 error:
<stdin>:1:10: ERROR: Expected ";" but found "EnterMicroAppParams"
Error: Transform failed with 1 error:
<stdin>:1:10: ERROR: Expected ";" but found "EnterMicroAppParams"
    at failureErrorWithLog (E:\project\changan-incall\node_modules\.pnpm\esbuild@0.15.6\node_modules\esbuild\lib\main.js:1624:15)
    at E:\project\changan-incall\node_modules\.pnpm\esbuild@0.15.6\node_modules\esbuild\lib\main.js:1413:29
    at E:\project\changan-incall\node_modules\.pnpm\esbuild@0.15.6\node_modules\esbuild\lib\main.js:678:9
    at handleIncomingPacket (E:\project\changan-incall\node_modules\.pnpm\esbuild@0.15.6\node_modules\esbuild\lib\main.js:775:9)
    at Socket.readFromStdout (E:\project\changan-incall\node_modules\.pnpm\esbuild@0.15.6\node_modules\esbuild\lib\main.js:644:7)
    at Socket.emit (node:events:526:28)
    at addChunk (node:internal/streams/readable:315:12)
    at readableAddChunk (node:internal/streams/readable:289:9)
    at Socket.Readable.push (node:internal/streams/readable:228:10)
    at Pipe.onStreamRead (node:internal/stream_base_commons:190:23) {
  errors: [
    {
      detail: undefined,
      id: '',
      location: [Object],
      notes: [],
      pluginName: '',
      text: 'Expected ";" but found "EnterMicroAppParams"'
    }
  ],
  warnings: [],
  code: 'PLUGIN_ERROR',
  plugin: 'esbuild',
  hook: 'renderChunk'
}

How Can I resolve this ?

But after pnpm build , I get this error:

I have encountered this error as well. It may be caused by esbuild trying to minify a ts/d.ts file.


Enable discussions

I'd like to suggest that github discussions be enabled or maybe there is some other forum where questions can be asked?

thanks for the suggestion. in general, we don't use GitHub discussions in the open source projects we maintain. as this is a repo with sample code I don't think a GitHub discussions space would see much participation; I can, however, recommend the Rust users forum and the rust-lang discord server as ways to engage with the broad community -- see https://www.rust-lang.org/community for links to those spaces.
There's also Ferrous Systems' rust-experts service if you would like to tap into our expertise.


Sparse file support operation unmap

longhorn/longhorn#836

Does the extra wrapper mean the IO Processors?

LGTM, not sure if we need the extra wrapper for FiemapFile since all the code seems to act on the os.File Fd

We still need another wrap for the below.
func (file *BufferedFileIoProcessor) UnmapAt(length uint32, offset int64) (int, error) {
	return int(length), file.PunchHole(offset, int64(length))
}

cc @shuo-wu


add EbrCell to trophy case

Thanks @Firstyear for letting me know about this :)

@bors r+

:pushpin: Commit 4069302a8ce80b51c4a6ae4dcdeb4fb14a0f4e2c has been approved by RalfJung

:hourglass: Testing commit 4069302a8ce80b51c4a6ae4dcdeb4fb14a0f4e2c with merge bc6528143855b02ec49b0ed07d9dbf75e66b1d9c...

:sunny: Test successful - checks-travis, status-appveyor
Approved by: RalfJung
Pushing bc6528143855b02ec49b0ed07d9dbf75e66b1d9c to master...


Add license

Please consider adding a license to the project so that others are permited to use it. You can easily do so by clicking this link: https://github.com/thecodeboss/AcousticSimulator/community/license/new?branch=master
Thank you!

@nwoeanhinnogaehr thanks for the suggestion, I added an MIT license to the project. If you're planning to use it, definitely let me know how it goes.


Raspbian Buster Lite

Hi.
I'm trying to use rpi-backlight on a raspberry-pi using Raspbian Buster Lite and the official 7" touchscreen.
I got the error /sys/class/backlight/rpi_backlight/bl_power: No such file or directory when I try to power off with rpi-backlight --p off
Directory /sys/class/backlight is empty.
Do I need to install some dependencies?

Do I need to install some dependencies?

Not that I'm aware of. Please make sure the display ist connected properly and get in touch with the raspberry pi kernel developers - I'm unable to troubleshoot issues not directly related to this project.


⚠ Bofrost.lu has degraded performance

In de0e134, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1849 ms

Resolved: Bofrost.lu performance has improved in 90e520b after 6 minutes.


Slack notifiers should only allow Slack webhook URLs

Clearly describe the problem and solution
Allowing all values for the Slack webhook URL introduces potential of attacks like SSRF. The only value allowed here should be a valid Slack webhook URL.
Relevant issue number(s) if applicable
HackerOne report # 721512
Did you run npm run lint from the viewer or parliament directory (whichever you are making changes to) and correct any errors
Did you Ensure that all tests still pass by navigating to the tests directory and running ./tests.pl --viewer
License
I confirm that this contribution is made under an Apache 2.0 license and that I have the authority necessary to make this contribution on behalf of its copyright owner.

Thanks, but we want to stay compatible with mattermost


Write a getting started for the Website

The idea is to have a getting started guide for the interaction with the Kuksa Databroker including the CLI and SDK.
I am aware of the documentations in https://github.com/eclipse/kuksa.val/tree/master/kuksa_databroker or https://github.com/eclipse/kuksa.val/tree/master/kuksa-client/docs .
But I am more thinking about an overview and hand-on description referencing the other documentation.
@SebastianSchildt @lukasmittag what do you think?

I do not know how much sense this makes. The way I see it that our repository is more popular than the website itself. I could be wrong. Either way a page how to get started could not hurt. Like we saw at the PS Hacksthon it would be good to have a hello world example. But we need to define a scope for the getting started. Is it just for the VAL or do we want to include feeders/providers and services too?

The "kind of" documentation you search is likely this or an extended take on it https://github.com/eclipse/kuksa.val/blob/master/doc/quickstart.md
If you did not find it, discoverability might be an issue. I think we should NOT use website (as in "this repo") to write some documentation on the website. we had it once (i think the fiels are still here), and is was disconnected form code, useless and outdated.
What I could imagine, if there were some way to transform/adapt all those mrkdowns here https://github.com/eclipse/kuksa.val/tree/master/doc into some form + CI that it is also Hugo generated. That might be seperate from the "main" marketing page (only linked prominently). That would go more in the direction of what Velocitas is doing https://websites.eclipseprojects.io/velocitas/docs/
Actually the issue was (as usual) "lack of time" in the team. If you can remix existing documentation and whip some static site generator/CI on top to turn it  into something more "approachable" that would be appreciated, however form Dev perspective is is important I think, that all that needs to be done to keep this updated,  updating markdown files in the main repo

Yes, https://github.com/eclipse/kuksa.val/blob/master/doc/quickstart.md is a very good foundation for what I was thinking of.
My target audience would have been interested developers who have seen the sticker and want to know what is behind that. So I would add a paragraph introducing the general idea of abstraction through VSS and the data broker.
I get your concerns to have multiple documentations which will diverge as soon as you merge the PR ;) . So an approach could be to link the Markdown file in GitHub from the website and keep the Markdown as single source of truth.
Regarding the "lack of time", I would volunteer to spend some time on adding the reference to the website and looking into how the site generator could use the markdown files from the main repo. The intention of creating this issue was therefore mostly to see if there is interest or if I am missing parallel activities.


Run with Tekton

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: carlossg
If they are not already assigned, you can assign the PR to them by writing /assign @carlossg in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/meow

@carlossg:

In response to this:

/meow

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@carlossg: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

serverless-jenkins
6fe671d456e80d6942bd94f032360b17fbcb5eaf
link
/test this

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

/test this

:star: PR built and available in a preview environment carlossg-croc-hunter-jenkinsx-serverless-pr-49

:star: PR built and available in a preview environment carlossg-croc-hunter-jenkinsx-serverless-pr-49

@carlossg: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

serverless-jenkins
e853165e521603c8cd68a976ae4376a7b536bfd0
link
/test this

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test

/test this

/meow

@carlossg:

In response to this:

/meow

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

:star: PR built and available in a preview environment carlossg-croc-hunter-jenkinsx-serverless-pr-49

/approve

/approve

/hold

/approve cancel

:star: PR built and available in a preview environment carlossg-croc-hunter-jenkinsx-serverless-pr-49

@carlossg: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

serverless-jenkins
e853165e521603c8cd68a976ae4376a7b536bfd0
link
/test this

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

:star: PR built and available in a preview environment carlossg-croc-hunter-jenkinsx-serverless-pr-49


⚠ ClamAV has degraded performance

In 8fc9168, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 120 ms

Resolved: ClamAV performance has improved in e541854.


pullsync: sync only in nn

#818

Hi @acud , you closed this PR, but is the issue still relevant? Do we put it in the backlog or in the active sprint backlog?

/run beekeeper


Send to Unreal - Discussion - Arrayed static meshes and hierarchical/instanced static meshes

Addon: Send To Unreal
Description:
It is common in archviz and other modelling techniques to take a complex mesh and instance it many times. This is possible in two ways inside Blender - array modifiers and alt+d (duplicate instance referencing same base mesh)
Meshes exported from Blender should maintain low polycounts even when used in large quantities (via arraying and instance referencing)
Workflow:
Blender:

Create a default cube
Add array modifier (relative offset -> 2 - No merge)
Alt+D to duplicate an instanced mesh and move 5 units away
Pipeline -> Export to Unreal

Unreal:

Current behaviour is to see two meshes produced - with two unique origin positions and 5 sets of polys in that mesh
Suggested new behaviour is to have a blueprint created including one mesh with a hierarchically instanced mesh in the relevant array positions [Open discussion point as to what outcome people would expect here]

This would be very useful indeed. I totally agree with the suggestion.


path of tsconfig didn't transform when use different default integrated terminal

VSCode Version:1.22.2
OS Version:Windows10 1703

Steps to Reproduce:

default integrated terminal to Git Bash in Windows
create a build task for a ts project
run the build task

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

Dups https://github.com/Microsoft/vscode/issues/38178


meetingID is allways generateRoomWithoutSeparator

Many thanks for this plugin.
However, we had to realize that for each meeting a random name is generated.
Is a special Mattermost version required to use the req.topic? Currently we are using version 5.8.0.

@w00dst0ck what was your experience setting this up? This project doesn't have updates for at least 11 months, so wondering if everything still works. Thanks in advance.

Hey @laurawadden  Yes, this does still work with all of the latest versions of Mattermost.
@w00dst0ck the plugin is currently set up to only have a random name generated. Is the functionality of having the name consistent based on the room name for example, or some other parameter something you would like to see?

@seansackowitz Thanks for the feedback.
Since the use of Jitsi is currently increasing rapidly due to Covid-19, it would be great if the following features were added.

Prefix for the Jitsi room name to avoid double assignments
Use of the Mattermost room name as Jitsi  room name
combination of both features

Best regards
Kai

Please have the option to use mattermost channel names or random names set in MM system console.
Why close the ticket if its still under analisys?

How is this coming along? Any progress?


Error in register of the ROTE - GATEWAY > EUREKA > APP

Good afternoon, I have an implementation of Spring Cloud Gateway and Eureka in boot version 2.x and spring 5. I have a problem that is happening when I configure the route without using yml. When configuring with yml like this:
server:
  port: 9090

eureka:
  client:
    serviceUrl:
      defaultZone: http://localhost:9761/eureka/

logging:
  pattern:
    console: "%d{yyyy-MM-dd HH:mm:ss} ${LOG_LEVEL_PATTERN:-%5p} %m%n"

spring:
  application:
    name: gateway-cloud
  cloud:
    gateway:
      discovery:
        locator:
          enabled: true
      routes:
      - id: acd-falta-aluno
        uri: lb://app-corp
        predicates:
        - Path=/**
        filters:
         - RewritePath=/app-corp/(?<path>.*), /$\{path}

Gateway> Eureka> Registered Service works.
I would like to know how to register the route without having to configure it in YML, since we are using and running ZUUL SERVER, which is configured using
@Configuration
public class ZuulServiceConfiguration {

	@LoadBalanced
	@Bean
	public RestTemplate restTemplate() {
		return new RestTemplate();
	}
}

. I already tried to use this in CLOUD GATEWAY, without success.
Thanks.
@spencergibb

http://cloud.spring.io/spring-cloud-static/Finchley.RELEASE/multi/multi__configuration_2.html#_discoveryclient_route_definition_locator
I'm on mobile or I'd proved l provide the example here

Can you provide a complete, minimal, verifiable sample that reproduces the problem?  It should be available as a GitHub (or similar) project or attached to this issue as a zip file.
If so, I'll reopen.

Hello,
As requested follows the file with the projects, thank you for the help.
Thanks


I tried using export $(cat .env) && astro build and it doesn't work

📚 Subject area/topic
environment variables
📋 Page(s) affected (or suggested, for new content)
https://docs.astro.build/en/guides/integrations-guide/node/
📋 Description of content that is out-of-date or incorrect
instead of using export $(cat .env.runtime) && astro build // Doesn't work
I use set -o allexport; source .env; set +o allexport; astro build// Work
🖥 Reproduction in StackBlitz (if reporting incorrect content or code samples)
No response

nvm.. both doesn't work most of the time


Spanish translation

it took me an embarrassingly long amount of time to figure out how to do a pull request, hope i did it right lmao

And it took me an embarrassingly long time to finally merge your hard work, whoops!
¡Muchas gracias, @santumerino ! :tada:

Yayy!!!


Robot exposes root access with no password to network

The OT-2 exposes passwordless root access to the entire network (I believe). Is this intended?

It's intended for troubleshooting. We know it's bad. We have work planned for Q1 and Q2 (but not yet on GH - will update with links to the tickets when they're created) to overhaul the userspace as a whole and move to an actual secure system. We take this very seriously and as a team are quite eager to fix it.

Following up on this. Will anything break if I change the root password?

As of 3.11.x, SSH access to the robot uses public key authentication (with no password access). You can add your public key to the robot via HTTP when you are connected via the USB-to-Ethernet connection
https://support.opentrons.com/en/articles/3287453-connecting-to-your-ot-2-with-ssh


limitedTo not working

I am trying to run some benchmarking tests so I want to limit the find to 1000 records
 let matchingEntities: CollectionSlice<Document> = try pages.find(nil, sortedBy: nil, projecting: nil, readConcern: nil, collation: nil, skipping: 0, limitedTo: 1000)
            print(try matchingEntities.count())

The print returns a count of 17696.  Am I misusing this command?

Ok, so the amount of documents returned is 1000, but the count returns a different number?

Your questions were a very hectic period that year. That API was very confusing in many ways, we've removed it since. The count was not using the limitedTo parameter, I'm pretty sure.


Patch artifacts

Reasons for making this change:
patch generates .rej and .orig files if all hunks were not applied.
Links to documentation supporting these rule changes:
https://linux.die.net/man/1/patch
If this is a new template:

Link to application or project’s homepage: https://en.wikipedia.org/wiki/Patch_(Unix) / http://savannah.gnu.org/projects/patch/

There are cases were patch uses # for rejects, and use ~ instead of .orig , but I am not confident about how what pattern could catch those without false positives.

@jayvdb thanks!


Fix parsing of data from PhiX removal

PR checklist
Updated REMOVE_PHIX_BBDUK module to correctly parse output for summary files. This PR will close #16

[x] This comment contains a description of changes (with reason).
[ ] If you've fixed a bug or added code that should be tested, add tests!
[ ] If you've added a new tool - have you followed the pipeline conventions in the contribution docs
[x] Make sure your code lints (nf-core lint).
[x] Ensure the test suite passes (nextflow run . -profile test,docker --outdir <outdir>).
[ ] Usage Documentation in docs/usage.md is updated.
[ ] Output Documentation in docs/output.md is updated.
[x] CHANGELOG.md is updated.
[ ] README.md is updated (including new tool citations and authors/contributors).

Be mindful of the possible differences in the changelog :)


In Information Entry => Honours Award Information, edit time there is no way to view the uploaded document.

Then please provide a UI sample so I can get a reference for proper design.


Cors Error

Hi, I am using this library to connect to Instagram api but when I run this I get CORS error and it doesn't connect. Is there any solution for it? I am using it inside my React JS project.

same


Added back support for mapping in gcloud config. This is not used by …

…Datalab
but will allow users of Datalab to use gcloud in bash cells.
Cleaned up some dead code around setting project ID.
Some other small cleanups.

LGTM


add Saúl Molinero's blog

Branch conflicts @PareshMayani. I guess https://github.com/GDGAhmedabad/Learning-Resources/pull/105/files commit has that blog link.

No this is different link! I will resolve this conflict and will send commit!

Yes. I checked, its different one. :)

It will require another commit to resolve this conflict and 2 commits should not be pushed for a single line contribution! Will send another PR


x1197—As TS (Katy) I want the Visium barcode associated with any destination labware to be on the release file for Visium columns

New column "Visium barcode" in the release file for the Visium release option

@KatyTudor  What precisely does "Visium barcode associated with any destination labware" mean? It sounds like it means the external barcode of the labware being released, which is usually in the Released labware barcode column. Or does it mean the external barcode linked to some labware (of some particular visium labware type?) prior to the one that is being released? Can you give us an example of the sequence of operation that would cause a value to be filled in for this column?

@khelwood in the visium process the external visium slide barcode is not the source barcode of the released labware, because the sample goes through numerous visium transfers. so on the release file the source barcode for the released labware is the previous 96 well plate barcode not the external visium slide barcode. But apparently Faculty use the external visium slide barcode when submitting to DNAP (only just found this out).

@KatyTudor Can you explain through example please? Either some specific history in the stan database, or like "Labware A (of labware type ) is transferred to labware B, which is of labware type , labware ? has an external barcode, labware ? is released, and the file should include the external barcode from labware ?"


Better CLI documentation

For reference, add one entire working CLI search command and one for download. Current instructions are too ambiguous

I've added a functioning CLI call to the quick start guide
https://github.com/SDFIdk/CDSETool/pull/29


Bugfix - add params to patch action service call

Params parameter was not forwarded to the service call from the dispatch method, just added the parameter to the call.

Thanks!  I'm going to merge this.  I recognize there are problems with the current solution when not using feathers-reactive.  It will take some more work to fix that scenario, later.  However, the primary use case it to use feathers-reactive, with which this fix works well.

Released as<EMAIL_ADDRESS>


featurep evil returns disabled even when enabled

Hi
Today I upgraded to the last commit (febeed252629cad0874bf0b76e82aae6cd6f63aa) and my configuration broke. My summarized configuration is:
(doom! :config (default +smartparens +evil +bindings))

This won't work because it doesn't detect that the evil feature is enabled. In this line, instead to load the evil configuration, it loads the emacs one. When I added to my configuration the :feature evil parameter, it worked like a charm. My guess is that featurep searchs the :feature parameter, but I haven't investigated it.
Thanks for your work, doom is very cool!

+evil is not a flag of default  module. evil load in :editor evil

You need to either have the :editor evil module enabled, or evil installed and loaded by the time the :config default module loads. The :config default module does not check for an +evil flag.


Grammar improvements in the SDK dev guides

Improved the Acrolinx score for how-to-dev-guide-csharp-sdk.md from 87 to 97.
Improved the Acrolinx score for how-to-dev-guide-java-sdk.md from 83 to 100.
Improved the Acrolinx score for how-to-dev-guide-js-sdk.md from 87 to 97.
Improved the Acrolinx score for how-to-dev-guide-py-sdk.md from 89 to 100.

#sign-off

@stevemunk @sinnypan - can you follow-up and check that the links work after this is live?
Also, IMPORTANT, can you address the suggestions by the validation and fix the "?view=" issues. You can do this in a follow up PR - ideally in the private repo. thanks.


pantheon.elementary-mail: 6.0.0 -> 6.1.0

Motivation for this change
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Relase notes)

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

[ ] Fits CONTRIBUTING.md.

I am using this with #130380 and find that when a Email account is set up, elementary-mail 6.1.0 stuck as it opened, can you reproduce this (probably will need the Pantheon 6 update as I haven't test it with Pantheon 5.x yet)?
Here is my workaround https://github.com/NixOS/nixpkgs/pull/130380/commits/b31302fa6784c0da04a99690d04afabb32553b55, however it will be okay to merge things here if the app works, as elementary-mail is not shipped as default in Pantheon before<PHONE_NUMBER>da972d15481e8a85a11752e376f6f7 is merged and I am not the maintainer of this package :-)

It seems you're correct. I'll close this PR in favor of yours, feel free to add yourself as a maintainer. 😀


Setting variable does not work when value has dash

For instance, setting dotenv -v AWS_REGION=eu-west-1 gives you an error of Unexpected argument AWS_REGION=eu-west-1. Expected variable in format variable=value

Using dotenv-cli v3.2.0 seems to fix this issue.

Another workaround is intercepting the dotenv cli command with cross-env to manually pass additional command line arguments
e.g.
dotenv -- cross-env NODE_OPTIONS='--inspect' next dev

We can improve this regex https://github.com/entropitor/dotenv-cli/blob/master/cli.js#L50
This is because of this security issue: https://github.com/entropitor/dotenv-cli/pull/57#issuecomment-951203381
Feel free to create a PR to fix this

I think the same bug applies here:
Input:

Output:

You're just seeing the default value because you haven't set PUBLICPATH.
That's how it's supposed to work I think
On Thu, 17 Mar 2022, 18:17 Moris rafol, @.***> wrote:

I think the same bug applies here:
Input:
[image: image]
https://user-images.githubusercontent.com/10247681/158856893-cd579c25-7f24-42f8-83dc-284c612c54ac.png
Output:
[image: image]
https://user-images.githubusercontent.com/10247681/158857001-5b7257d1-aa03-4fed-b7ba-8e9579fca1e2.png
—
Reply to this email directly, view it on GitHub
https://github.com/entropitor/dotenv-cli/issues/65#issuecomment-1071112570,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAG2YXJV55354FBIT5OFDHDVANSLNANCNFSM5NX4CTGA
.
You are receiving this because you commented.Message ID:
@.***>

yeah but } shouldn't be part of the default value :/

that's from https://www.npmjs.com/package/dotenv-expand

That's probably in dotenv-expand. This package is just a simple wrapper around that one...

https://github.com/motdotla/dotenv-expand/issues/70
yeah... it's a bug there, thanks.

Had a crack at a PR https://github.com/entropitor/dotenv-cli/pull/69


Set Username

How can users set there username as opposed to WHMCS randomly generating one?

Hi,
Currently this isn't possible out of the box, but you can modify the line https://github.com/pterodactyl/whmcs/blob/8783734bba56e9f1c30fd8c8030b0dced59b04e3/pterodactyl/pterodactyl.php#L276 to something like 'username' => pterodactyl_GetOption($params, 'username', pterodactyl_GenerateUsername()), and setup a configurable option with the name "username". Though keep in mind that the deployment will fail if it conflicts with any existing usernames and will require manual intervention through the admin area for the service to deploy.


Add text_on_image 138 by  ShatilKhan

Issue #638
Description:
Added text on image with some hover buttons as depicted in this pic
Preview:

Heartfelt Thanks for Your Contribution to Our Open Source Project!
            Dear @ShatilKhan,

            I trust this message finds you in good health and high spirits. I am reaching out with a heart full of gratitude for your recent contribution to our open source project. Your dedication has not only elevated the functionality of our project but has also exemplified the collaborative spirit that lies at the core of thriving open source communities.

            A heartfelt congratulations are in order for your exceptional work! Your commitment and expertise have played a pivotal role in the success of our project, and we consider ourselves truly fortunate to count you among our community members.

            I must extend my sincere apologies for the delay in responding to your pull request. Your patience and understanding during this time are immensely appreciated. As our project continues to grow, we acknowledge the imperative need to enhance the quality and response time for contributions. To address this, we've kickstarted a call for maintainers and reviewers who can help us uphold the high standards of our project and ensure swift responses to pull requests.

            Your involvement and insights would be invaluable in this initiative. If you, or someone you know, is interested in taking on a more active role in the project, please don't hesitate to reach out. We firmly believe that with a robust team of maintainers and reviewers, we can elevate both the quality of the project and the speed of our responses.

            On an exciting note, I am thrilled to share that we have activated the sponsorship button for our project. This development allows supporters like yourself to contribute to the sustainability and growth of the project. Your ongoing support, whether through code contributions or sponsorship, is genuinely appreciated.

            Once again, congratulations on your outstanding contribution, and thank you for being an indispensable part of our open source community. We eagerly anticipate your continued collaboration and shared success.

            Best regards,

            Ntem Kenyor K.
            Lead
            Kwickerhub


Tokens used in the program may not be used as configuration cell name or attributes.

$ cat test.k
module TEST
    configuration <foo> .K </foo>
                  <k> .K </k>

    syntax KItem ::= "foo"
endmodule

$ ./ext/k/k-distribution/target/release/k/bin/kompile --backend haskell test.k
-[Error] Compiler: Had 1 parsing errors.
[Error] Inner Parser: Parse error: unexpected token 'foo'.
	Source(/home/njr/co/github.com/nishantjr/boogie-semantics/./test.k)
	Location(2,20,2,23)

While in the above case we may simply rename <foo> to <FOO> and work around this issue, the following case is less easy to resolve:
$ cat test.k
module TEST
    imports ID

    configuration <foo multiplicity="*" type="Set"> .K </foo>
                  <k> .K </k>

    syntax KItem ::= "type" Id ";"
endmodule

$ ./ext/k/k-distribution/target/release/k/bin/kompile --backend haskell test.k
[Error] Compiler: Had 1 parsing errors.
[Error] Inner Parser: Parse error: unexpected token 'type'.
	Source(/home/njr/co/github.com/nishantjr/boogie-semantics/./test.k)
	Location(4,41,4,45)

This was apparently a regression. We had an earlier bug fix but refactoring of the inner parser generator caused that bug fix to break, and we apparently did not have any tests. It should be fixed in the PR I have upcoming.


[AIRFLOW-386] limit scope to user email only for github oauth

Dear Airflow Maintainers,
Please accept this PR that addresses the following issues:

AIRFLOW-386

Testing Done:

tested login on my dev instance

Current coverage is 64.68% (diff: 100%)

Merging #1700 into master will not change coverage

@@             master      #1700   diff @@
==========================================
  Files           125        125
  Lines          9308       9308
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           6021       6021
  Misses         3287       3287
  Partials          0          0

Powered by Codecov. Last update eb989dc...a1c4cd9

+1

Commit nazi here: please adhere to guidelines next time makes for tidy history ;-)

@bolkedebruin sorry -- what did i miss?

@bolkedebruin Good catch - I admit that I didn't see it either at first.
The title in the commit is limit scope to user email only AIRFLOW-386, which does not adhere to guidelines, where as the title in the PR follows the correct guidelines : [AIRFLOW-386] limit scope to user email only for github oauth
@jlowin I thought the pr tool did some checks on the title, but maybe it's checking the PR title not the commit title?

@r39132 indeed, it only checks (and will modify, if you let it) the PR title but not commits. Perhaps I can add that as an enhancement.