Refactor: APPX-87: Fix input passing

Changes:

use --kind flag for serviceDiscovery
use first argument passed by args as serviceName
require --kind flag to be set

I didn't dive into the appcat-cli codebase very deeply but from what I can see the changes look reasonable.

We'll have to replace golang.org/x/exp/slices with the stdlib one's once we upgraded to Go 1.21


Groupper spelling?

Is there a reason that Groupper is spelled with 2 p's? Group, Grouping, Grouper appear to be the proper spelling here. Is it a play on words? The spelling has made it all the way into our docs as well
https://react.fluentui.dev/?path=/docs/components-menu--default#groupping-items
Would like to know if this is somehow intentional, and if it was a misspelling, if it'd be possible to fix it in tabster.

Well, it was an attempt to use English of a non-native speaker. Like when you have run, you have runner — I thought when you have group, you also have groupper. Eventually I've discovered that it should have been one p, and the double p is a fish. So, it's the fishy part of our legacy :). And it won't be trivial to change it given the current consumption scenarios.

That totally makes sense. I think converting to standard spelling would be best for any external facing code/docs. The fish reference wouldn't be very obvious to an average user, so they would just see it as a misspelling.

Actually Groupper avoids the fish thing 😄
https://en.wikipedia.org/wiki/Grouper

"Groupper" looks very strange to me. Perhaps, just use "Group"


Question about Strings in APIs

I see a lot of String's used in contract parameters. Does this have to be the case? Could it be &String or &str. To put it another way, does the same data get deserialized with &String?
This leads to a lot of cloning when internally calling external API's.

I see a lot of String's used in contract parameters. Does this have to be the case? Could it be &String or &str. To put it another way, does the same data get deserialized with &String?
This leads to a lot of cloning when internally calling external API's.

Absolutely you can! But note that if the function is public and directly callable, the type that you have a reference or mutable reference to (& or &mut) must be something that implements BorshDeserialize and is therefore owned.
For example:
    pub fn get(&self, key: &String) {
        // sth here
    }

within #[near_bindgen] works because String can be deserialized into, where:
    pub fn get(&self, key: &str) {
        // sth here
    }

does not because str is not sized and cannot be deserialized into, and it cannot infer that you wanted to deserialize as a String and then only get a reference to the str.
Any functions that are private and not externally visible to call into (any without pub) you can have whatever types you want because there is no codegen for these methods.
Hope this info helps :) let me know if there is anything I can clarify

Thanks! That makes a lot of sense!
A follow up then would be what is the motivation of the Promise API to take an AccountId and not &AccountId?  I see in the recent PR to add events Cow was used to prevent clones.  I understand passing valid account ids, but shouldn't the internals use Cow or would this over complicate the API?
https://github.com/near/near-sdk-rs/blob/2d867cb27732fd680c40d18c9ef9a3b89eb4e0f5/near-sdk/src/promise.rs#L213

Thanks! That makes a lot of sense!
A follow up then would be what is the motivation of the Promise API to take an AccountId and not &AccountId? I see in the recent PR to add events Cow was used to prevent clones. I understand passing valid account ids, but shouldn't the internals use Cow or would this over complicate the API?
https://github.com/near/near-sdk-rs/blob/2d867cb27732fd680c40d18c9ef9a3b89eb4e0f5/near-sdk/src/promise.rs#L213

The reason why this can't be changed with the API's current form is:

Promise API relies on Drop to schedule the calls when it drops out of scope. This isn't a problem on its own but this would cause a lot of churn because a common pattern is to retrieve the AccountId within the current function, but then return the promise from the function and this is impossible to assign a lifetime in this case.
Can't use Cow for the same reason as above, but also this adds additional code size for this additional type to every contract, which isn't ideal.

We are making changes to how cross-contract calls happen, and this may come with a change to this Promise API, and this will be top on mind to make sure it is ergonomic.

Closing because unactionable right now (re-open if you feel otherwise)


Fix Dry::Core::Equalizer() usage

As raised by @bkuhlmann in bkuhlmann/wholable’s README, the version of dry-equalizer as moved to dry-core:

doesn’t appear to work properly or be well supported. Even the documentation is not fully linked on the main page.

This PR addresses the “doesn’t appear to work properly” issue. See details below.
Once this is merged and released I’ll also update the 1.0 version of the docs to ensure Equalizer properly appears in the sidebar.
Once both of these are done, I would also hope that the entirety of @bkuhlmann’s statement above is no longer true and can be updated 😄
Fix Dry::Core::Equalizer() usage
Provide a Dry::Core.Equalizer method that builds the module when called as include Dry::Core::Equalizer() as documented. Previously, this method did not exist at all, so only usages of the "legacy" Dry.Equalizer method would work. With this change, we can now appropriately expect Dry::Core.Equalizer to be the mainstream usage.
To ensure this, update the specs to expect Dry::Core::Equalizer() to be used by default, and provide a single spec to ensure that the legacy Dry::Equalizer() method continues to work.
Also update specs to remove direct requires of dry/core/equalizer, since a require of dry/core alone should be sufficient to load Equalizer when it is used, given we use Zeitwerk.
Remove all cherry-picked requires from specs and docs
While we're here, make the equivalent change of removing cherry-picked requires from all our other modules, both in specs and docs. These are no longer needed since we use Zeitwerk. A simple require "dry/core" alone is enough.

Tim: I would also hope that the entirety of @bkuhlmann’s statement above is no longer true and can be updated 😄

Thanks for fixing this! Yep, as soon as this change is folded in, I'll update the documentation on my side too. :bow:

FYI, I had a chat with @flash-gordon and we determined that the new Dry::Core.Equalizer method should be moved into dry/core.rb for it to be properly located in the right file for the namespace. I've done this above. With this change, everything works and we're happy to merge and then cut a release.

This is released as 1.0.1 and the 1.0 version of the docs now properly includes Equalizer in its section list: https://dry-rb.org/gems/dry-core/1.0/equalizer/
cc @bkuhlmann

:bow: Fixed! Updated documentation can be found here.


course resources revisions

Pre-Flight checklist

[x] Screenshots and design review for any changes that affect layout or styling

[x] Desktop screenshots
[x] Mobile width screenshots

[x] Testing

[x] Code is tested
[x] Changes have been manually tested

What are the relevant tickets?
No Ticket :) but this stuff was discussed in this thread
What's this PR do?

Switches taxonomy templates (after renaming terms.html to term.html). For context: View this. All thanks to @gumaerc!
Adjusts resource title so it looks fine on smaller screens
Hides/Does not render the learning resource type which doesn't have any resource
Checks if all learning resource types have no resources then print a single message "No Resources found."
Some more little tweaks.

How should this be manually tested?

Checkout this branch.
Verify all the above-mentioned points.
Verify that all things work as expected. (Some testing instructions are mentioned in the description of this PR)
Any use-case that needs to be tested.

Screenshots

@MAbdurrehman12 @pdpinch @Ferdi
A major issue I'm noticing here with our strategy is that if learning_resource_types is not set at all, the resource will not be included in this list.  A large amount of courses in mitocwcontent do not have this set on their resources.  Since we are using this taxonomy to generate these pages, if the taxonomy is not set then Hugo will not include the resource when compiling it.  Do we want to maybe include an "Other" category on the taxonomy template itself, querying specifically for pages that have learning_resource_types empty or null?

@MAbdurrehman12 Also, this isn't something that needs to be addressed in this PR but in general there seems to be some confusion surrounding the hyphen delimiters used in the syntax here i.e. {{- if etc... -}}.  The purpose of the hyphen delimiters is to trim white space surrounding the tag.  Mainly this should be used when inserting template syntax directly into something like an href property where you want to be certain there are no white space or line breaks i.e. .  I don't think there's really any harm in using it outside of that use case, it just looks a bit confusing.  For instance:
{{- $numberOfResourcesLimit := 10 -}}
{{- $showNoResourcesFoundMessage := true -}}
{{ $taxonomy := "learning_resource_types" }}

Someone looking at this might ask themselves why the hyphen syntax is on 2 of these variable assignments but not on the 3rd one.  There are some examples here but the gist of it is that if you do something like:

  {{ .Title }}

...then you get:

  The Title

...but conversely if you do:

  {{- .Title -}}

...then on the output you will get:

The Title

Again, this isn't really a big concern as really the only issue is some potential confusion as to why some stuff is being trimmed and some isn't as well as some messy HTML output.  I think we should probably look into using the --minify option on our HTML output anyway to reduce file size and load times.

seems to be some confusion surrounding the hyphen delimiters used in the syntax here i.e. {{- if etc... -}}

@gumaerc Yes, I think we should maintain consistency here. I'll go on and update the PR and exclude these unnecessary hyphen delimiters to improve consistency and avoid any confusion in the future.
Your suggestion is to use --minify instead of hyphen delimiters? (I was not sure that it improves the load time, if thats the case then we should definitely go with it)


Tasmota compilation

Does the tasmota build support temperature sensors like sht3x?
If not, could you please provide a compilation with support for these sensors? I would like to replace the original temperature sensor with the sht30 sensor.

https://tasmota.github.io/docs/Compile-your-build/

didn't check, but if it wasn't removed in the nspanel build it should be included by default
https://tasmota.github.io/docs/SHT30/

This feature is included only in tasmota-sensors and tasmota32 binaries


CronTaskController is unable to return list of tasks

The CronTaskController#getConfigurations is unable to return the list of cron tasks - it returns 406 Could not find acceptable representation. This is likely because of broken serialization to JSON. We need to create a test case which checks if the endpoint actually returns a valid response with at least one result inside.
NOTE: The returned crontasks JSON should look like this:
{
   "crontasks": [
      {uuid: ...., name: ....., etc },
      {uuid: ...., name: ....., etc },
      ......
   ]
}

@ptirador would you be interested in picking this one? :)

Sure! I'll take it.


Bug: touch/mobile devices don't scroll vertically at drag boundary

I have a table that contains draggable items that takes up about 25vh. On desktop, dragging to the overflow boundary scrolls the table. On mobile/responsive/touch, it does not.

@wesharper v0.0.37 has an assortment of scroll bug fixes. Is this fixed for you? And if not, can you provide a reproduction I can take a look at?

Looks like it works, thanks @sashamilenkovic!


Null pointer exception: educatorMainPanel

Blocked in authentication process:

Student user works as expected
Guest or Instructor authentication does not work even though the process is the same. Returns a null JPanel named educatorMainPanel.

Each user has a "Status" code in the excel document. I am using this code in a switch to determine which class to instantiate (student, guest, or instructor). It also determines which background image to display. This latter part is working.
When the status char is 'S' everything works - the user ID and panel display on authentication and the correct image is used. When the char is 'I' or 'G' - the correct image loads but it never displays as the panel is null.
Stack Trace:
Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
at com.arcologydesigns.Views.EducatorMainForm.createAndShowGUI(EducatorMainForm.java:61)
at com.arcologydesigns.Views.EducatorMainForm.(EducatorMainForm.java:36)
com.arcologydesigns.Views.AuthenticationForm$2.actionPerformed(AuthenticationForm.java:91)
Edit:
Possible Resolution:
Added { educatorMainPanel.setSize(300,300); } to EducatorMainForm constructor and it seems to work - instructors and guests are authenticated and the correct panel, user ID, and image show. I am not sure if this is correct or why it works. Leaving issue open for now

Closed as out of date / no longer applicable.


[otbn] Auto-generate linker script based on otbn.hjson

This patch creates mem_layout.py, which uses reggen and the data in
otbn.hjson to find the memory layout for the block. We then use this
in otbn-ld to automatically make a linker script with the right
layout.
We'll also want to use it in the simulator (which needs to know the
size of memory) and when we load up ELF files in a block-level
testbench (which needs to know which LMAs correspond to imem and
dmem).

This looks good, as we already do something similar in the main earl grey linker scripts.
It would be nice if we kept to only one templating library in the OT repo: we use mako in topgen (all the .tpl files), is there a reason we cannot use it here?

I've replaced the regexes with Mako. Sledgehammer installed :-D

Oops! Force-push fixes the deliberate typo that I was using to check error handling :-)

Force-push adds missing licence header. @lenary: would you mind giving this a quick once-over? I would pester Philipp, but it seems a little unfair since he's on holiday!


[$500] Task - Chat is not scrolled to bottom when edit a task

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

**Version Number:**1.4.56-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/runs/view/20318&group_by=cases:section_id&group_order=asc&group_id=291935
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause - Internal Team
Action Performed:

Open an existing chat
Click the '+' > Assign Task
Give the task as Title, Description and Assignee > Confirm Task
Click on the created task in the chat
Change the name > Save
Verify there's an audit trail of the change in the chat thread
Make more changes to the task (you can change the title, description, assignee) > click Save
Verify that eventually the chat isn't fully scrolled to the bottom of last message (or audit trail change)

Expected Result:
Chat should always scroll to the bottom of the last message ( you should see the last message after making a change)
Actual Result:
After making multiple edits to a task, the chat thread isn't scrolled to the bottom.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/e27c103f-3f38-4b7d-82f5-021d620d12a9
https://github.com/Expensify/App/assets/51066321/f0ac74bf-5975-40a1-9c3b-15521378e3ce
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01d64def74796adc96
Upwork Job ID:<PHONE_NUMBER>557900800
Last Price Increase: 2024-03-26
Automatic offers:

fedirjh | Reviewer | 0
gijoe0295 | Contributor | 0

@Christinadobrzyn FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

We think that this bug might be related to #vip-vsp

Same issue as https://github.com/Expensify/App/issues/38855

Hum, I'm not able to reproduce this, it looks like the chat is at the bottom and the cursor is in the chat field when making edits to an assigned task.
https://github.com/Expensify/App/assets/51066321/9edda5d6-c31d-4004-88be-91f74ee70776
Asking QA to clarify what the bug is.

Ah okay, I think I'm able to reproduce, updated the OP so it's more clear how to reproduce. I think this can be external and part of vip-vsb

Proposal
Please re-state the problem that we are trying to solve in this issue.
After making multiple edits to a task, the chat thread isn't scrolled to the bottom.
What is the root cause of that problem?
We don't call notifyNewAction after we edit the task.
https://github.com/Expensify/App/blob/d8361eee3486ccb5472060f2af7655e96461471e/src/libs/actions/Report.ts#L344-L345
If it's currently a bug, it also happens when we edit a money request.
What changes do you think we should make in order to solve the problem?
After edit a task or edit a money request, we should call notifyNewAction to scroll to bottom of the report.
https://github.com/Expensify/App/blob/d8361eee3486ccb5472060f2af7655e96461471e/src/libs/actions/Task.ts#L299
https://github.com/Expensify/App/blob/d8361eee3486ccb5472060f2af7655e96461471e/src/libs/actions/Task.ts#L365
https://github.com/Expensify/App/blob/d8361eee3486ccb5472060f2af7655e96461471e/src/libs/actions/Task.ts#L429
https://github.com/Expensify/App/blob/d8361eee3486ccb5472060f2af7655e96461471e/src/libs/actions/Task.ts#L504
https://github.com/Expensify/App/blob/d8361eee3486ccb5472060f2af7655e96461471e/src/libs/actions/Task.ts#L795
https://github.com/Expensify/App/blob/d8361eee3486ccb5472060f2af7655e96461471e/src/libs/actions/IOU.ts#L2053
https://github.com/Expensify/App/blob/d8361eee3486ccb5472060f2af7655e96461471e/src/libs/actions/IOU.ts#L2030
https://github.com/Expensify/App/blob/d8361eee3486ccb5472060f2af7655e96461471e/src/libs/actions/IOU.ts#L2014
https://github.com/Expensify/App/blob/d8361eee3486ccb5472060f2af7655e96461471e/src/libs/actions/IOU.ts#L1975
https://github.com/Expensify/App/blob/d8361eee3486ccb5472060f2af7655e96461471e/src/libs/actions/IOU.ts#L1952
https://github.com/Expensify/App/blob/d8361eee3486ccb5472060f2af7655e96461471e/src/libs/actions/IOU.ts#L1936
https://github.com/Expensify/App/blob/d8361eee3486ccb5472060f2af7655e96461471e/src/libs/actions/IOU.ts#L1913
https://github.com/Expensify/App/blob/d8361eee3486ccb5472060f2af7655e96461471e/src/libs/actions/IOU.ts#L3551
What alternative solutions did you explore? (Optional)
NA

Hello @gijoe0295
I'm on Windows OS above issue is only happening in the staging environment.
Below is the video comparing both staging and the new website.
https://github.com/Expensify/App/assets/62931444/d4ad3906-9688-4180-8212-bae6a6b0c890
I found this third-party cookies warning in the staging website this might be causing the issue(scrolling).

Thanks,
Akhil Achary
(Upwork)

Contributor details
Your Expensify account email<EMAIL_ADDRESS>Upwork Profile Link: https://www.upwork.com/freelancers/akhilachary
Try pointing staging.new.expensify.com(staging website) to new.expensify.com URL temporarily The party cookies warning will disappear and test it (scrollbar problem).
If you use Cloudflair this can be done easily.

@fedirjh The reason is here
https://github.com/Expensify/App/blob/9d9757ffb3dfdad5dbd3dc13cd962faf5e1249d4/src/pages/home/ReportScreen.tsx#L436-L440
The transaction thread report is a chat thread and If we don't add any comment in transaction thread report, the notification of this report is hidden. When we edit the money request, the report is unfocused and then it's focused again after we submit the edit. That makes this useEffect above is triggered when we edit the money request and then the chat scrolls to bottom.
I think it's another bug and we should add isSingleTransactionView check in this useEffect to prevent openReport is called.
To replicate this bug, we can send a message in this report and edit the money request again.
https://github.com/Expensify/App/assets/161821005/afc1860f-47ce-428a-85d6-412a25c61d2d

@nkdengineer
Is this issue happening only on the staging site?

@0018akhil This appears to be happening in production too
https://github.com/Expensify/App/assets/51066321/d7f2006a-a246-4ed9-9f69-4329387ae511

Hi @Christinadobrzyn . Okay
So your in Mac. Can you send me the screenshot of your console.

I see this in the console when changing the task not sure if those are helpful

I was not able to replicate the bug with the money request edit flow. Hence, I don't think we should extend the fix to cover that flow.

To replicate this bug, we can send a message in this report and edit the money request again.

@fedirjh As I mentioned above, we can replicate this bug on latest main or staging by sending a message in transaction thread report before we edit the money request. I think we should fix all of them in a issue to verify that this bug doesn't happen in the further.

Not overdue, working on the PR.

Thanks @gijoe0295 - not sure why the payment auto-message didn't trigger for this.
Payouts due:

[X] Contributor: $500 @gijoe0295 (paid in upwork - https://www.upwork.com/nx/wm/workroom/36856139/overview)
[X] Contributor+: $500 @fedirjh (paid in upwork - https://www.upwork.com/nx/wm/workroom/37008381/overview)

Upwork job is here.
@fedirjh or @gijoe0295 do we need a regression test for this?

Oh thank you for clarifying! Okay, gotcha!
Please ignore my refund requests. Closing since we don't need a regression. Let me know if I'm missing anything.


why do we have this require?

https://github.com/gnosis/dutch-exchange/blob/eb05f60416a19b8c8d954fd4b50776ecfdca500e/contracts/DutchExchange.sol#L781
it prevent me from adding a tokenpair

it's there so we don't have to check that that token pair exists. i'll fix it now


Union between Valid type should be Valid

First check

[x] I added a very descriptive title to this issue.
[x] I used the GitHub search to find a similar issue and didn't find it.
[x] I searched the FastAPI documentation, with the integrated search.
[x] I already searched in Google "How to X in FastAPI" and didn't find any information.
[x] I already read and followed all the tutorial in the docs and didn't find an answer.
[x] I already checked if it is not related to FastAPI but to Pydantic.
[x] I already checked if it is not related to FastAPI but to Swagger UI.
[x] I already checked if it is not related to FastAPI but to ReDoc.
[x] After submitting this, I commit to one of:

Read open issues with questions until I find 2 issues where I can help someone and add a comment to help there.
I already hit the "watch" button in this repository to receive notifications and I commit to help at least 2 people that ask questions in the future.
Implement a Pull Request for a confirmed bug.

Example
from fastapi import FastAPI, Query
from enum import Enum
from typing import List, Union

app = FastAPI()

class Tag(str, Enum):
    a = 'a'
    b = 'b'
    c = 'c'

@app.get("/")
async def read_file(tag_or_list: Union[Tag, List[str]] = Query(...)):
    return tag_or_list

Description
I got the following error for running uvicorn test:app:
  File "/usr/local/lib/python3.8/site-packages/fastapi/dependencies/utils.py", line 333, in get_dependant
    assert isinstance(
AssertionError: Param: tag_or_list can only be a request body, using Body(...)

a List[str] is valid and Enum instace for Query is valid why Union between theme is not valid?
Environment

OS: Linux
FastAPI Version : 0.61.2
Python version: 3.8.6

Additional context

In the example, tag_or_list is just a one query parameter, but since you are giving more than one Models to it with Union, it understands the situation and assumes it is a Body that's why it expects a Body
You can send an array as a query parameter like this /products?color=blue,green,red  but you have one query parameter in this case. So you can not serialize two different things in one query parameter in any way.
So there are two options for you.
You can create a new model.
from fastapi import FastAPI, Depends
from enum import Enum
from pydantic import BaseModel
from typing import List

app = FastAPI()

class Tag(str, Enum):
    a = "a"
    b = "b"
    c = "c"

class NewModel(BaseModel):
    tag: Tag
    my_list: List[str]

@app.post("/")
async def read_file(tag_or_list: NewModel = Depends()):
    return tag_or_list

Or you can create two different query parameters.
from fastapi import FastAPI, Query
from enum import Enum
from pydantic import BaseModel
from typing import List

app = FastAPI()

class Tag(str, Enum):
    a = "a"
    b = "b"
    c = "c"

@app.post("/")
async def read_file(tag: Tag, my_list: List[str] = Query(...)):
    return tag_or_list

There is no way to merge two different things in one Query parameter, if you need one result, you need to use a Request Body.

Is your issue is resolved @ihakh ?

I had the same problem and I'm wondering what is a problem to check if you got only tag or only list but not two or multiple together.
I'm not expecting validation or some special behavior, but just to get generate auto docs which will say "Please don't use these parameters together, anyway will work only one of them"


error bigquery defog gen command : too many values to unpack (expected 2)

Defog install failed with the following error when trying to generate the schema for bigquery.
The schema consisted of only 2 tables, 12 and 2 columns respectively.
`(base) c:\Users\aburnazyan\AppData\Roaming\Python\Python39\Scripts>defog gen  miabanutyun-84e26.fb_dataset.fb_data  miabanutyun-84e26.fb_dataset.members
Connection details found. Reading connection details from file...
Connection details read from C:\Users\aburnazyan.defog\connection.json.
Getting the schema for each table in your database...
Sending the schema to Defog servers and generating a Google Sheet. This might take up to 2 minutes...
{'status': 'error', 'message': 'too many values to unpack (expected 2)'}
Traceback (most recent call last):
File "C:\Users\aburnazyan\AppData\Roaming\Python\Python39\site-packages\defog_init_.py", line 618, in generate_bigquery_schema
gsheet_url = resp["sheet_url"]
KeyError: 'sheet_url'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "C:\ProgramData\Anaconda3\lib\runpy.py", line 197, in run_module_as_main
return run_code(code, main_globals, None,
File "C:\ProgramData\Anaconda3\lib\runpy.py", line 87, in run_code
exec(code, run_globals)
File "c:\Users\aburnazyan\AppData\Roaming\Python\Python39\Scripts\defog.exe_main.py", line 7, in
File "C:\Users\aburnazyan\AppData\Roaming\Python\Python39\site-packages\defog\cli.py", line 41, in main
gen()
File "C:\Users\aburnazyan\AppData\Roaming\Python\Python39\site-packages\defog\cli.py", line 258, in gen
gsheets_url = df.generate_db_schema(table_name_list)
File "C:\Users\aburnazyan\AppData\Roaming\Python\Python39\site-packages\defog_init.py", line 634, in generate_db_schema
return self.generate_bigquery_schema(tables)
File "C:\Users\aburnazyan\AppData\Roaming\Python\Python39\site-packages\defog_init.py", line 622, in generate_bigquery_schema
raise resp["message"]
TypeError: exceptions must derive from BaseException`

Hi @aburnazy, thanks for raising this issue. We've identifed and fixed the issue on our backend (not an issue with this client), and also updated the client to print out a more informative message (see #7). Could you try again and let us know if you still encounter the same issue!

Just re-run the defog init - now the error points out that:
{'status': 'error', 'message': 'Please verify your email before using this feature. You can get a new verification email by logging in to your account, and clicking "resend".'}

Fair enough I haven't.
However logging in to the system again as the error message suggests I can not find where the Resend button is.

@wongjingping @NirantK @rishsriv any help appreciated.

Hi @aburnazy, could you drop us (support (at) defog.ai) a private email with your api key - we can help to retrigger the email verification flow. We will be closing this issue as this doesn't relate to the repository in question.


Register ChequeFormViewModel class and add business logic to the ViewModel

Create a class for ChequeFormViewModel
Register ChequeFormViewModel to the ViewModelLocationProvider at ChequeModule
Add business logics to the ChequeFormViewModel
This is only use for Form purpose.

Start after this work item is closed. #27

On hold for #72


Uncaught Error: Module did not self-register.

[Enter steps to reproduce:]

...
...

Atom: 1.46.0 x64
Electron: 4.2.7
OS: Unknown macOS version
Thrown From: Atom Core
Stack Trace
Uncaught Error: Module did not self-register.
At ELECTRON_ASAR.js:160

Error: Module did not self-register.
    at process.module.(anonymous function) [as dlopen] (ELECTRON_ASAR.js:160:31)
    at Object.Module._extensions..node (internal/modules/cjs/loader.js:722:18)
    at Object.module.(anonymous function) [as .node] (ELECTRON_ASAR.js:169:18)
    at Module.load (internal/modules/cjs/loader.js:602:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:541:12)
    at Function.Module._load (internal/modules/cjs/loader.js:533:3)
    at Module.require (/app.asar/static/index.js:72:46)
    at require (internal/modules/cjs/helpers.js:20:18)
    at customRequire (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:1:695134)
    at get_KeyboardLayoutManager (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:2778162)
    at get_manager (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:2778264)
    at Object.getCurrentKeyboardLayout (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:2778390)
    at e.keystrokeForKeyboardEvent (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:1081110)
    at KeymapManager.keystrokeForKeyboardEvent (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:1253631)
    at KeymapManager.handleKeyboardEvent (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:1251616)
    at WindowEventHandler.handleDocumentKeyEvent (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:284744)

Commands
     -0:05.2.0 core:save (input.hidden-input)

Non-Core Packages
busy-signal 2.0.1
intentions 1.1.5
linter 2.3.1
linter-flake8 2.4.0
linter-json-lint 0.1.3
linter-ui-default 1.8.0
minimap 4.29.9
platformio-ide-terminal 2.10.0
python-autopep8 0.1.3
script 3.25.0

It worked after I updated the Linter and rebuilt platformio. =)


10.1 64 Bit on osmtest - Properties for OSMFeatureProperties.xml file can never be set

Steps to replicate:
Launch ArcMap with OSM Editor for 64 Bit Geoprocessing installed.
Click on Editor toolbar, and select Properties.
Go to OpenStreetMap tab.
The setting for OSMFeatureProperties.xml file (the last on the page) is not populated. When attempt to populate, click Apply, click OK, and then reopen, the file mapping is not persisted. Closing and reopening ArcMap doesn't make a difference.

Fixed with https://github.com/Esri/arcgis-osm-editor/commit/09356f40f00ec7896900daf075e3a8e59ada21cb#diff-25d902c24283ab8cfbac54dfa101ad31


Increase TOE threshold from 300 to 500

300 was triggering inside system tables
Fixes #265
@knifewine to review

This seems to be system dependent... on my system, it fails with a value below 250. LGTM.


Add hyperspy-gui-ipywidgets and test v1.3.1

Add hyperspy-gui-ipywidgets and test v1.3.1

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


user group forum for Wiz-master

Hello Group, Adam Kennedy here from the Andrews Forest LTER in Oregon, USA. I recently read your 2020 paper on Wiz and wanted to test an install of the web-based app on the Andrews Forest webpage (https://andrewsforest.oregonstate.edu/data/streaming). I have the local instance running on the included Iris dataset on a Windows 10 machine (py 3.7.x and would like to build out a full featured instance using data from our streaming/realtime logger output.
Is there an active user group that focuses on continued development and troubleshooting of this app? I'd like to get involved if there is something like that.
Thanks in advance,
Adam

Hi Adam,
Thanks for your interest in the app. Currently, we upkeep the version of the app in the paper. We have daily users of that app, mostly from the academic community. A few users have forked this repository to build their own apps, but there is not much activity in this repo. Each full featured app will have unique requirements when deployed. The Dash community forum page has lots of contributions and will be much more useful than any user group from this repo anyway.
Realtime outputs in a Dash app are relatively straightforward to do using the dcc.Interval() component. You can read about using that for live updates here. We use that component in one of our examples to "stream" live data in a plot as it is recorded.
If you only want live data for a handful of plots or logger outputs, you can either embed Dash into your current website or it may be easier to use JavaScript alone to retrieve and display plots/logger output.

Hey thanks again for this. Sounds like a good plan. Can you send out a password to the online version you upkeep? I've only been able to access the local version and the examples. Have a great rest of your week.


How to modify code action ui width

The current width is full screen

There is a setup:
require'navigator'.setup({
  width = 0.75, -- max width ratio (number of cols for the floating window) / (window width)

AFAIK, the default is 0.62 and it should not go to 1.00. Unless the calculation is not correct. E.g. failed to get the current windows size.

my current configuration

seems to still have the same problem

Pushed a fix for this issue.

Thanks, the fix worked!


Test: notebook self-host improvement

Refs: https://github.com/microsoft/vscode/issues/109549

[ ] anyOS @btholt
[ ] anyOS @bamurtaugh

Complexity: 3
Authors: @rebornix @misolori
Create Issue

This month we improved the self hosting experience for the native notebook. Most issues were reported in https://github.com/microsoft/vscode/issues/109549 so please go through the checked-off ones and test if they work reasonably well now. You can test with GitHub Issue Notebook https://marketplace.visualstudio.com/items?itemName=ms-vscode.vscode-github-issue-notebooks or the Jupyter extension if you want to play with real world scenarios https://marketplace.visualstudio.com/items?itemName=ms-toolsai.jupyter .

It's always great being able to see the work being done on notebooks- I enjoyed using this and it looks really nice! If we're able to figure out a solution to the most recent issue I opened above (error when opening a notebook with changes from source control), I could take a closer look at the reported items related to diffing to verify they're working properly.


Issue with Default Chromium Version in Playwright

Issue with Default Chromium Version in Playwright
It seems chromium-1076 is set as the default. However, I consistently encounter an issue where the system fails to locate this version. When executing npx playwright install, it installs a newer version. My current workaround involves manually specifying the path in the playwrightOptions:
playwrightOptions: {
  ignoreHTTPSErrors: false,
  executablePath: '../../Caches/ms-playwright/chromium-1080/chrome-mac/Chromium.app/Contents/MacOS/Chromium',
},

Thanks for raising the issue, Could you tell us how you are using this library @elastic/synthetics and what was the version used to test this issue?

Hi!, thank you for the response! I am using `"@elastic/synthetics": "1.4.0"

When you install npm install @elastic/synthetics, Chromium gets auto installed at the PW specified location, Could you please help us with the steps you followed and how you are using the library to run the test suite. Thanks.

Closing as there has been any activity. Please do reopen if the issue persists.


Auto-deploy prevents successful initial service creation with linked environments

I'm creating webservices with linked environment groups. When i do the first terraform apply the service creation fails because the environment link is created only after the webservice is created (because it needs the service id) but the webservice creation waits for the first deployment to succeed (which obviously fails because it's missing the linked environment variables).
A subsequent terraform apply always tries to re-create the service (because it's marked as tainted after the first failure) so i'm basically stuck there.
I can mitigate this by manually terraform state rm the service and re-import it with terraform import (because it's created successfully on render). Then another terraform apply also creates the environment link which allows for successful deployments. But i guess that's not how it's supposed to work :)
I'd suggest to not wait for deployment results when creating a webservice.

Thanks for the report @extronics. v0.2.0 of the provider no longer waits for a successful deploy when creating services, which should prevent this issue.


Babel

Would it be safe to use babel to transform untrusted code before using the sandbox, or is would that open up opportunity for an attack?

I can't see why not, Babel doesn't execute code and its plugins interact with it in a clean AST form. The probability of abuse is negligible.


disabled cell variable definition remains active

Describe the bug
I made a cell that defined a variable a, then decided I wanted to try a different definition while keeping the first around. I disabled that cell and made a new cell that defines the variable a. The new cell has an error because of the first cell. I expected the first cell to have no effect on the evaluation of the notebook since it is disabled.
Environment
{
"marimo": "0.7.17",
"OS": "Windows",
"OS Version": "10",
"Processor": "Intel64 Family 6 Model 85 Stepping 4, GenuineIntel",
"Python Version": "3.10.11",
"Binaries": {
"Browser": "126.0.6478.127",
"Node": "--"
},
"Requirements": {
"click": "8.1.7",
"importlib-resources": "missing",
"jedi": "0.19.0",
"markdown": "3.6",
"pymdown-extensions": "10.8.1",
"pygments": "2.16.1",
"tomlkit": "0.12.5",
"uvicorn": "0.30.1",
"starlette": "0.37.2",
"websockets": "12.0",
"typing-extensions": "4.7.1",
"ruff": "0.5.5"
}
}
Code to reproduce
import marimo

__generated_with = "0.7.17"
app = marimo.App(width="medium")

@app.cell(disabled=True)
def __():
    a = 4
    return a,

@app.cell
def __():
    a = 5
    return a,

@app.cell
def __():
    return

if __name__ == "__main__":
    app.run()

This is actually intentional.
Disabling a cell doesn't remove it from marimo's internal graph, nor does it remove the cell's variables from memory. The main use case that at least I thought of was to allow you to iterate on "earlier" parts of a notebook without triggering execution of a later parts, as described in the docs. When you're ready, you can re-enable the cell and your notebook will be made up-to-date.

Perhaps the documentation could be clearer.
Perhaps the word "disable" is not quite right. Maybe "pause" would be better?
To confirm: you would like an option to remove a cell from marimo's graph, and remove its variables from memory, while still keeping the code in your notebook. Is that correct?

To confirm: you would like an option to remove a cell from marimo's graph, and remove its variables from memory, while still keeping the code in your notebook. Is that correct?

If this is indeed the case (and apologies if I have misunderstood), I have a couple of suggestions:

Temporarily wrap the code that you want to "disable" in a function:

def _():
  # disabled code here
  ...

Try using the scratchpad panel, which gives you a single-cell REPL that is allowed to redefine variables. Perhaps this works for your use case, perhaps it doesn't.

I don't feel strongly that this behavior needs to change, so I agree this is more of a naming/documentation issue. To me disable means it's like it's not there. I think it should be called autorun, which is already used in the settings pane to describe identical (or maybe only very similar) behaviors. Part of why I thought it would remove it from the graph is that it was a different name from autorun, so clearly it must have different intention.

Thanks for writing back. I will think about improving the naming.


chore: release 1.1.0

:robot: I have created a release *beep* *boop*
1.1.0 (2019-06-05)
Bug Fixes

default options to empty object (a5a4a83)

Features

support apiEndpoint override in client constructor (#57) (c9b0914)

Release is at https://github.com/googleapis/nodejs-irm/releases/tag/v1.1.0

The release build has started, the log can be viewed here. :sunflower:

:egg: You hatched a release! The release build finished successfully! :purple_heart:


Fix all FMI Cross-Check testcase errors

#19 #21

See also issue #39

Works with commit cffb3d4ba2613ef812953c8353fe4a794a9fc1b1


Added words in Chinese related to relative time

hours ago, days ago, weeks ago, months ago, years ago

Thank you!
Note, however, that you need to edit different files for your changes not to be lost due to later changes: https://dateparser.readthedocs.io/en/latest/contributing.html#guidelines-for-editing-translation-data


feat: Add transfer feature

Test scenarios:

User will see form error if transfer amount is greater than their balance
User will see form error if they submit transfer without an address
User can transfer wrapped token to another account:

Select account with wrapped token balance
Initiate transfer and wait for success message
Transfer to second account and check balance has updated

Switching tokens will clear the form fields.

Notes:

This PR includes Dan's changes to env.regtest and docker-compose which fixed the local dev environment.
Collateral tokens and transfer fees not tested locally. Governance tokens not supported yet as discussed.
Alternative networks have been removed - there is a ticket to reinstate these when necessary, but this will need some UX input.

@crypto-engineer I'd really appreciate your opinion on how I've added a variant to the balances component, and passed the variant name down. I've tried to keep it consistent, but I couldn't find a directly comparable example so I'm not confident that I've done it correctly.
Interlay

Kintsugi (border around select field is focus state)

Can you please add the governance token support before we merge this? Of the features we need to launch, this is the only token we actually need. Collateral and wrapped token will follow later on.

When the user hits "Transfer" it would be good not to reset the amount and recipient. They should just remain in the input field. Right now they vanish when the user hits Transfer

Fractional transfers below 1 don't seem to be working with an error. Fractional transfers above 1 are working.

👍

Invalid address input is not handled gracefully. We should at least quickly check if the entered address is 32 bytes long so that it can be encoded

Work to validate the address properly is nearly done, so I'll add it in this ticket - it's a very small change.

KINT is missing form the dropdown. I'm actually curious why? It's implemented exactly the same way as the other two currencies on the backend so there should be no difference in the front-end implementation for KSM, KINT and kBTC.
I took this out because it was broken - when I added the governance token as part of the dropdown, there was a bit of code I was unaware of. Most of the work for this is done now, but I'd prefer to keep it out of this PR.

The KINT and kBTC icons are swapped. I created an extra ticket for that: https://www.notion.so/interlay/KINT-and-kBTC-logos-are-swapped-in-the-UI-debd0b81a5f24479b4bb35d4b5100f9f
👍

@nud3l
Latest changes:

Amount and recipient is now cleared at the same time as the success notification (i.e. fields remain populated while transaction is processing).
Dan helped me get to the bottom of the fractional transaction failing, so that's working now.
I've used a polkadot.js util to validate the address. This happens on input as well as submission, which is consistent with the token amount field.

The missing governance token, and the swapped icons are being handled in separate tickets. These will be merged into master and then back into this branch before merging, so the feature will be merged in complete.

Can you please add the governance token support before we merge this? Of the features we need to launch, this is the only token we actually need. Collateral and wrapped token will follow later on.

This is being handled as a separate ticket, and because there are some specific questions/issues I have around this I'd prefer it to be a separate PR if that's ok. The majority of the work is done and it can be put up for a PR straight away.
That said very happy to add it to this PR if you want :) just flagging why I wanted to keep it separate.

That makes sense to me. I'll review the other PR!

@crypto-engineer you can review this now - all the updates Dom asked for have been made now and master has been merged in so this feature now supports the governance token.
Would you mind adding the comments by starting a review rather than adding single comments? Makes it a bit easier for me as that way I know when you've finished the review :)

@crypto-engineer on this comment: https://github.com/interlay/interbtc-ui/pull/114#discussion_r790748418
That doesn't work - it was there originally, but the form data was clearing immediately (something Dom flagged). I moved it into its own instance of useEffect so the actions happen simultaneously.

I'd really appreciate your opinion on how I've added a variant to the balances component, and passed the variant name down. I've tried to keep it consistent, but I couldn't find a directly comparable example so I'm not confident that I've done it correctly.

@tomjeatt
To be honest, I felt a little strange about this approach since it could make the component less reusable and granular but in consideration of timeline, let me create a separate ticket to follow up with it.


nbconvert unable to export notebooks as PDFs

Describe the bug
nbconvert is unable to export notebooks as PDFs
To Reproduce
Run jupyter nbconvert --to=pdf notebookname.ipynb and an error occurs. Example error message available here: https://github.com/ucbds-infra/otter-grader/issues/766#issuecomment-1913309226.
This prevents ottr from auto-generating PDFs.
Expected behavior
jupyter nbconvert --to=pdf notebookname.ipynb creates a PDF.
Additional context
@chrispyles asked me to open this bug at https://github.com/ucbds-infra/otter-grader/issues/766#issuecomment-1913325540.

Apologies, wrong repo.


[Feature Request] Allow specific users to bypass flag state

As the title says; Allowing specific users to bypass the state of one or more feature flags.
This'd allow developers to ensure proper functionality even on their live site by bypassing the off state, and seeing the flagged code regardless.

That's a great idea! I will try to get to it in my own time. However PRs are always appreciated


ImageWMS "ratio" option only works when "serverType" is set

OL 4.6.4
The "ratio" option of ImageWMS only works together with "serverType". Otherwise the ratio option is ignored.
The correct image size is also requested with serverType: 'blabla'. It doesn't work when serverType is not defined.
I recommend it should also work, even when serverType is unset. Or the documentation should be updated. Thanks.
See longer description here: https://gis.stackexchange.com/a/269950/1230

It's already mentioned in the documentation:
http://openlayers.org/en/latest/apidoc/ol.source.ImageWMS.html
For serverType:
The type of the remote WMS server: mapserver, geoserver or qgis. Only needed if hidpi is true. Default is undefined.
And because hidpi is true by default, serverType is mandatory.

It's already mentioned in the documentation
From your viewpoint as OL developer that documentation probably makes sense.

But from a pure API users point of view, that is not very clear. Where is the connection between the ratio-option and serverType? The default value of serverType is incompatible with the default value of hidpi? Why is blabla a valid input for serverType ?
I think OL can improve the documentation here. Would it be helpful that I make a suggestion? Or does the majority not share my confusion? (Could be just me being bad at reading OL documentation)
So, feel free to close the ticket or I will try to help. ;-)

I think there is some confusion here. The ratio option works fine, even without serverType set. The serverType is only required when hidpi is set to true, because requesting images with a non-standard dpi is different for the supported server types.


Concurrent docker pull, end of stream

Problem: The stream end on the second pull (tty2) even though it should return the same error as tty1
INFO:  The daemon has been setup as insecure en purpose in order to trigger an error
On a machine, i have two tty, 1 and 2. Try to launch these two commands almost at the same time.
On tty 1 I launch: docker pull registry2:5000/thumbor:4.7.0-37
FATA[0002] Error: Invalid registry endpoint https://registry2:5000/v1/: Get https://registry2:5000/v1/_ping: EOF. If this private registry supports only HTTP or HTTPS with an unknown CA certificate, please add `--insecure-registry registry2:5000` to the daemon's arguments. In the case of HTTPS, if you have access to the registry's CA certificate, no need for the flag; simply place the CA certificate at /etc/docker/certs.d/registry2:5000/ca.crt

on tty 2, I launch: docker pull registry2:5000/thumbor:4.7.0-37
Repository registry2:5000/thumbor already being pulled by another client. Waiting.

However, it ends straight away. I was expecting to have the same error as tty1.

@dmcgowan @dmp42 can you resolve

@florentvaldelievre are you able to pull that image with just one docker pull?
Is this about having the same error when things go wrong like they do here? Or about allowing simultaneous pull of the same image?

Actually, when docker daemon replies "Repository XXX already being pulled by another client. Waiting.", i was expecting to have some kind of queue mecanism for this request until a slot get free.
Example:
1)If we try to pull and there is already an existing pull of the same image:
=> "Repository XXX already being pulled by another client. Waiting." : Working as expected

When the first pull is finished and a slot is free, the pull request should get picked and attempt to download the image.

At the moment, it is waiting, and when a slot is free, it just quit the stream, without doing anything.
You can easilly reproduce the problem:
tty1: docker pull busybox => Downloading
tty2: docker pull busybox => "Repository busybox already being pulled by another client. Waiting."
When tty1 finishes downloading the image, tty2 is just ending the stream

closing as dup https://github.com/docker/docker/issues/8385


Suggestion to use the "chalk" package

If you are interested, I suggest using the "chalk" package for log message/error that will make the tool more attractive.

I actually really like your use of chalk in your project and I'm also planning to use it in the next release. Thanks for the suggestion!


Cant host on render

thread 'main' panicked at '$REDIS_URL is not set: NotPresent', src/main.rs:34:43
I have it as REDIS_URL: 'redis://redis:6379'. I am new to it, someone can help?

Are you adding that as an env variable to the motorhead container?


Failed  to load other language data.

This is Chinese NL2SQL dataset. It has same format with wikisql. https://github.com/ZhuiyiTechnology/TableQA
Except a little of difference.
"sql": [2]. It uses list to wrap the value.
When I load this dataset, I get error.

I change the tokenizer to bert_base_chinese.  still no working.
So, what can I do to finetuen your model in Chinese NL2SQL dataset?
Thank You very much!!!

Interesting. To do so you can replace the BERT model we used with a multilingual LM such as mBERT.

have you solved this problem?


Use namespaced Dockerfile.ddev-cron

DDEV v1.19.3 has namespaced Dockerfiles, so we don't need to run the risk of overwriting a Dockerfile that somebody already has. This PR changes to that approach, and guards against it being used in previous DDEV versions. There are also minor docs updates.
This can be tested with ddev get https://github.com/rfay/ddev-cron/tarball/20220605_use_namespaced_dockerfile or by checking out the PR and ddev get /path/to/checkout

Please take a look and see if it's good @tyler36 - If so, please create a new release after pulling.

Attempted to manually install this but am getting an error
$ ddev start
------
 > [drud/ddev-webserver:v1.19.3-1-test-sync-built 6/6] ADD cron.conf /etc/supervisor/conf.d:
------
', stderr='failed to solve: failed to compute cache key: "/cron.conf" not found: not found'

When you say "Manually install" what does that mean? Does .ddev/web-build/cron.conf exist after your ddev get ?

.ddev/web-build/cron.conf was in there before this PR, https://github.com/tyler36/ddev-cron/blob/main/web-build/cron.conf

I didn't use ddev get, I "manually" added the files by following the install.yaml

I see now. I was bouncing between checking this and browser-sync and missed that file. My bad.


Update to Swift 2.0

@beltex Hi beltex, I totally agree and I'm seeking volunteers to help be bring Pythonic.swift from 1.2 to current Swift :-)

I'll take a stab at it!

Should I just remove the custom Set class?

The project is now compiling under Swift 2.1. Clean-ups and improvements are welcome! :-)


pull request for e2e test at 02/03/2017 22:26:05

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


bump up jsfmt version to 0.5.1

includes better support for ES2015 features

Thanks! I'll test this later and make sure everything looks good, and then I'll merge.


What is the role of Tensor-related classes?

There are many classes related to Tensor in this framework.
(Tensor/TensorData/TensorInfo/TensorObject/TensorShape)
I would like to let you know the role of each class. If you like, you can write in Korean.

simple UML diagram would likely do.
from a type perspective, the fact that tensors can be chained is going to be the tricky part. So tensor compositions such as Kronecker products, Trace-Singh, etc. should be expressible.
Secondly, automatic differentiation is required as well to support the back propagation phase.
That are two requirements that are none trivial, especially when there is multi-precision tensors at play.

@Ravenwater Oh, I asked the question to @jwkim98.
I've heard answer through Facebook messenger, so I'll close the issue. Thanks. 🎉

:-) no problem:
Here is a good link on the AD part: https://arxiv.org/abs/1711.01348

@Ravenwater Can you reply this? https://gist.github.com/utilForever/1e0a93263ff5d52ae35304906e9c980a


bugfix log interpolation

I changed the microclimate log interpolation function to get rid of the warning in vr_run.
Type of change

[ ] New feature (non-breaking change which adds functionality)
[ ] Optimization (back-end change that speeds up the code)
[x] Bug fix (non-breaking change which fixes an issue)

Key checklist

[x] Make sure you've run the pre-commit checks: $ pre-commit run -a
[x] All tests pass: $ poetry run pytest

Further checks

[x] Code is commented, particularly in hard-to-understand areas
[x] Tests added that prove fix is effective or that feature works

Codecov Report

Merging #244 (19a6479) into develop (1fe8e44) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff            @@
##           develop     #244   +/-   ##
========================================
  Coverage    95.37%   95.37%
========================================
  Files           42       42
  Lines         1643     1644    +1
========================================
+ Hits          1567     1568    +1
  Misses          76       76

Impacted Files
Coverage Δ

...l_rainforest/models/abiotic_simple/microclimate.py
100.00% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Cyrillic folder names aren't parsed in Cyrillic

I have a mail.ru address, which is a Russian-speaking service, and its folders have Russian names.
This is how they look normally (aerc):

This is how himalaya outputs them:

(Sent and Trash were autocreated by other clients, default folders have Russian names)
For comparison, Cyrillic in letters is shown normally:

For more context, if needed: OS is Artix Linux, Himalaya is packaged by Arch, latest version at the moment

Interesting issue, it looks like mailbox names are not parsed correctly. It is strange because I remember having tested with one of my french account and special chars like é were displaying properly. Thanks for pointing it out!

I described this problem here #370

@pleshevskiy is right, this issue is the same as described in his issue. I close this one for duplicate then. @acid-bong I invite you to follow #370 instead. Thank you!


Wrong / missing implementation of HmIP-WRC6 in WebUI

Not possible to use / control the device HmIP-WRC6 in WebUI:

Funktion should be like this on older device HM-PB-6-WM55:

Firmware is <IP_ADDRESS>71216

sorry wrong button.. case is not closed

I think my above answer was right. Your screenshot (a month ago) shows a actuator not a transmitter. To my knowledge there are no push-buttons in the WebUI for any HmIP transmitter.

Are you really running <IP_ADDRESS>71216? Have you tried the latest one?

@HMside you may be right... i checked again ... here is the correct picture.

I am really dissapointed that this device is not manageble over the Web interface. There is no information in the manual that it is not supported / implemented in the web interface. Where did you got this information?
@ Jens -  i think it is not a bug in RaspberryMatic, because it is also not working int the last CCU2 Firmware, but maybe you can foreward this request to your Eq3 contacts to implementate this - if technically possible.


Check for ml connectives in ml_pred_to_bool

Addressing post-merge comments in #610

Excellent, thank you for the high quality change and tests.


trouble munching newly bought content

Describe the bug
Hey! Thanks for the tool :)
I bought the player's handbook on the 29.04.2024, which I could import into my foundry instance (with patreon I also could import all races & classes, etc.).
In session 0 today a player wanted to play a race available in mordenkainen presents monsters of the multiverse -- so we bought it instead of building it by ourselves.
While buying I found that D&D Beyond looked different, but I just bought it and tried to munch races -- nothing new was created.
I tried multiple times and different settings, but I just didn't get it to work.
It seems like D&D beyond changed something in their payment process / order history after april '24, could it be that they changed their API too?
I checked multiple times that I really own the content on D&D Beyond, and I do.
EDIT: I have just learned that I also can't select those races inside D&D Beyond Character Creation itself, maybe they have borked something and it is not a problem on the ddb-importer. Still, I can't use the bought content, I can just read the digital book.
To Reproduce
Steps to reproduce the behavior:

Buy something new that contains races -- like mordenkainen presents monsters of the multiverse
Do a race munch

Expected behavior
After a race munch I'd like to see the races I own in D&D Beyond. (And other munches too, I didn't get any content of the freshly bought things to work).
Environment:

Browser: Firefox 123.0.1
Foundry Version: 11.315
5e System Version 3.1.2
Module Version: 5.1.22
Does this behaviour occur with other modules disabled: Not tested yet

Hopefully something else has this problem too, or it just works magically tomorrow.

If you can't select the races in the character creator the Importer won't be able to see them. You likely need to "sync your entitlements" in DND beyond.
https://dndbeyond-support.wizards.com/hc/en-us/articles/7747224765588-Sync-Entitlements#:~:text=If they're out of,Entitlements are back in sync.
I'm going to close this as it's not a DDB Importer issue, as it's working correctly, able to see the content you have access to on DDB.

Hi! Thanks for the link and the fast response!
After going to bed it just worked now -- probably, because ddb syncs regularly (?).
I was a little perplexed because I am pretty sure the last time I bought something it was available instantly, but that might just be bad timing.
Have a nice day :)


Add LAB colour support

Will be added after the bug fixes.

Can this be added in the next milestone?

Maybe :) The focus is definitely more on converting string that browsers already recognize natively, which is nearly complete. LAB colour (and CMYK, for that matter) aren't recognized by browsers as far as I know, so they're a lower priority.
That said, I do have the math for converting both, and a means to verify correctness, so I think a 1.2 release should include them.


Quest 2 custom input action not working

Describe the bug
Oculus Quest 2 custom input action assignments not working.
To reproduce
Steps to reproduce the behavior:

Load the MRTK examples InputActionsExample scene.
Assign any digital button to Rotate on either Quest controller mapping
Build and run on Quest 2
Pressing any button mapped to any custom input is not registered by the InputActionHandler

Expected behavior
When I press the A or B buttons, the InputActionHandler should be triggered and the cube should rotate, but the action is not registered by the InputActionHandler and the cube does not rotate.
Screenshots
Input Action Assignments:

Input actions:

Input data provider:

InputActionHandler:

Your setup (please complete the following information)

Unity 2019.4.11f1
MRTK v2.5.1 through UPM

Target platform (please complete the following information)

Oculus Quest 2

Additional context
I've tried single axis based input too and no custom input actions have worked for me.

Hi @trentpiercy , I am facing the same issue while building for Quest 2 using MRTK package in Unity. How did you solve this issue?


New Statement format

It seems the barclays have changed their statement format from Jan 2015. Your tool does not work with this format. Any hope for supporting this:

The pull request https://github.com/penrosestudio/barclays-bank-pdf-to-csv/pull/13 solved it for me

Please check the notes on my pull request carefully, and manually check every entry - there may be other changes that I have not yet caught, and am unlikely to as I have data feeds directly to my accounting package now.

I've merged in #13, thanks!


Extra Semicolon in .NET SDK after assigning ttl value

There should not be any ';' as this gives error while building the solution.
 DefaultTimeToLive = 90 * 60 * 60 * 24;

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

ID: ae1b1c71-a8ca-2bba-202e-f74a9414ced4
Version Independent ID: 65ce6b13-2e3d-39ea-167a-ebdc1864f66b
Content: Configure and manage Time to Live in Azure Cosmos DB
Content Source: articles/cosmos-db/how-to-time-to-live.md
Service: cosmos-db
GitHub Login: @anfeldma-ms
Microsoft Alias: anfeldma

@Arvind-0120 Thank you for the feedback. We are actively investigating and will get back to you soon.

@Arvind-0120 Thanks for letting us know, we have updated the document content accordingly.
The changes should be live once the PR is approved.
We will proceed to close this issue now.
If there are further questions regarding this matter, please comment and we will gladly continue the discussion.


RequestOptions is optional when using agents but System.NullReferenceException is thrown if RequestOptions isn't specified

Hi,
Just upgraded to latest version (<IP_ADDRESS>9) of the WebApi package.
We get errors on every use of agents when RequestOptions isn't specified.
Ex throwing System.NullReferenceException:
var contact = await contactAgent.GetContactEntityAsync(1);
Stacktrace:
 at SuperOffice.WebApi.Agents.AgentBase.Merge(RequestOptions options)
Workaround is to always specify the optional RequestOptions parameter:
var contact = await contactAgent.GetContactEntityAsync(1, new RequestOptions());
I would consider this a bug since the RequestOptions parameter is optional, but is required by internal logic.
Best regards
Gunnar Stensby, i-Centrum AB

Hi Gunnar,
Not sure how this slipped through, the samples and tests were working prior to pushing so I need to look into this ASAP...
Will try to get a new working one out soon.

Great news, thank you Tony!

OK, the latest packages (<IP_ADDRESS>) pushed this morning should address the RequestOptions issue.


HTTPRoute Fails to Get Reconciled

An HTTPRoute fails to get reconciled when other HTTPRoutes exist that reference an unmanaged GatewayClass. Reconciliation for the HTTPRoute is bypassed with httproute doesn't reference any managed gateways. This is b/c the controller lists all HTTPRoutes (including ones that reference an unmanaged GatewayClass). If the HTTPRoute that references an unmanaged GatewayClass is the first in the list, then reconciliation is bypassed instead of continuing to iterate through the HTTTPRoute list.

Fixed by #702


Shared implementation for Azure Service Bus subscriptions for binding and pubsub

Description
This PR changes the Azure Service Bus components (binding and pubsub) to rely on a shared implementation for the subscription part to the queue or topic.
This makes it possible to bring into the binding a lot of the additional capabilities that were present in the pubsub component, for free, including support for multiple handlers in parallel (although it's limited to 1 by default for backwards-compatibility) and support for renewing locks (fixes #1789)
Issue reference
Fixes #1789 by bringing in the missing features
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[X] Code compiles correctly
[X] Created/updated tests
[X] Extended the documentation https://github.com/dapr/docs/pull/2529

I hate to be the person, but can you also swap us to azservicebus version 1.0.1 which is out now? And that requires a swap to azidentity 1.0 and azcore 1.0?

I made that change


Angular rc2 breaks dependencies

When trying to follow Angular 2.0 Meteor Tutorial I get an error when executing:
meteor npm install --save angular2-meteor @angular/platform-browser-dynamic
...
npm ERR! peerinvalid The package<EMAIL_ADDRESS>does not satisfy its siblings' peerDependencies requirements!
npm ERR! peerinvalid Peer<EMAIL_ADDRESS>wants<EMAIL_ADDRESS>npm ERR! peerinvalid Peer<EMAIL_ADDRESS>wants<EMAIL_ADDRESS>npm ERR! peerinvalid Peer<EMAIL_ADDRESS>wants<EMAIL_ADDRESS>npm ERR! peerinvalid Peer<EMAIL_ADDRESS>wants<EMAIL_ADDRESS>npm ERR! peerinvalid Peer<EMAIL_ADDRESS>wants<EMAIL_ADDRESS>

I'm getting the same issue trying to install any new packages like the router.
npm install --save gives me missing dependency warnings for both platform-browser-dynamic and router, and meteor npm install errors out completely.

Ups I realised I added this issue on wrong repository. It is relevant to Urigo/angular2-meteor.
Will post issue there.

https://github.com/Urigo/angular2-meteor/issues/300


Import Functions from dll

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (f6a251f) 94.12% compared to head (dc1245a) 80.64%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##           master     #224       +/-   ##
===========================================
- Coverage   94.12%   80.64%   -13.49%
===========================================
  Files          85       85
  Lines        5178     5178
  Branches      488      488
===========================================
- Hits         4874     4176      -698
- Misses        177      901      +724
+ Partials      127      101       -26

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Apple Juice with Amaretto definately needs to be added

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

LGTM!! 😉


Apple Review Rejected

I am using the latest version(1.2.8) of weaklock_plus
but my app be rejected by apple due to this issue
anyone know how to fix it?
Thanks
"Frameworks/wakelock_plus.framework/wakelock_plus", which includes wakelock, an SDK that was identified in the documentation as a privacy-impacting third-party SDK. If a new app includes a privacy- impacting SDK, or an app update adds a new privacy-impacting SDK, the SDK must include a signature file. Please contact the provider of the SDK that includes this file to get an updated SDK version with a signature. For details about verifying the code signature for a third-party SDK

@codeLockers looks like one of your dependencies is still using wakelock. Nine months ago, I added the privacy manifest to this library, so it should be good as far as Apple is concerned.

If you're able to contact your reviewer, then please point them to the privacy manifest file found in this library.

Please ensure that you're running on the latest Flutter version and that you run flutter pub upgrade when using the latest version.
Dunno why Apple thinks that the privacy manifest is not there when it is.

Also, please ensure that your Cocoapoda are upgraded as well.
Worst case, you may need to sign your frameworks manually, since the Flutter command for signing iOS frameworks may not automatically sign this one.


chore: make format the whole repo

runs make format and make proto-format to format the whole repo.
This will make it easier to format next contributions without having to touch unrelated files.

yes, its weird it doesn't. want me to investigate?


Bootstrap 4 support?

I tried to use it with Rails 5.1 and Bootstrap 4 but the default alert messages is showing instead of the modal. There are no JS errors in the console. Do you have any plan to support Bootstrap 4 and Rails 5?

You can try out my pull request #55. Please share your opinions on it. So far no update from official maintainer.

Fixed in #55. Thanks everyone!

I'm not seeing support for BS4. The modalClass argument ends up colouring the whole background and the title is on the right. Is BS4 support in the roadmap?

It would be nice to have a template override for the HTML of the modal so we could spin our own support and make any mods we want.


Add fallbacks for default configs

Switched to commonjs default for commonjs users.
Added fallback types for default tsconfig users.
Maintains support for es modules.

Found breaking issue in emitted esm declaration files not keeping extensions.


command /aiwaz mass up

the script d.setMinutes(d.getMinutes + HK_Offset) still effect other time object

Installed<EMAIL_ADDRESS>Problem Fixed
See README.md For More Information


Importing interface error The requested module '/_nuxt/stores/.ts' does not provide an export named ''

Environment

Operating System: Windows_NT
Node Version:     v18.17.1
Nuxt Version:     3.8.0
CLI Version:      3.9.1
Nitro Version:    2.7.0
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      ssr, runtimeConfig, nitro, css, typescript, build, modules, components, imports, veeValidate, pinia, vite, app, vue, pwa
Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>()
Build Modules:    -

Reproduction
Stackblitz reproducible link: https://stackblitz.com/edit/nuxt-starter-wowrr5?file=package.json,app.vue,pages%2Findex.vue,nuxt.config.ts,stores%2FbackofficeStore.ts
Describe the bug
Simply import an interface from a store file inside a SFC.
Additional context
In a earlier version (3.7.4), import a interface or a type does not cause any issues.
Logs
500
The requested module '/_nuxt/stores/backofficeStore.ts' does not provide an export named 'NewsInterface'

You should now use import type:
import type { NewsInterface } from '~/stores/backofficeStore';
import { useBackofficeStore } from '~/stores/backofficeStore';

See the bit in the release notes under Type import changes.


[Feature Request] Alarms

(React to this bug with a thumbs up to vote for faster implementation. I'll generally work the most requested features first)
This feature would allow some sort of visual and/or audible alert when a value passed a threshold, such as when depth passes under 40ft.
Visual display could be done using inverse video in night mode and black-on-yellow for day mode. There are several UI and implementation questions. e.g.

Whether to allow a separate threshold for each display cell (e.g. one cell that triggers when depth is below 40ft and another that triggers when its below 20ft) or have a single global threshold
What to do if a threshold is triggered while the data is not currently display (e.g. its on a different page or maybe isn't mapped to a page at all - only possible with a global alert)
Whether to support an audible alarm. Obviously this could be very useful but it raises questions, while a visual display obviously only works when the app is foreground some users might expect an audible alarm to work even if the device screen is off - actually getting OS permissions for the services needed to do that would be hard, would force users to go through more OS approvals, and could eat a decent chunk of battery life, and still wouldn't be reliable due to some vendors turning off radios in low power states.
How to silence the alarm. Maybe for visual only this wouldn't be necessary - the highlight would persist while the condition persisted
Should heading/course/bearings be included. Could be useful for things like danger bearings but explaining the logic is non-trivial (e.g. does a heading of >090 mean 090-270 is over while 270-090 is under)

At the moment I'm leaning towards a global definition for alarms managed through a new top level menu. An optional tone and popup would appear when the condition is triggered, dismissing the popup would silence the audible alarm. Any cells displaying the data would be highlighted/inverse while the condition persists. There would be a warning that audible warnings won't work unless the screen is on and the app is running in foreground.
If you've got opinions on this add a comment!

Hello,
my personal opinion is, not to overload the app. Alarms should be given by the navigation-device, not by an android display. Navigational devices are much more reliable and much louder.
e.G.: I have a radio, that receives AIS and can give a loud CPA-alarm. => I can contact the other ship directly.
The nav-program (I use OpenCPN) can give every alarm you want.
I think, its much more reliable not to knit all the systems too tight together, so that every feature works on stand allone, when the network fails.
Kind regards
Stefan


Add initial documentation

Provide documentation on how to install, what features are included, etc.

Maybe we should consider moving the documentation to a GitHub Wiki?

The current docs are fine as far as I can see, only if we need to go more in-depth we could use the wiki indeed.
The only downside is we can't use https://guides.github.com/introduction/flow/index.html, but perhaps that's just a minor thing

I agree.
Seems like we're good for the initial release although we might want some coverage on #4 by @msjonathan? Or am I exagurating?


Fixes #180: Added support for the date string format.

This adds support for the "date" json schema string format, which is supported by the jsonschema lib and provide a much more convenient way to describe dates comparatively to datetimes.
This also updates the AltDateWidget behavior accordingly.
/cc @kaihendry
r=? @natim @glasserc @leplatrem

I'm landing this.


stacker_yaml doc: fix indenting

the section header depths made the outline weird, they should match the levels in the actual yaml, except for import hash which seemed enough like a sub-type that I left it as a child of import.
Signed-off-by: Michael McCracken<EMAIL_ADDRESS>

what i meant by the outline being weird


[BUG] Class name must be a valid object or a string

Laravel Version
php artisan --version

Laravel Framework 10.18.0

Package Version
composer show opcodesio/log-viewer

name     : opcodesio/log-viewer
descrip. : Fast and easy-to-use log viewer for your Laravel application
keywords : arukompas, better-log-viewer, laravel, log viewer, logs, opcodesio
versions : * v3.0.0
type     : library
license  : MIT License (MIT) (OSI approved) https://spdx.org/licenses/MIT.html#licenseText
homepage : https://github.com/opcodesio/log-viewer
source   : [git] https://github.com/opcodesio/log-viewer.git 40ed823a38a8b5b175778bc460162ab1d72a8244
dist     : [zip] https://api.github.com/repos/opcodesio/log-viewer/zipball/40ed823a38a8b5b175778bc460162ab1d72a8244 40ed823a38a8b5b175778bc460162ab1d72a8244

BUG
\Opcodes\LogViewer\Logs\LogType::name()
https://github.com/opcodesio/log-viewer/blob/40ed823a38a8b5b175778bc460162ab1d72a8244/src/Logs/LogType.php#L48

Second to this.

hey guys, thanks for reporting this! This happened when there were unsupported files found with the glob patterns :)
This is now fixed in v3.0.1. Any files found that are not supported by Log Viewer will be excluded by default. To change the behaviour, set 'hide_unknown_files' config option to true.
See more here - https://log-viewer.opcodes.io/docs/3.x/configuration/including-excluding-files#hidingshowing-unknown-log-files
Have a great week!


Docs: Typos in Documentation. Minor

The end of "Manual installation" body is cut off.
The end of "Contributing" is cut off.

Good catch. This has been fixed and will be included in the upcoming release!


Update README.md with new instructions:

Use official documentation to install ros2 dashing
Use rosdep to install dependencies
Explain how to compile on Docker

Signed-off-by: Lander Usategui <lander.usategui@gmail,com>

I added your comments. @mlanting


App Crash on IOS 9.3

Hello, I use your plugin for my ionic project. It's works perfectly on android device but crash at start on IOS device. I don"t know why. I insert my GoogleService-info.plist like google-services.json. Can u help me please . the only information missed on Firebase App is my App Store ID
because i work on dev app .
Best regards

Related to #93 ?

No sorry , but my App crash on SplashScreen . its not a problem to logEvent() function.

What does the XCode console say about the crash ?

I dont use xcode. My porject is on windows.  But. I will try to install my
porject on osx fot test soon
Le mar. 15 nov. 2016 17:40, SebT<EMAIL_ADDRESS>a écrit :

What does the XCode console say about the crash ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/arnesson/cordova-plugin-firebase/issues/94#issuecomment-260694281,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFV7ZOgx4rEutbGCiiSBHtzWiCSZzOc7ks5q-eB2gaJpZM4KxapE
.


Convert vocos checkpoint to bigvgan

Checks

[X] This template is only for question, not feature requests or bug reports.
[X] I have thoroughly reviewed the project documentation and read the related paper(s).
[X] I have searched for existing issues, including closed ones, no similar questions.
[X] I confirm that I am using English to submit this report in order to facilitate communication.

Question details
Is there a fast way to convert a F5 checkpoint that was trained using vocos mels to one that is compatible with bigvgan? For example, can we freeze most of the parameters, and only continue-train a small number of parameters? Or must we train all parameters again?

But we also release BigVGAN F5 checkpoint in the huggingface ( Why we need to convert the checkpoint?

Because I have finetuned many models on top of the vocos checkpoint. Now I wonder if I need to redo all that if I want bigvgan.

Got it, this is a great idea. I think you can first compare the differences between these two MelSpec implementation methods, then add a preprocessing network or postprocessing network, and freeze all the dit blocks. You can try this approach to see if the model can map the vocos features space to the bigvgan features space on its own. （Intuitively, this seems to be the simplest approach


chore: Add Jitsi capture screen permission dialog

Hello,
Would it be possible to implement it for BigBlueButton?
With the latest version, I cannot connect with the microphone either (due to permission problems).
Thanks,

Hello, Would it be possible to implement it for BigBlueButton? With the latest version, I cannot connect with the microphone either (due to permission problems). Thanks,

@Karting06 sorry but I didn't understood. You are using BBB instead of Jitsi? Could you please open an issue and explain more the problem? Then I will take a look on this ASAP.


[ADD ME TO THE ORGANIZATION]

Name: Aditya Mishra
Location: India
Interest: Android Development
Discord username(optional):-
Why do you want to join?

I want to join because I am excited to collaborate with peoples having similar interest and also expand my network to learn new skills

You can also join our Discord community here
Feel free to check out other cool repositories of the DevsInTech Community here.

@this-Aditya  Thanks for your interest. Invitations have been sent to you. Kindly accept it and make your membership public.


Cannot join games - error 268 (Kicked due to unexpected client behavior)

Restarting my PC did nothing

Go to #193 since that person is reporting the same issue.

Oh, didn't see that


finalizers should be added to resources to generated ClusterRole when appropriate

/cc @andreaTP

Unfortunately, the commit doesn't fix the issue for the Keycloak project, we are setting all of the controllers to not use Finalizers:
https://github.com/keycloak/keycloak/blob/f11573eeb26f5a3c47374d9f01a2f6616b7ca82b/operator/src/main/java/org/keycloak/operator/v2alpha1/KeycloakController.java#L51
https://github.com/keycloak/keycloak/blob/f11573eeb26f5a3c47374d9f01a2f6616b7ca82b/operator/src/main/java/org/keycloak/operator/v2alpha1/KeycloakRealmImportController.java#L51
But the exception is still being thrown in OpenShift.
cc. @vmuzikar


feature: Use non-String datatypes

Thanks for that fine piece of software you have provided us with! I like it very much.
If I have just missed something in the documentation in the first place, I'm happy and this issue can be closed right away.
Currently, the only thing with DotEnv that feels cumbersome, is using non-String datatypes. We can validate them, but we have to convert their value manually, e.g.
$dotenv->required('FOO')->isBoolean();
$_SERVER['FOO'] = $_ENV['FOO'] = filter_var($_ENV['FOO'], FILTER_VALIDATE_BOOLEAN);

In most cases I check datatypes for, I will use exactly those datatypes in my code, so I have to cast them. I would appreciate if DotEnv would take care of that for me automatically. This approach would lead to a discrepency between getent() and the $_ENV/$_SERVER vars. There might also be a configuration flag for switching that behaviour, so users can opt in manually.
What do you think about it?

I use a custom version of Laravel's env() function that does type conversion, unquotes strings and allows for fallback values in case of unset variables. I guess you could make the argument that phpdotenv should include that function or something similar since it'll probably come up anyway.

Thanks for the suggestion, but I don't want to implement this. I think it's very straightforward if environment variables are always going to be strings, especially if people are using strict typing. It's hard to do good static analysis if the env functions can return other stuff.


Update package.json to add moment npm dependency that fails on Mac OS

Update package.json to add moment npm dependency that fails on Mac OS.
This issue was reported on https://github.com/adobe/aem-project-archetype/issues/891
npm was complaining on a dependency: [INFO] Error: Cannot find module 'moment'
Description
Added the latest moment npm dependency under the /ui.tests/test-module/package.json
"moment": "^2.29.1"
Related Issue
https://github.com/adobe/aem-project-archetype/issues/891
Motivation and Context
build failed when executing the tests when running the command:  mvn verify -Pui-tests-local-execution
How Has This Been Tested?
Tested this locally on two Macs
Screenshots (if appropriate):

Types of changes

[X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[X] I have signed the Adobe Open Source CLA.
[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 read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

same problem on windows - the ui.tests are failing both locally and inside docker image without this dependency.
i've validated that this PR fixes the problem.


python310Packages.wasabi: 0.9.1 -> 0.10.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.wasabi is: "A lightweight console printing and formatting toolkit"
meta.homepage for python310Packages.wasabi is: "https://github.com/ines/wasabi"
meta.changelog for python310Packages.wasabi is: "https://github.com/ines/wasabi/releases/tag/v0.10.1"
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
Warning: a test defined in passthru.tests did not pass
found 0.10.1 with grep in /nix/store/qkvdihpxnbpp8qbxax3df14ah13f3yra-python3.10-wasabi-0.10.1
found 0.10.1 in filename of file in /nix/store/qkvdihpxnbpp8qbxax3df14ah13f3yra-python3.10-wasabi-0.10.1

Rebuild report (if merged into master) (click to expand)

12 total rebuild path(s)

12 package rebuild(s)

First fifty rebuilds by attrpath
python310Packages.spacy
python310Packages.spacy-loggers
python310Packages.spacy-transformers
python310Packages.textacy
python310Packages.thinc
python310Packages.wasabi
python39Packages.spacy
python39Packages.spacy-loggers
python39Packages.spacy-transformers
python39Packages.textacy
python39Packages.thinc
python39Packages.wasabi

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/qkvdihpxnbpp8qbxax3df14ah13f3yra-python3.10-wasabi-0.10.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python310Packages.wasabi https://github.com/r-ryantm/nixpkgs/archive/59547cb543a3d3def95aa7aeeaf95219db335983.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/qkvdihpxnbpp8qbxax3df14ah13f3yra-python3.10-wasabi-0.10.1
ls -la /nix/store/qkvdihpxnbpp8qbxax3df14ah13f3yra-python3.10-wasabi-0.10.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

8 packages failed to build:

python310Packages.spacy
python310Packages.spacy-transformers
python310Packages.textacy
python310Packages.thinc
python39Packages.spacy
python39Packages.spacy-transformers
python39Packages.textacy
python39Packages.thinc

4 packages built:

python310Packages.spacy-loggers
python310Packages.wasabi
python39Packages.spacy-loggers
python39Packages.wasabi

Maintainer pings

nixpkgs-review reveals no new failures, macos 10.15 & nixos x86_64.
But many of the reverse-dependencies are already broken, so hard to know if they'd be happy with this.


Update ibm-security-guardium-insights.template.yaml

Description of changes: Minor PR to trigger S3 push
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Taskcat test uploaded to #1


Update Path in the main Readme

The author name changed

Why did you close this? The current name is wrong and hence the command does not work:
> remotes::install_github("darwin-eu/PaRe")
Error: Failed to install 'unknown package' from GitHub:
  HTTP error 404.
  Not Found

  Did you spell the repo owner (`darwin-eu`) and repo name (`PaRe`) correctly?
  - If spelling is correct, check that you have the required permissions to access the repo.
> remotes::install_github("darwin-eu-dev/PaRe")
Downloading GitHub repo darwin-eu-dev/PaRe@HEAD

Yes you are correct. I reopened the PR. Once the actions finish I'll merge.

I updated the README.Rmd and the base branch accordingly.

Now you've applied this change with https://github.com/darwin-eu-dev/PaRe/pull/59/files#diff-b335630551682c19a781afebcf4d07bf978fb1f8ac04c6bf87428ed5106870f5 as well. So this PR is no longer of need.


Add missing dependency.

Signed-off-by: Henner Zeller<EMAIL_ADDRESS>

Codecov Report

Merging #1160 (db91626) into master (0834c7f) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1160   +/-   ##
=======================================
  Coverage   92.86%   92.86%
=======================================
  Files         337      337
  Lines       22745    22745
=======================================
  Hits        21123    21123
  Misses       1622     1622

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 0834c7f...db91626. Read the comment docs.


Changes on 0.9.0

Various changes applied to the 0.9.0 site to get the website and docs updated for the new release.

Looks good.

@richardcloudsoft Is this good to be merged or do you want to incorporate other pull requests first?

Thanks @sjcorbett, since these changes are already live on the public website (sourced from the 0.9.0 branch) they should probably be merged first!


(WIP) Initial attempt at setting up CI

Adds a .travis.yml file
Copies the docker container definitions from https://github.com/apple/coremltools/tree/master/docker

@TobyRoseman Now that https://github.com/apple/turicreate/pull/961 is merged in, you may want to rebase or merge it to see if it fixes the mysterious test linking issues.

Just squashed all the previous commits and rebased. We'll see if this fixes our test linking issues.

@srikris - please review this pull request.


Update Grant-SPOSiteDesignRights.md

If you are setting to a security group then you need the name only, doesn't work when FQDN is used. When adding a specific user a FQDN works correctly as shown.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-online/Grant-SPOSiteDesignRights.md
:white_check_mark:Succeeded
View (sharepoint-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

:x: Validation status: errors

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-online/Grant-SPOSiteDesignRights.md
:x:Error

Details

:x:Error

Details

whiteboard/whiteboard-ps/whiteboard/whiteboard.md
:warning:Warning

Details

sharepoint/sharepoint-ps/sharepoint-online/Grant-SPOSiteDesignRights.md

[Error] Unable to load file: sharepoint-ps/sharepoint-online/Grant-SPOSiteDesignRights.md via processor: AzureCliDocumentProcessor: :34:(510) '```powershell
Grant-SPOSiteDesignRights `
...'
Expect Heading

[Error] Markdown.MAML.Transformer.HelpSchemaException: :34:(510) '```powershell
Grant-SPOSiteDesignRights     ...' Expect Heading at Microsoft.DocAsCode.Build.Engine.SingleDocumentBuilder.GetInnerContexts(DocumentBuildParameters parameters, IEnumerable1 processors, TemplateProcessor templateProcessor, IHostServiceCreator creator)
at Microsoft.DocAsCode.Build.Engine.SingleDocumentBuilder.BuildCore(DocumentBuildParameters parameters)
at Microsoft.DocAsCode.Build.Engine.DocumentBuilder.BuildCore(DocumentBuildParameters parameter, IMarkdownServiceProvider markdownServiceProvider, BuildInfo currentBuildInfo, BuildInfo lastBuildInfo)
at Microsoft.DocAsCode.Build.Engine.DocumentBuilder.Build(IList`1 parameters, String outputDirectory)
at Microsoft.DocAsCode.SubCommands.DocumentBuilderWrapper.BuildDocument(BuildJsonConfig config, TemplateManager templateManager, String baseDirectory, String outputDirectory, String pluginDirectory, String templateDirectory)
at Microsoft.DocAsCode.SubCommands.DocumentBuilderWrapper.BuildDocument()

whiteboard/whiteboard-ps/whiteboard/whiteboard.md

Line 12: [Warning] Invalid file link:(~/whiteboard-ps/whiteboard/overview.md).

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

:x: Validation status: errors

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-online/Grant-SPOSiteDesignRights.md
:x:Error

Details

:x:Error

Details

whiteboard/whiteboard-ps/whiteboard/whiteboard.md
:warning:Warning

Details

sharepoint/sharepoint-ps/sharepoint-online/Grant-SPOSiteDesignRights.md

[Error] Unable to load file: sharepoint-ps/sharepoint-online/Grant-SPOSiteDesignRights.md via processor: AzureCliDocumentProcessor: :35:(510) '```powershell
Grant-SPOSiteDesignRights `
...'
Expect Heading

[Error] Markdown.MAML.Transformer.HelpSchemaException: :35:(510) '```powershell
Grant-SPOSiteDesignRights     ...' Expect Heading at Microsoft.DocAsCode.Build.Engine.SingleDocumentBuilder.GetInnerContexts(DocumentBuildParameters parameters, IEnumerable1 processors, TemplateProcessor templateProcessor, IHostServiceCreator creator)
at Microsoft.DocAsCode.Build.Engine.SingleDocumentBuilder.BuildCore(DocumentBuildParameters parameters)
at Microsoft.DocAsCode.Build.Engine.DocumentBuilder.BuildCore(DocumentBuildParameters parameter, IMarkdownServiceProvider markdownServiceProvider, BuildInfo currentBuildInfo, BuildInfo lastBuildInfo)
at Microsoft.DocAsCode.Build.Engine.DocumentBuilder.Build(IList`1 parameters, String outputDirectory)
at Microsoft.DocAsCode.SubCommands.DocumentBuilderWrapper.BuildDocument(BuildJsonConfig config, TemplateManager templateManager, String baseDirectory, String outputDirectory, String pluginDirectory, String templateDirectory)
at Microsoft.DocAsCode.SubCommands.DocumentBuilderWrapper.BuildDocument()

whiteboard/whiteboard-ps/whiteboard/whiteboard.md

Line 12: [Warning] Invalid file link:(~/whiteboard-ps/whiteboard/overview.md).

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


ES6 import with TypeScript

We are using TypeScript with "esModuleInterop": true.
Importing polly using the default export like so:
import polly from 'polly-js';

throws TypeError: polly_js_1.default is not a function at runtime.
If we try to use a namespace import:
import * as polly from 'polly-js';

Then we get TS2349: This expression is not callable. Type 'typeof polly' has no call signatures during compilation.
Help? :)
Thanks!

The runtime error was coming from tests, which have a project reference to the code in question. The tests did not have esModuleInterop set, whereas the project that was referenced did. Adding esModuleInterop to tests solved the issue for us. Not sure if there is a better way to address this though.

The runtime error was coming from tests, which have a project reference to the code in question. The tests did not have esModuleInterop set, whereas the project that was referenced did. Adding esModuleInterop to tests solved the issue for us. Not sure if there is a better way to address this though.

Ahh, I was about to ask you for a repo because I am using esModuleInterop all the time without problems. Guess I can close this then as the problem is solved.

Ahh, I was about to ask you for a repo because I am using esModuleInterop all the time without problems. Guess I can close this then as the problem is solved.


Set Docker Install URL 26.1 as the default for RKE1

Is your feature request related to a problem? Please describe.
According to user request in
(https://github.com/rancher/dashboard/issues/10996), set the Set Docker Install URL 26.1 as the default for RKE1
Describe the solution you'd like
Set Docker Install URL 26.1 as the default

@IsaSih , the default was changed on backend when https://github.com/rancher/rancher/pull/45632 was merged on June 21st and I've just tried on v2.9-head (13494c6a541d408b00e78415d18128f642f295d0) and 26.1 is the default even when shown by UI:

Could you please either close this issue or provide more information on how to reproduce it?(looks like our conversations happened right around June 21st (e.g. https://github.com/rancher/dashboard/issues/10996#issuecomment-2182902358) and it wasn't checked after that time and before this issue was filed on July 4th).