Return a Column class for relationship attributes that require it

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 SQLModel documentation, with the integrated search.
[X] I already searched in Google "How to X in SQLModel" 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 SQLModel but to Pydantic.
[X] I already checked if it is not related to SQLModel but to SQLAlchemy.

Commit to Help

[X] I commit to help with one of those options 👆

Example Code
class UpsertByModelMixin(SQLModel):
    created_by_id : Optional[int] = Field(default=None, foreign_key="users.id")
    created_by: Optional["User"] = Relationship(sa_relationship_kwargs={ 'foreign_keys': [created_by_id] })

    updated_by_id : Optional[int] = Field(default=None, foreign_key="users.id")
    updated_by: Optional["User"] = Relationship(sa_relationship_kwargs={ 'foreign_keys': [updated_by_id] })

class Team(UpsertByModelMixin, SQLModel, table=True,):
    __tablename__ = 'teams'

    id: Optional[int] = Field(default=None, primary_key=True)
    name: str = Field(max_length=500)

### Description

Use models such as above will result in the following error because the `foreign_keys` argument expects a `Column` (not a SQLModel `Field`).

| sqlalchemy.exc.ArgumentError: Column expression expected for argument 'foreign_keys'; got FieldInfo(default=PydanticUndefined, extra={'exclude': None, 'include': None}).

As such, its impossible right now for anyone to deploy classes with multiple FKs to the same identity and/or use custom named foreign key columns.

### Wanted Solution

To be able to specify foreign keys to the same related entity and/or name my fk columns however I choose.

### Wanted Code

```python
Just like above

Alternatives
No response
Operating System
Linux
Operating System Details
No response
SQLModel Version
0.0.4
Python Version
3.9.5
Additional Context
No response

Maybe something like this would work ...
owner_id : Optional[int] = Field(foreign_key="users.id")
owner: Optional["User"] = Relationship( sa_relationship_kwargs={ 'foreign_keys': owner_id.as_sa_column })

I've encountered the same issue and after reading this SQLAlchemy guide I've managed to work around it.
Based to the guide your code should look like this:
class UpsertByModelMixin(SQLModel):
    created_by_id : Optional[int] = Field(default=None, foreign_key="users.id")
    created_by: Optional["User"] = Relationship(sa_relationship_kwargs={ 'foreign_keys': '[UpsertByModelMixin.created_by_id]' })

    updated_by_id : Optional[int] = Field(default=None, foreign_key="users.id")
    updated_by: Optional["User"] = Relationship(sa_relationship_kwargs={ 'foreign_keys': '[UpsertByModelMixin.updated_by_id]' })

it would be nice if Relationship had a local_key param to handle this, to avoid a lot of boilerplate.
Ex, instead of
from sqlmodel import Field, Relationship, SQLModel

class User(SQLModel, table=True):
    ...

class Example(SQLModel, table=True):
    id: int = Field(primary_key=True)
    adminc: str | None = Field(default=None, foreign_key="users.username")
    techc: str | None = Field(default=None, foreign_key="users.username")
    adminc_alt: str | None = Field(default=None, foreign_key="users.username")
    techc_alt: str | None = Field(default=None, foreign_key="users.username")

    admin_contact: User | None = Relationship(sa_relationship_kwargs={"foreign_keys": 'Network.adminc'})
    tech_contact: User | None = Relationship(sa_relationship_kwargs={"foreign_keys": 'Network.techc'})
    admin_contact_alt: User | None = Relationship(sa_relationship_kwargs={"foreign_keys": 'Network.adminc_alt'})
    tech_contact_alt: User | None = Relationship(sa_relationship_kwargs={"foreign_keys": 'Network.techc_alt'})

It would be nice to be able to write
from sqlmodel import Field, Relationship, SQLModel

class User(SQLModel, table=True):
    ...

class Example(SQLModel, table=True):
    id: int = Field(primary_key=True)
    adminc: str | None = Field(default=None, foreign_key="users.username")
    techc: str | None = Field(default=None, foreign_key="users.username")
    adminc_alt: str | None = Field(default=None, foreign_key="users.username")
    techc_alt: str | None = Field(default=None, foreign_key="users.username")

    admin_contact: User | None = Relationship(local_key='adminc')
    tech_contact: User | None = Relationship(local_key='techc')
    admin_contact_alt: User | None = Relationship(local_key='adminc_alt')
    tech_contact_alt: User | None = Relationship(local_key='techc_alt')


Add new features

Please Add Subscript and Superscript

Hi,
You can use Subscript and Superscript using SpanStyle:
richTextState.toggleSpanStyle(
    SpanStyle(
        baselineShift = BaselineShift.Subscript
    )
)

richTextState.toggleSpanStyle(
    SpanStyle(
        baselineShift = BaselineShift.Superscript
    )
)

Not work correctly please test it

It's working fine for me, can you share some code.

Ok I will check and if any issue I will send video

Please export it to html and import it
And will see issue in html
Because not make sup and sub

Sorry for the late reply, I didn't see your comment.
Please can you try the latest version, I can't notice any problem with it.


absl.flags._exceptions.UnrecognizedFlagError: Unknown command line flag 'nwords'

how to solve this?

Hi , You can modify two files as follows.
in main.py:
...
#FLAGS.nwords = len(word2idx)
...
model = MemN2N(FLAGS, len(word2idx) ,sess)
...
in model.py:
...
class MemN2N(object):
def init(self, config, nwords, sess):
self.nwords = nwords
...


Watchtower (Dynamax Crystal Den) Missing/Not Missing (Den 16 actually missing)

Hello Leanny,
I noticed the Watchtower den which can only be activated with Dynamax Crystals (aka DLC) is not currently in the Display Raids list. The datamine of the latested Nest Report datamine confirm we will be receiving the 3rd Dynamax Crystal (DLC) shortly.
Will be den location be added to plug-in at some point?

Hi. This den requires more research, as it has different mechanics from the normal ones (i.e. IVs for specific slots are fixed and others). As soon as I finished my research I will add it

Would providing any sample saves help?

No it's fine, I am just not able to do any research before new year. After that I will start with my research and implement the special den

For anyone reading this:
The difference starts at Den 16/17. For the Raid Viewer to be correct, the Raid Viewer would have to show 100 Dens (not 99) as the broken down Watchtower for DLC Raid Crystals is in that 100. Without that placeholder, every Raid past 16, is 1 number higher than the actual.
Also, the Den just north of the tower, isn't actually showing in the Raid Viewer currently as a result of this mix-match.

Hey Leanny,
I know that there's delays with working on the Watchtower but would you re-add the missing Den and fix the numbering (by adding a placeholder of some-sort for the DLC Den)? Sometime soonish?
Thanks

Den 16 was skipped on purpose as its the event den. I will have a new version ready early next year, probably tomorrow or the day after.

You appear to have this backwards.
I have a clear image of the Den 16 active but according to the Raid Viewer there are no active Dens ranged #10 to #20 in my save. Where's Den 16 in Raid Viewer?
Den #17 is the Watchtower Den!
You may be showing the correct Nests for #16 in the Raid Viewer but as you can see from the SEED, the Raid Viewer is clearly showing the Watchtower Den (#17) in the Raid Viewer's #16 slot.
I am saying your Raid Viewer is NOT showing Den (#16) at all.

Ah sorry for misunderstanding. I will fix this in the next version as well!

Thanks!!
(I cannot wait for your drop down to show 100 instead of 99! I know you may not have it all figured out thus far but for those of us with back-ups with these Dynamax Crystals... being able to see the Shiny Frame Advancement is huge. Even if we are unsure how to advance it.)


[Bug]: SimpleDirectoryReader fails to find files if recursive=True is set

Bug Description
  documents = SimpleDirectoryReader(
    input_dir = document_dir,
#    recursive = True,
    exclude_hidden=False,
  ).load_data()

works
However
  documents = SimpleDirectoryReader(
    input_dir = document_dir,
    recursive = True,
    exclude_hidden=False,
  ).load_data()

produces
ValueError: No files found in /document/dir.
Version
0.11.2
Steps to Reproduce
Use SimpleDirectoryReader with recursive=True
Relevant Logs/Tracbacks
No response

@dosu
SimpleDirectoryReader fails only when "recursive=True" is provided but there are no subdirectories within the directory that is specified.
If /tmp_dir contains 1.pdf, 2.pdf, and 3.pdf, they will not be found.


1st commit

Install requirements.txt

set paths in cron.py

run cron.py python file

set log file in crontab -e

File "cron.py", line 4, in
cron = CronTab(user='username')

Add server username ?

Replying to self: yes. Ready. Maybe add in the readme how to set the log file in the crontab -e and in what python version and OS this runs successfully.


EnvelopeTemplate modele definition bug

Example EG008CreateTemplate.php here
        # Create top two objects
        $envelope_template_definition = new EnvelopeTemplateDefinition([
            'description' => "Example template created via the API",
            'name' => $this->template_name,
            'shared' => "false"
        ]);
        # Top object:
        $template_request = new EnvelopeTemplate([
            'documents' => [$document], 'email_subject' => "Please sign this document",
            'envelope_template_definition' => $envelope_template_definition,
            'recipients' => new Recipients([
            'signers' => [$signer], 'carbon_copies' => [$cc]]),
            'status' => "created"
        ]);

this declaration :
   'envelope_template_definition' => $envelope_template_definition,

does not match anything in the EnvelopeTemplate model definition => here so it does not work...
Can you check the mistake please, thanks!

Thank you for reporting this issue to us. EnvelopeTemplate does not contain envelope_template_definition. "description", "name" and "shared" are properties of EnvelopeTemplate

@DylanGD fix deployed


fix(costmodels): show close dialog on wizard close if >20% is filled

Display the exit confirmation dialog in the following cases:

Cost model type is Amazon or AWS and the user is in one of these steps: sources, review
Cost model type is Openshift and the user is in one of these steps: markup, sources, review
Cost model type is Openshift and the user is in the price list step and added at least one rate

Codecov Report

Merging #1589 into master will increase coverage by 0.38%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1589      +/-   ##
==========================================
+ Coverage   78.33%   78.71%   +0.38%
==========================================
  Files         235      237       +2
  Lines        3886     3881       -5
  Branches      716      712       -4
==========================================
+ Hits         3044     3055      +11
+ Misses        743      727      -16
  Partials       99       99

Impacted Files
Coverage Δ

src/store/djangoUtils/query.ts
100.00% <0.00%> (ø)

src/store/djangoUtils/pagination.ts
89.47% <0.00%> (ø)

src/store/sourceSettings/selectors.ts
100.00% <0.00%> (+8.33%)
:arrow_up:

src/store/costModels/selectors.ts
82.35% <0.00%> (+17.64%)
:arrow_up:

Continue to review full report at Codecov.

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

LGTM

Thanks you @nlcwong and @ddonahue007 :bow:


Error drill-down keys

The keys are stored as strings, for some reason kview wants to convert them to BigInt
An error occurred during route handling or page rendering.
SyntaxError: Cannot convert user_2b8oavU3nky3ec2MeMO82vMlwm to a BigInt
    at BigInt (<anonymous>)
    at pathToKey (https://deno.land/x/kview@0.9.1/utils/kv.ts:52:16)
    at GET (https://deno.land/x/kview@0.9.1/routes/api/kv/[id]/[...path].ts:30:39)
    at routes.<computed>.methods.<computed> (https://raw.githubusercontent.com/denoland/fresh/main/src/server/context.ts:461:20)
    at handler (https://raw.githubusercontent.com/denoland/fresh/main/src/server/router.ts:226:28)
    at ctx.next (https://raw.githubusercontent.com/denoland/fresh/main/src/server/compose.ts:97:32)
    at setStateFromCookies (https://deno.land/x/kview@0.9.1/routes/_middleware.ts:49:21)
    at eventLoopTick (ext:core/01_core.js:168:7)
    at async handler (https://raw.githubusercontent.com/denoland/fresh/main/src/server/context.ts:293:14)
    at async ext:deno_http/00_serve.js:455:18

Hmmm... must be a logic error in how keys are being deserialized from paths. Thanks for reporting.

🤦 the logic to detect BigInt keys in a path was very flawed. I hardened the whole serializing/deserializing.  It is available in 0.10.1.

Works, thanks for the quick turnaround.


No Devices Shown

Hey Konstantin,
first Thank you for the Work!
I do not get any Devices by now, the Hub has been created succesfully but then... nothing.

Best regards

Here the Log:
Logger: homeassistant.util.loop
Quelle: util/loop.py:136
Erstmals aufgetreten: 20:40:38 (1 Vorkommnisse)
Zuletzt protokolliert: 20:40:38
Detected blocking call to load_verify_locations with args (<ssl.SSLContext object at 0x7f8336f740d0>,) inside the event loop by custom integration 'groheblue' at custom_components/groheblue/init.py, line 58: devices = await client.get_devices() (offender: /usr/local/lib/python3.12/site-packages/httpx/_config.py, line 149: context.load_verify_locations(cafile=cafile)), please report it to the author of the 'groheblue' custom integration For developers, please see https://developers.home-assistant.io/docs/asyncio_blocking_operations/#load_verify_locations Traceback (most recent call last): File "", line 198, in _run_module_as_main File "", line 88, in _run_code File "/usr/src/homeassistant/homeassistant/main.py", line 223, in  sys.exit(main()) File "/usr/src/homeassistant/homeassistant/main.py", line 209, in main exit_code = runner.run(runtime_conf) File "/usr/src/homeassistant/homeassistant/runner.py", line 189, in run return loop.run_until_complete(setup_and_run_hass(runtime_config)) File "/usr/local/lib/python3.12/asyncio/base_events.py", line 674, in run_until_complete self.run_forever() File "/usr/local/lib/python3.12/asyncio/base_events.py", line 641, in run_forever self._run_once() File "/usr/local/lib/python3.12/asyncio/base_events.py", line 1990, in _run_once handle._run() File "/usr/local/lib/python3.12/asyncio/events.py", line 88, in _run self._context.run(self._callback, *self._args) File "/usr/src/homeassistant/homeassistant/config_entries.py", line 742, in async_setup_locked await self.async_setup(hass, integration=integration) File "/usr/src/homeassistant/homeassistant/config_entries.py", line 594, in async_setup result = await component.async_setup_entry(hass, self) File "/config/custom_components/groheblue/init.py", line 58, in async_setup_entry devices = await client.get_devices()

Hey, thanks for sending the log. The error message you get generally shouldn't be responsible for breaking the integration. Because I only wrote the integration in the past few days, I haven't really implemented error logging.
To make this easier, would you be able to run this simple python script with your credentials and tell me if it works? Just to make sure that the reason isn't the package for connecting with the grohe api.
First:
pip install asyncio groheblue

Then:
from groheblue import GroheClient

client = GroheClient("<EMAIL>", "<PASSWORD>")

async def main():
    await client.login()

    devices = await client.get_devices()  # get all devices
    print(devices)

    for device in devices:
        print(device.appliance_id)

if __name__ == "__main__":
    import asyncio

    asyncio.run(main())

Also, a major disadvantage of the groheblue python package is that it currently only supports one location and one room in the Grohe app. Could you check in your Grohe App that only one location (house) and only one room exists?

This issue should be resolved in the latest release (0.2.0). Feel free to open another issue if the problem persists.


add agent.py in-code documentation

Can you add newlines between arguments & returns? I believe this is the convention we have most other places.


License for the released Smart TV/remote QA data

Hi,
For the released 684 samples for Smart TV/remote QA, I could not locate a clear indication of any sort of license terms. The only relevant info I could find is this closed issue (https://github.com/abhi1nandy2/EMNLP-2021-Findings/issues/1#issuecomment-1080094627), where you mentioned "Also, since this project was in collaboration with Samsung, we are unable to provide with the full S10 QA Dataset at this time. However, the questions follow the format of data/smart_remote_QA.zip, and I would request you to use that for the time being. Also, if you would like to use our code/data, we request you to cite our paper."
Can you please put a license for the released Smart TV/remote QA data?
Thanks!

We have put a license on the repo.


[6.0] SILOptimizer: Allow inlining of transparent functions in @backDeployed thunks

In order for availability checks in iOS apps to be evaluated correctly when running on macOS, the application binary must call a copy of _stdlib_isOSVersionAtLeast_AEIC() that was emitted into the app, instead of calling the _stdlib_isOSVersionAtLeast() function provided by the standard library. This is because the call to the underlying compiler-rt function __isPlatformVersionAtLeast() must be given the correct platform identifier argument; if the call is not emitted into the client, then the macOS platform identifier is used and the iOS version number will be mistakenly interpreted as a macOS version number at runtime.
The _stdlib_isOSVersionAtLeast() function in the standard library is marked @_transparent on iOS so that its call to _stdlib_isOSVersionAtLeast_AEIC() is always inlined into the client. This works for the code generated by normal if #available checks, but for the @backDeployed function thunks, the calls to _stdlib_isOSVersionAtLeast() were not being inlined and that was causing calls to @backDeployed functions to crash in iOS apps running on macOS since their availability checks were being misevaluated.
The SIL optimizer has a heuristic which inhibits mandatory inlining in functions that are classified as thunks, in order to save code size. This heuristic needs to be relaxed in @backDeployed thunks, so that mandatory inlining of _stdlib_isOSVersionAtLeast() can behave as expected. The change should be safe since the only @_transparent function a @backDeployed thunk is ever expected to call is _stdlib_isOSVersionAtLeast().
Resolves rdar://134793410.

@swift-ci please test

@swift-ci please test


Fixed issue #2173

Title
Fixed issue #2173
run spotdl-4.2.7-win32.exe but show 404
Description

Edited web.py to handle the download of web app files and create a dist folder when using a cached web app.

Improved error handling: when no web app files are present, the script now opens localhost:8080 and returns a {'details': 'not found'} response instead of failing silently.

Enhanced test_matching.py to gracefully handle AssertionError exceptions.

Verified that everything works by testing the .exe file with an internet connection, confirming that the Web UI opens as expected.

Related Issue
Issue #2173

Motivation and Context
It solved showing of Error 404 in WEB UI app. Thus, improving the user experience and ensuring the application functions as intended.
How Has This Been Tested?
Tested changes by running pytest in test folder of spotdl(only).
Had to edit test_matching.py test_artist.py in spotdl folder to gracefully handle the error and update the outdated artists link.
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

[ ] 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 read the  CORE VALUES document
[x] I have added tests to cover my changes
[x] All new and existing tests passed

Has this issue been fixed in the public release? I'm currently experiencing the same issue after downloading and running 4.2.7.

Has this issue been fixed in the public release? I'm currently experiencing the same issue after downloading and running 4.2.7.

Not yet, try following these steps https://github.com/spotDL/spotify-downloader/issues/2177#issuecomment-2324712272

I have this issue here even after up/downgrading python version.


setup a module-sync for puppet community

we should probably offer (our own?) module sync, to keep puppet-community module's up-to-date with the current recommended technology.

Here is some less than optimal scripting around the secrets:
https://github.com/puppet-community/plumbing/blob/travis/share/travis_secrets and puppet-community/plumbing#1

so i thought i should move this issue to plumbing, then realised, this is @github. you can't do that.

btw, closing as per our CONTRIBUTING.md since this repository is for issues with the community, and we haven't really done anyone any wrong yet ;)


(REQUEST) spectator list

does anyone have a code for me to put "Spectators" on top of osiris spectator list?
Example: https://prnt.sc/tugt9w

its already on osiris

its already on osiris

but I want to put the text "Spectators" on top of the normal spectator list

Just use the "Watermark" code as base.

Just use the "Watermark" code as base.

thank u


Typo in the README example [Fix Provided!]

I was going through the README and noticed this typo against raylib-cpp's getter/setter methods and the declared variable. In particular, this line:
		// Object methods.
		logo.Draw(
			screenWidth / 2 - texture.getWidth() / 2,
			screenHeight / 2 - texture.getHeight() / 2);

should be changed to:
		// Object methods.
		logo.Draw(
			screenWidth / 2 - logo.GetWidth() / 2,
			screenHeight / 2 - logo.GetHeight() / 2);

P.s. getWidth and getHeight should be capitalized as GetWidth and GetHeight too.


remove the duplicate validatePodSandboxConfig

Signed-off-by: yanxuean<EMAIL_ADDRESS>the duplicate  	validatePodSandboxConfig check.
https://github.com/Mirantis/virtlet/blob/bff613a8c6b817a0ccc5eed50d957af493650e3f/pkg/manager/manager.go#L163
https://github.com/Mirantis/virtlet/blob/bff613a8c6b817a0ccc5eed50d957af493650e3f/pkg/manager/manager.go#L174

This change is

Review status: 0 of 1 files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Review status: 0 of 1 files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Review status: 0 of 1 files reviewed at latest revision, all discussions resolved, all commit checks successful.

Comments from Reviewable


Acond: fix issue of parsing attestation data from client

Changes look good. But did you run cargo fmt?

I run 'rustfmt --edtion=2018'. Both commands should have the same functionality?


[PBS] No support for Indpendent Lens and videos.pbs.org not selecting highest quality.

Hi all,
It appears that the PBS IE presently doesn't support PBS' Independent Lens documentaries. Some example URLs
http://www.pbs.org/independentlens/graduates
http://www.pbs.org/independentlens/waiting-room/
Also, when using the extractor on direct video URLs like: http://video.pbs.org/video/2365163372/ I notice that the IE grabs the video URL via JSON from http://video.pbs.org/videoInfo/2365163372/?format=json as specified by "alternate_encoding".  This appears to be a direct link to a -medium- quality version of video. In this case, resolution 416x234.
If instead, the IE downloaded the M3U8 file as specified by "recommended_encoding", inside the M3U8 there is an option for better quality, in this case: 640x360.
#EXTM3U
#EXT-X-STREAM-INF:PROGRAM-ID=1,BANDWIDTH=504000,CODECS="avc1.4d001f, mp4a.40.5",RESOLUTION=416x234
inle1511_mezz16x9-16x9-hls-400k.m3u8
#EXT-X-STREAM-INF:PROGRAM-ID=1,BANDWIDTH=1093000,CODECS="avc1.4d001f, mp4a.40.5",RESOLUTION=640x360
inle1511_mezz16x9-16x9-hls-800k.m3u8
#EXT-X-STREAM-INF:PROGRAM-ID=1,BANDWIDTH=229000,CODECS="avc1.4d001f, mp4a.40.5",RESOLUTION=320x180
inle1511_mezz16x9-16x9-hls-150k.m3u8
#EXT-X-STREAM-INF:PROGRAM-ID=1,BANDWIDTH=64000,CODECS="mp4a.40.5"
inle1511_mezz16x9-16x9-hls-64k.m3u8

There are no videos embedded on http://www.pbs.org/independentlens/graduates/ and http://www.pbs.org/independentlens/waiting-room/.


[WIP] decode chunks without sample limit and compare against normal decoding

Adding a test which generates a block with chunks and then decodes the chunks once with the sample count limit and once without the sample count limit. Then it compares the decoded samples to each other and prints chunks which resulted in different sets of samples.
This is an experiment, I have no intention to merge it, there is no need to review it.

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


COMP: build with legacy off

Linux Python packages fail due to:
Wrapping/Modules/MeshToPolyData/itkMeshToPolyDataFilterPython.cpp:17561:1: fatal error: error writing to /tmp/ccHel69H.s: No space left on device

Hmmm, build restarted.


Maya Scene exports do not default to including nodes that not children of members

Changelog Description

On Maya scene exports only include the relevant history for the selected nodes downstream and upstream and not upstream, and also their downstream descendant children.
Additional info

Note: This may affect maya scene exports, camera rig exports and layout exports. However, I do feel this is a way more sensible default behavior on exports with construction history enabled.
With this change, if you have a hierarchy like:
grp/
    obj1
    obj2

And obj1 is inside the instance then after this PR obj2 is not included.
Before this PR all other descendents from upstream groups would be included, regardless of whether they were "inputs" to the obj1.
After this PR, if obj2 is an input connection to obj1 (e.g. there are active constraints driving obj1 or some other connections driving it) then obj2 will still be included.
As such, only objects actively contributing to members of the instance will still be included in the output.
Testing notes:

Recreate the aforementioned hierarchy from additional info.
Publish obj1 - it should not include obj2
Now make a connection from obj2 to obj1 (e.g. translate connection)
Now obj2 should also be included in obj2.
Now disconnect the connection, and key obj1  transforms.
The keys should be exported along just fine.

Fingers crossed this won't break expectancy for anyone else. :) 🤞


programmatically setting dimension show to false doesn't hide element on chart

In the dimensions if you init a dimension to "show":false and then you programmatically set it to true it should then show up on the chart... the same should be true the other way. If I set it to true and then I set it to false it should disappear from the chart.
The reason i'm asking for something like this is to achieve something like this:
http://jsfiddle.net/panubear/q8h39/
Right now the only way you could do this is to remove it from the data set all together which doesn't make any sense and in a real world scenario where this is being used in a web application you wouldn't want to have to do that.
I would do this via chart.show and chart.hide but you are using an isolate scope on the directive thus making it impossible to use AngularChartService injected into my own controller in order in order to gain access to the chart object via AngularChartService.getInstance.

Hi @btm1 thank you for testing the directive and reporting back your findings. There was a bug, so the configuration was not reset when updating the options. I fixed this and uploaded version 0.3.1.
Please check if this suits your needs, accessing the chart directly using the service is a bigger task, if you really need it please open a new issue for this.

@maxklenk thanks for taking care of this.  As for accessing the chart... I'm not sure whether or not this is a good idea. If it were me and I had written it though I might have not used an isolate as I'm not really sure it's necessary in this case and then using the method in the service could return the state of the chart via the getInstance method. This would at least allow for some more advanced use cases where a  user may need to use the c3 methods stored on this object.
I would keep it in mind for the future...

We will think about a solution. It is always good to work with isolated scopes to avoid unintended behaviour. On the other hand it is not that easy to access a chart using a service, because there may be multiple charts and a way to distinguish between them has to be found.

You are creating unique IDs for to apply to the dom element so you could use that same idea as a key and store the chart objects in a variable in a service and then create a method to retrieve a chart object based on a key value.


invalid syntax trainer.trainModel()

Help me currently i am doing at Jupyter notebook anaconda i got invalid syntax when running this code trainer.trainModel()
currently i am using Python 3.7.6 TenserflowVer: 2.4.0
from imageai.Detection.Custom import DetectionModelTrainer
trainer = DetectionModelTrainer()
trainer.setModelTypeAsYOLOv3()
trainer.setDataDirectory(data_directory="hololens")
trainer.setTrainConfig(object_names_array=["hololens"], batch_size=4, num_experiments=100, train_from_pretrained_model=("pretrained-yolov3.h5")
trainer.trainModel()

Currently i am trying to leaning doing this tutorial https://medium.com/deepquestai/train-object-detection-ai-with-6-lines-of-code-6d087063f6ff

My bad i add bracket in the code that why syntax error here is the working code
'''
from imageai.Detection.Custom import DetectionModelTrainer
trainer = DetectionModelTrainer()
trainer.setModelTypeAsYOLOv3()
trainer.setDataDirectory(data_directory="hololens")
trainer.setTrainConfig(object_names_array=["hololens"], batch_size=4,
num_experiments=100, train_from_pretrained_model="pretrained-yolov3.h5")
trainer.trainModel()
'''

My bad i add bracket in the code that why syntax error here is the working code
''' from imageai.Detection.Custom import DetectionModelTrainer
trainer = DetectionModelTrainer() trainer.setModelTypeAsYOLOv3() trainer.setDataDirectory(data_directory="hololens") trainer.setTrainConfig(object_names_array=["hololens"], batch_size=4, num_experiments=100, train_from_pretrained_model="pretrained-yolov3.h5") trainer.trainModel()
'''

has the issue been resolved? if so, you can close this issue

@faisalhazry,
I was not able to find any Syntax error while executing the mentioned code. Kindly find the gist of it here.


[otbn, doc] Add information-flow data to OTBN ISA YAML.

As part of the YAML function specifications, write a machine-readable description of how data flows through each instruction, including consideration of flags, memory, indirect references, and special registers.
Information flow to/from DMEM treats memory as a monolithic block. However, the existing lsu field already provides information about where memory loads/stores occur and how many bytes they are.

This looks neat! I wonder whether you'll need to strengthen things a bit to avoid (spurious) apparent leakages through the flag registers, but it'd be cool if this just works.

True, if you extract the flags and then xor to get only one flag, this system will still think that register has information from all the flags, which could cause false positives. I'm planning to first implement an overly-conservative approach and then add extra capabilities as needed -- soundness before completeness!


[Super parameters] Super Key for Widgets

Describe the issue
When I was trying to migrate the key parameter to a super parameter, I got a linter hint that the key is never used.
Before:
After:
Could it be a false-positive? But the _MethodListView is a private class here, so it could be somehow valid in this case.
Expected behavior
I expected the key here not to be detected as unused, as providing the key parameter is a preferred rule in Flutter. And the previous style is valid since the key is actually used in the super call.

I am suspecting there are two things:

Is the parameter used in the method
Is the parameter being passed in the caller invocation as an argument

Consider
class A {
  A({this.s});

  String? s;
}

class _B extends A {
  _B({super.s});
}

class _C extends A {
  _C({String? s}) : super(s: s);
}

class _D extends A {
  _D({String? s}) : super();
}

void f() {
  _B();
  _C();
  _D();
//  _B(s: 'a'); // uncomment this
//  _C(s: 'a');
//  _D(s: 'a');
}

the super.s in _B is marked as unused, however once the _B(s: 'a') is used, the parameter is considered used.
_C has always s considered used, because it it used to call the super constructor.
_D behaves as _B
From the documentation of the diagnostic

Assuming that no code in the library passes a value for y in any invocation of _m, the following code produces this diagnostic:

I am guessing the behavior of _B and _D is correct, but _C should actually be changed, so that it is considered used only when the parameter is passed in an invocation, and not just being used in the method body.

We're facing more of this when we apply the rule to production projects, as we have various file-read-only widgets that have the key argument.

I am guessing the behavior of _B and _D is correct, but _C should actually be changed, so that it is considered used only when the parameter is passed in an invocation, and not just being used in the method body.

Indeed. IMO private constructors have better callers finding like arguments usage. A super argument seems to be defined as a private field here.

Duplicate of #49025


Issue #3803: Make ManagedLedger read batch size configurable

Motivation
Fixes #3803
Hardcoding is a very bad practice. It means we have no way to alter system behavior
when production issues occur.
Modifications
introduce a few read batch related settings to make them configurable
Does this pull request potentially affect one of the following parts:
If yes was chosen, please highlight the changes

Dependencies (does it add or upgrade a dependency): NO
The public API: NO
The schema: NO
The default values of configurations: NO
The wire protocol: NO
The rest endpoints: NO
The admin cli options: NO
Anything that affects deployment: NO

Documentation

Does this pull request introduce a new feature? NO
If yes, how is the feature documented? (not applicable)
If a feature is not applicable for documentation, explain why?

it is self-documented in configuration file

If a feature is not documented yet in this PR, please create a followup issue for adding the documentation

run cpp tests
run integration tests

Merged in 2.3.1 at
e091a9b


Frame parameter not included

I tried to include "frames" property to the object returned by the script  (as described in Plotly's documentation https://plotly.com/javascript/plotlyjs-function-reference/ )
return {data:data, layout:layout, frames: frames};
and I got this error message
plotly.js:107614 WARN: API call to Plotly.animate rejected. undefined
Do you have plans to include this in the feature?

Here is the example I tested : https://github.com/ae3e/ae3e-plotly-panel/wiki/Frames

Thank you! I did several animations and I can confirm that it works very well.


TensorFlow Lite Metadata duplication

@lu-wang-g @petewarden @aselle
ModelMetadata defines another description property when one is already defined on Model.
Would it make sense to encode model level properties like author, name, license, version as Metadata properties on the model itself instead of defining them in ModelMetadata. That way all TensorFlow Lite models could carry such information as part of the model without relying on model_schema.fbs for decoding (similar to metadata_props in ONNX).
See lutzroeder/netron#481

@lu-wang-g Could you please take a look?

@lutzroeder ModelMetadata is a centralized place to hold all metadata of the model. It is easier for users to get the information at once by parsing ModelMetadata. Since TFLite metadata is still at experimental, we didn't mix it with the existing fields in the TFLite schema. It is a good point that we should remove the duplicated fields in the TFLite schema once metadata_schema.fbs is mature.
Seems like you had a protptype to show TFLite metadata in Netron. It will be super help for the metadata use cases. Thanks for working on it. Let me know if you have any questions when implementing the feature. Thanks!

It is easier for users to get the information at once by parsing ModelMetadata.

Is that true? Since it's hidden inside the metadata of the .tflite file and requires a separate schema there is more complexity involved getting to this information. Especially for top-level model properties like author, description and license why require another standard if they can be added as (string, byte[]) to the model itself without changing the schema - isn't that the purpose of the metadata table in .tflite?

It is a good point that we should remove the duplicated fields in the TFLite schema once metadata_schema.fbs is mature.

Will the files that exist right continue to work if the schema keeps changing or will files that get created with the experimental feature right now be in an invalid format in the future?
Having two separate model descriptions is problematic right now. Which one should a tool show? The one in the .tflite file or the one in TFLITE_METADATA? Showing both is confusing as well...

Seems like you had a protptype to show TFLite metadata in Netron. It will be super help for the metadata use cases.

Netron 4.1.4 will show the metadata if present.

Was there a reason the metadata variable was called TFLITE_METADATA upper case instead of using lower case as other metadata property names? TFLITE seems redundant?

@lutzroeder

Is that true? Since it's hidden inside the metadata of the .tflite file and requires a separate schema there is more complexity involved getting to this information. Especially for top-level model properties like author, description and license why require another standard if they can be added as (string, byte[]) to the model itself without changing the schema - isn't that the purpose of the metadata table in .tflite?

The metadata can be extracted through the metadata extractor library. We've launched the Java version, and the C++ library will come soon. Users can also convert the metadata into a Json file through the metadata displayor. Where the metadata is located in the model should be transparent for users.
The metadata table in .tflite is not designed for the information like TFLite metadata, but mainly used as buffers of internal data that is supposed to be hidden from users. Having users editing against it directly is error prone, because the metadata table accepts data in any types and any formats in general.

Will the files that exist now continue to work if the schema keeps changing or will files that get created with the experimental feature right now be in an invalid format in the future?

Having two separate model descriptions is problematic right now. Which one should a tool show? The one in the .tflite file or the one in TFLITE_METADATA? Showing both is confusing as well...

Up to now, we're pretty confident about the Metadata schema and the fields it defines. It works well on common image models such as classification and object detection. It will be moved out from experimental soon. Regarding my previous comment of deprecating certain fields, it was an impression at first sight, and it's mainly just about the description field of the model. I agree it's quite confusing to have two model description fields, and I think the one in .tflite is not used that much so that should be OK to deprecate it. But we'll need to further evaluate if it makes sense to do so.

The UI looks so cool! Really appreciate your efforts for support TFLite Metadata in Netron!

Was there a reason the metadata variable was called TFLITE_METADATA upper case instead of using lower case like other metadata property names? TFLITE seems redundant?

It's just a token to identify the TFLite Metadata, and to be differentiated from other metadata entries in the metadata table. Again, it is supposed to be transparent to users, a nuance that hopefully won't be noticed by them.

Closing this issue for now. Feel free to reopen it if you have another questions. Thanks!


Does not work with a plain create-react-app project

Try:
npx create-react-app test
cd test
npx butler-cli src

npx: installed 2 in 0.992s
Object.values(...).flat is not a function

It works if I switch to node version 13.
Btw. great idea :)

Thanks, I'll add a polyfill for flat, I want this to work on older versions of node too.


Aura reserves no mana if socketed in Alleopathy gloves

Specifically i am using: Malevolence with Spell Totem and Blight in the same group for gloves.

im dumb af
totems are in fact able to be linked with auras. <.<


Make Inventory atomic and transactional

Would be nice to be able to make changes to the Inventory atomically (or require that all changes be like this)

Self assigned, working on this quietly.

Removing from stable milestone because this isn't actually necessary for anything


[GH-2384] Update resource names

Describe the changes you have made to improve this project
Update various resource names.
/!\ This PR will need a big in-depth review /!\
I have tried to create relatively small commits (except for style) per resource type and I have tried to follow as closely as possible the pattern we discussed about.
Final naming convention is:
MahApp.
+ the resource type which is one of Colors., Brushes., Templates., Sizes., Fonts., Storyboard., Styles.
+ the target control for Templates and Styles (e.g. Slider., Window....)
+ an optional specifier which gives more context (e.g. Slider for MahApps.Styles.Thumb which helps understanding that this thumb style is for the context of a slider)
+ an optional category (e.g. Metro, Flat, Square, Win10...)
Closed Issues
Fix #2384

@punker76 I don't know, how would you name MahApps.Sizes.ScrollBar.HorizontalRepeatButtonWidth?

And also do you want the names to be MahApps.Styles.Thumb.ScrollBar for the style of the Thumb that belongs to the scrollbar or MahApps.Styles.ScrollBar.Thumb? For now I have always put the target type first.

@punker76 I don't know, how would you name MahApps.Sizes.ScrollBar.HorizontalRepeatButtonWidth?

I must agree with you there.

And also do you want the names to be MahApps.Styles.Thumb.ScrollBar for the style of the Thumb that belongs to the scrollbar or MahApps.Styles.ScrollBar.Thumb? For now I have always put the target type first.

Yes, you're right. But we must explain this explicitly with some samples for better understanding. Can you add such a description to this PR above? Thx!

I'll update the PR first post and add a similar comment at the end of the issue with the final naming:
MahApp.

the resource type which is one of Colors., Brushes., Templates., Sizes., Fonts., Storyboard., Styles.
the target control for Templates and Styles (e.g. Slider., Window....)
an optional specifier which gives more context (e.g. Slider for MahApps.Styles.Thumb which helps understanding that this thumb style is for the context of a slider)
an optional category (e.g. Metro, Flat, Square, Win10...)

But while writing this summary I have realized that not all my renames are actually matching. For example, shall I go for MahApps.Styles.ButtonBase.MetroCircle (current) or MahApps.Styles.ButtonBase.Circle.Metro?

@Evangelink I'm more and more in favor of removing Metro from the name. Because this Name already exists in the name of this library.

@punker76 Are you referring to the .Metro suffix? If so I can do a new commit.
Are you ok with the other changes I made?
I think we are close to the end with those changes :D

@punker76 Are you referring to the .Metro suffix? If so I can do a new commit.

Yes.

Are you ok with the other changes I made?
I think we are close to the end with those changes :D

Yes 😃
Maybe these changes will be create some issues, but I think we are now clear with the naming style.

@punker76 Just a crazy thought, WDYT about creating a static class (or using resx) to host the name of the resources instead of relying on strings? This could be useful to provide some kind of strong typing.

@Evangelink You mean something similar to this?
https://github.com/MahApps/MahApps.Metro/blob/a5d0c3afbbc78dc33fc3a120d270ad80a33c870a/src/MahApps.Metro/Controls/Helper/TextBoxHelper.cs#L19-L36
Mh... 🤔  has also Pros and Contras...

@punker76 yep (for what I meant and for it has pros and cons)

I'm curious, what are the cons of strong typing the resource keys using the ComponentResourceKey?

@mgnslndh would this enable IntelliSense in XAML
Happy Coding
Tim


RSS feed from repository issues

You can use https://docs.rsshub.app/en/#issue 379 services.
You can filter by title and description

Repo Issues
support browser extension support rssbud
Author: @HenryQW @AndreyMZ
Example: https://rsshub.app/github/issue/DIYgod/RSSHub/all/bug
Route: /github/issue/:user/:repo/:state?/:labels?
Parameters:
user, required - GitHub username
repo, required - GitHub repo name
state, optional - the state of the issues. Can be either open, closed, or all. Default: open.
labels, optional - a list of comma separated label names

Repo Issues
support browser extension support rssbud
Author: @HenryQW @AndreyMZ
Example: https://rsshub.app/github/issue/DIYgod/RSSHub/all/bug
Route: /github/issue/:user/:repo/:state?/:labels?
Parameters:
user, required - GitHub username
repo, required - GitHub repo name
state, optional - the state of the issues. Can be either open, closed, or all. Default: open.
labels, optional - a list of comma separated label names

https://rsshub.app/github/issue/bacen/pix-perguntas-e-respostas/all

https://rsshub.app/github/issue/bacen/pix-perguntas-e-respostas/all

#Repo Pull Requests
support browser extension support rssbud
Author: @hashman
Example: https://rsshub.app/github/pull/DIYgod/RSSHub
Route: /github/pull/:user/:repo
Parameters:
user, required - User name
repo, required - Repo name

#Repo Pull Requests
support browser extension support rssbud
Author: @hashman
Example: https://rsshub.app/github/pull/DIYgod/RSSHub
Route: /github/pull/:user/:repo
Parameters:
user, required - User name
repo, required - Repo name

#Files Commits
support browser extension support rssbud
Author: @zengxs
Example: https://rsshub.app/github/file/DIYgod/RSSHub/master/lib/router.js
Route: /github/file/:user/:repo/:branch/:filepath+
Parameters:
user, required - User name
repo, required - Repo name
branch, required - Branch name
filepath+, required - File path

#Files Commits
support browser extension support rssbud
Author: @zengxs
Example: https://rsshub.app/github/file/DIYgod/RSSHub/master/lib/router.js
Route: /github/file/:user/:repo/:branch/:filepath+
Parameters:
user, required - User name
repo, required - Repo name
branch, required - Branch name
filepath+, required - File path

GitHub
TIP
GitHub provides some official RSS feeds:
Repo releases: https://github.com/:owner/:repo/releases.atom
Repo commits: https://github.com/:owner/:repo/commits.atom
User activities: https://github.com/:user.atom
Private feed: https://github.com/:user.private.atom?token=:secret (You can find Subscribe to your news feed in dashboard (opens new window)page after login)

GitHub
TIP
GitHub provides some official RSS feeds:
Repo releases: https://github.com/:owner/:repo/releases.atom
Repo commits: https://github.com/:owner/:repo/commits.atom
User activities: https://github.com/:user.atom
Private feed: https://github.com/:user.private.atom?token=:secret (You can find Subscribe to your news feed in dashboard (opens new window)page after login)


Breadcrumb URL is generated without route parameters for HttpPost action

Sitemap.xml (fragment):
  <mvcSiteMapNode title="Home" controller="Home" action="Index">
    <mvcSiteMapNode title="Projects" controller="Projects" action="Index">
      <mvcSiteMapNode title=" " controller="Projects" action="View" preservedRouteParameters="ProjectId">
        <mvcSiteMapNode title="Quote composition" controller="QuoteConfiguration" action="AddMoreSections" preservedRouteParameters="ProjectId,ProjectName"/>
        <mvcSiteMapNode title="Quote Generation" controller="QuoteGeneration" action="GenerateQuote" preservedRouteParameters="ProjectId,ProjectName" />
...

when I navigate to /QuoteConfiguration/AddMoreSections?ProjectId=1&QuoteId=2 (via HttpGet), the breadcrumb works fine: Project URL has ID:
/Projects/View?ProjectId=1
however when I navigate to: QuoteGeneration/GenerateQuote (HttpPost, ProjectId is in the post data), the ID in Project URL is missing:
/Projects/View
my action methods look similar for post and get:
        [HttpPost, ValidateAntiForgeryToken]
        [SiteMapTitle("ProjectName", Target = AttributeTarget.ParentNode)]
        public ActionResult GenerateQuote(GenerateQuoteViewModel model)
...
        [SiteMapTitle("ProjectName", Target = AttributeTarget.ParentNode)]
        public async Task<ActionResult> AddMoreSections(int projectId, int quoteId, string returnUrl)
...

In its current incarnation, preserved route parameters first tries to get the value from route values, then from the query string. The form collection is not part of the equation.
But then, this does seem like a good idea so I will consider this a feature request.


Add summary at the end of all run tests

Output as comments, similar to https://www.npmjs.com/package/tape#example.

These changes look sensible.  I'm no longer using this library and was about to archive it on GitHub.  Would you be interested in taking over maintenance?  If so, I'll transfer ownership of this repository to you.  Then you can merge the pull request and release a new pack.

Probably I'm the most active user of library(tap), so I would like to take over the project.

I tried the transfer a couple different ways, but GitHub keeps complaining because you already have a fork of this same repository.  It looks like we have two options:

you can delete your tap repository and I can transfer this one to you
I can archive this repository and update the README to point at your repo

The second one seems easier, but I'll do whatever you want since you'll be the new maintainer.

Hi Michael,

you can delete your tap repository and I can transfer this one to you

I just did this. I was wondering what would happen with this PR, but apparently GitHub did not crash :angel:

I initiated the transfer. Thanks for being willing to maintain this project.


400 Error when promoting Edge-App from Apple Silicon Machine

Environment Information:
CPU - Apple M1 Pro
OS - 14.1.2 (23B92)
Terminal - Warp
CLI Installed via Homebrew
brew info screenly-cli
==> screenly/screenly-cli/screenly-cli: stable v0.2.1
Command line interface is intended for quick interaction with Screenly through terminal.
https://github.com/Screenly/cli
/opt/homebrew/Cellar/screenly-cli/v0.2.1 (7 files, 11.9MB) *
  Built from source on 2023-12-07 at 21:57:22
From: https://github.com/screenly/homebrew-screenly-cli/blob/HEAD/Formula/screenly-cli.rb
License: MIT
==> Dependencies
Build: rust ✔

When promoting an Edge-App using the command screenly edge-app version promote --latest or  screenly edge-app version promote --revision=X the CLI returns Response: Ok("") Failed to promote edge app version: unknown error: 400. All other commands leading up to the promotion step successfully complete without issue.

This behavior has been reproduced using the Weather and Power BI sample Edge Apps that are located in the Screenly/Playground repository.
Working directory copied from the Mac to a Windows 11 machine and was able to successfully promote both Edge-Apps using the CLI.
Was able to reproduce the issue, from different locations, using different networks/ISP providers to rule out possible network traffic filtering.

Thanks for raising this @cocallaw. @korvyashka can you please investigate this tomorrow?

Updated now showing 0.2.3 and testing.
brew info screenly-cli
==> screenly/screenly-cli/screenly-cli: stable v0.2.3
Command line interface is intended for quick interaction with Screenly through terminal.
https://github.com/Screenly/cli
/opt/homebrew/Cellar/screenly-cli/v0.2.3 (7 files, 11.9MB) *
  Built from source on 2023-12-13 at 07:30:11
From: https://github.com/screenly/homebrew-screenly-cli/blob/HEAD/Formula/screenly-cli.rb
License: MIT
==> Dependencies
Build: rust ✔

@vpetersson I am able to successfully promote Edge-Apps using v0.2.3 of the CLI
screenly edge-app version promote --latest
Edge app version successfully promoted.

Awesome. Let me know if you need anything else.
It was a bit of a boo-boo on our end where we had to break the backward compatibility of an API endpoint.


Loading a nonexistent file displays exception

no additional text

#TODO
##Links

#55

#60

#57

Catch error for "File does not exist"


Toggle does not work over hidden icons

When I double-click an empty space on the desktop, the visibility of my icons won't change if that position has an icon.
For example, on my desktop below...

If I try double-clicking in the circled area where the GitHub Desktop icon would be, the visibility won't toggle.
I hope that makes sense.

double-clicking the blank space on the desktop
— From ReadMe

I'm not sure if you meant blank, even when hidden too
If so, I'll just decrease my icons size😅

I made a pull request (#4) which I tested and believe should fix this issue

Fixed in version 1.4 (https://github.com/hexandcube/desktop-icon-toggle/releases/tag/v.1.4)


refactor ui and calculate func

use list and map for showing buttons
use switch statement for onTap func of every buttons
a few ui change

use list and map for showing buttons
use switch statement for onTap func of every buttons
a few ui change


[Doc][5.10.0] Proper Resources are not Given for Writing a Custom Federated Authenticator

Is your suggestion related to a missing or misleading document? Please describe.
To get the sample source codes for building custom federated authenticators, the documents [1] & [2] still have references to the SVN repository which is no longer in use.
And the pom file is not given for the custom-authenticator-for-facebook in the "Click here to view the source code" section in [1].
Describe the improvement.

SVN links can be removed or the reference links to the source codes can be updated to link a working repository and the source codes should also be updated.
The pom file should be added in [1].

[1] - https://is.docs.wso2.com/en/5.10.0/develop/writing-a-custom-federated-authenticator/#custom-authenticator-for-facebook
[2] - https://is.docs.wso2.com/en/5.10.0/develop/writing-a-custom-federated-authenticator/#custom-authenticator-for-google

fixed with https://github.com/wso2/docs-is/pull/2745


Multi concept benchmarking

Hello.
Thanks for creating this repo.
Multi concept case is not working. It is not referenced properly in 'benchmarking.py' file.
I was able to run it after making few fixes. It will be great if you could take a look at it.
Thanks,
Ajitabh

Hi, can you elaborate more on the issue and how you fix it on your side?

I will look into it this month.

Fixed at bbe44d3


Always run python tests

These are required for PRs to land.

I don't like the wastefulness of triggering unnecessary CI runs, but after looking around in the branch protection tab of the repo's settings, I can't see any way around this.
Implementing something akin to a custom merge-gating bot is out of scope for us.


Break out component libraries

Many components of icenet are generalisable for data centric workflows. Many companion projects are relying on this happening in the not too distant future (though they can just use the icenet library in the meantime!)

[x] Data downloaders split out from library

This can drive the decoupling of the EASE2 projection which will be awesome for the downloaders

[x] Preprocessors split out from library
[ ] ...

Consider other components around the place, particularly in icenet-etl

Having undertaken the download-toolbox and preprocess-toolbox split for icenet 0.4, as well as looked at the structure within the icenet 0.3 for training and prediction, I've updated the description accordingly. There is possibility for too much to go into this, but might be some interesting harness / plotting movements into other tools that can reside in a more generalised environmental forecasting sense. Worth reviewing w.r.t icenet 0.4 once the work there is concluded.


Undefined confirmed_at on signup

Bug report
Describe the bug
When calling the signup method while email confirmations are enabled, an error is thrown: TypeError: Cannot read property 'confirmed_at' of undefined.
To Reproduce
First enable email confirmations in Supabase dashboard. Then:
const supabase = client.createClient(SUPABASE_URL, SUPABASE_API_KEY);
const createUser = async () => {
  try {
    const { body: { user } } = await<EMAIL_ADDRESS>"password");
    return user
  } catch (e) {
    return Promise.reject(e)
  }
};

createUser()
  .then(res => console.log(res))
  .catch(err => console.log(err))

Looks like the error is happening here:
https://github.com/supabase/supabase-js/blob/8318ed14ff33a7ee4a847509c97308420b1fafe1/src/Auth.js#L23
The response returned doesn't include a user key in the body. I think the body is the user object itself.
Expected behavior
Signup should return response from POST request to /signup on auth url .
System information

OS: macOS
Version of supabase-js: 0.36.4
Version of Node.js: 14.5.0

good catch @hansy will get this patched asap


⚠ SceneTime has degraded performance

In 08a262e, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 1446 ms

Resolved: SceneTime performance has improved in 94512e0 after 5 minutes.


[WIP] add QA to docs

What does this PR do?
just WIP collecting the frequent questions... @awaelchli do you have any other suggestions?
PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃

Not really a QA but maybe adding a note that inspecting the docs for formatting errors is also important when updating the docs (not just making the tests pass). If this is just for QA then we can consider adding this later.

@awaelchli could you please add it to this PR as a suggestion?

I can't add as suggestion, if you want you can copy paste from here.

changelog need to be rebased on new release #1419

One more idea: Maybe a QA for how to deprecate and keep backward compatibility. For how many versions should there be a warning until it gets removed? etc.

One more idea: Maybe a QA for how to deprecate and keep backward compatibility. For how many versions should there be a warning until it gets removed? etc.

it depends on what change - 0.2 for small or till major version for large...
but the deprecation shall be discussed in the particular PR/issue as it won't happen very often


Update for Swift 1.2

Another shot at #24

Ugh, my local copy was still pointed at Carthage's fork, apologies


Add full support for search Oct preview release

Given the preview SDK supporting Oct preview was done to unblock GA. We are aiming to have an SDK preview release targeting Oct preview with the following

Full convenience API
Samples
Full testing.

This is expected to be RC which we expect to GA in March end.

We have already added full support for the Oct preview, incorporating all the GA changes, convenience APIs, samples, and test cases. A preview version has been released and can be found at https://www.nuget.org/packages/Azure.Search.Documents/11.6.0-beta.1.


Bug: invalid asset json

https://cdn.gamelockerapp.com/semc-vainglory/eu/2017/11/15/23/28/b6a9f69a-ca5c-11e7-94fc-0a586460aaa8-telemetry.json

        {
            "time": "2017-11-15T23:23:50+0000",
            "type": "HealTarget",
            "payload": {
                "Team": "Left",
                "Actor": "*Idris*",
                "TargetActor": "*Idris*",
                "TargetTeam": "Left",
                "Source": "Buff_Item_HealingFlask",
                "Heal": 52,
                "Healed": 52,
                "IsHero": 1,
                "TargetIsHero": 1
            }
        },
        {
            "time": "2017-11-15T23:23:51+0000",
            "type": "DealDamage",
            "payload": {
                "Team": "Right",
                "Actor": "*Grace*",
                "Target": "*Idris*",
                "Source": "Buff_Item_StormguardBanner_Damage",
                "Damage": 11,
                "Dealt": 11,
                "IsHero": 1,
                "TargetIsHero": 1
            }
        },
        {
            "time": "2017{ "
            time ": "
            2017 - 11 - 15 T23: 23: 55 + 0000 ", "
            type ": "
            DealDamage ", "
            payload ": { "
            Team ": "
            Left ", "
            Actor ": " * Idris * ", "
            Target ": " * Grace * ", "
            Source ": "
            Ability__Idris__DefaultAttack ", "
            Damage ": 372, "
            Dealt ":  139, "
            IsHero ": 1, "
            TargetIsHero ": 1 } },{ "
            time ": "
            2017 - 11 - 15 T23: 23: 55 + 0000 ", "

FUBAR.

Oh wow, what happened here? Super busted.

We have minimal involvement with the telemetry data. We only turn it into JSON and perform a few string operations on it. This data appears to have come through broken, so we will be sending it over to SEMC for review.


I've updated to <IP_ADDRESS>, but still blur, acrylic and opaque is not working only black screen, using Windows 11 22H2

not fixed.

Same problem here. Did the feature update to Win11 22H2. Since then, the blur function no longer works. Taskbar stays black.

Also same here, tempted to make the bottom of my bg blurry for the time being


Consider refactoring LruCacheLayer with list_with_metakey and concurrent_stat_in_list

Consider refactoring it with list_with_metakey and concurrent_stat_in_list
Originally posted by @WenyXu in https://github.com/GreptimeTeam/greptimedb/pull/3058#discussion_r1438831103
https://github.com/GreptimeTeam/greptimedb/blob/2f98fa0d97195c4818a9db52032924700ca49577/src/object-store/src/layers/lru_cache/read_cache.rs#L135-L157

I want to try this. Please assign it to me. 😊

@suyanhanx Welcome! You're assigned and feel free to leave a comment here to share updates; or ping me as a reviewer if the patch is submitted.

I checked the current implementation and it currently calls mainly on the Accessor.
If we want to refactor it with the new methods list_with_metakey and concurrent_stat_in_list, I think we may need to change it to Operator instead.
This will remove the trait currently relied upon here and its upstream since Operator is not a trait.
https://github.com/GreptimeTeam/greptimedb/blob/2f98fa0d97195c4818a9db52032924700ca49577/src/object-store/src/layers/lru_cache/read_cache.rs#L72-L110

Hi,Can I give it a try?

Hi,Can I give it a try?

Have fun

Can I replace Access trait to Accessdyntrait?

/opendal/src/raw/accessor.rs:398
/// AccessDyn is the dyn version of [Access] make it possible to use as
/// Box<dyn AccessDyn>.

The Operator struct can build from dyn AccessDyn.
This way, I can construct the Operator in the recover_cache function, and then use list_with_metakey and concurrent_stat_in_list.

Can I replace Access trait to Accessdyntrait?

/opendal/src/raw/accessor.rs:398
/// AccessDyn is the dyn version of [Access] make it possible to use as
/// Box<dyn AccessDyn>.

The Operator struct can build from dyn AccessDyn. This way, I can construct the Operator in the recover_cache function, and then use list_with_metakey and concurrent_stat_in_list.

Yes, I think we should.


HIVE-25067: Add more tests to Iceberg partition pruning

What changes were proposed in this pull request?
More tests for Iceberg partition pruning
Why are the changes needed?
Better test coverage
Does this PR introduce any user-facing change?
No
How was this patch tested?
Query tests

This has been merged as 312763350d0e8df63ea78687ffcbf21e5f15ad66


Support inline parameter binding in manifest file

Currently, parameter binding can only be done in a deployment file.  It should be possible to do in-line parameter binding in the manifest, especially for values that are set by the developer and not intended to be changed.
One possible change is to add a value to the schema "inputs" key:
actions:
   myAction:
        inputs:
             myParam:
                  type: string
                  value: "Hello World!"
            anotherParam:
                  type: int
                  value: 1

However, we also allow simple name value pairs without types in the deployment file.  We could allow the same in the manifest without using the schema keys:
actions:
   myAction:
       parameters:
            myParam: "Hello World"
            anotherParam: 1

My preference is the second option.

This is supported in the 0.8.3 Packaging Spec. as the first "use case" for hello world: Manifest Files:
Manifest files
Example 1: No parameter descriptions
helloworld:
  version: 1.0
  license: Apache-2.0
  actions:
    hello:
      version: 1.0
      function: src/hello/hello.js
      inputs:
        personName: string
      outputs:
        greeting_string: string

I'd like a simpler approach where the version of the package applies to the entities so we don't need to explicitly assign a version number to actions, triggers, rules.

Inferred types would be good.  The inputs in the YAML file get converted into a JSON based request body underneath the covers before calling the OpenWhisk REST API so any inference we make explicit in the spec should follow the JSON type inference rules.

If we overload the inputs/outputs section to include values, I see two options.
Option 1: Preserve existing keys and add a value key
actions:
    hello:
      version: 1.0
      function: src/hello/hello.js
      inputs:
             name:
                  type: string
                  value: Paul

Option 2: Allow name value pairs as inputs
    hello:
      version: 1.0
      function: src/hello/hello.js
      inputs:
             name: Paul

Would we allow mixing and matching of the two? Or are they mutually exclusive.  One problem is not all inputs will take parameter values.  If at some point in the future we need to know all the inputs, what is the best way to handle that case?

merged


Add Object Subscripting to links

I'm working on integrating ios-deeplink-sdk into an app that uses many links.
I find my code cluttered with link.routeParameters[@"something"] and link.queryParameters[@"something"].
I added object subscripting to my local fork to make this code more expressive. Perhaps you'll enjoy it in your project.
This allows, for example:
self.router[@"/say/:title/:message"] = ^(DPLDeepLink *link) {
    [[[UIAlertView alloc] initWithTitle:link[@"title"]
                                message:link[@"message"]
                               delegate:nil
                      cancelButtonTitle:NSLocalizedString(@"OK", nil)
                      otherButtonTitles:nil] show];
};

Hey Aaron!
Thanks for the PR. Wes and I actually spent a while talking about this while building and couldn't find a way to object-subscript it without making it unintuitive in some situations.
E.g. How do you prioritize route vs query parameters?
In this case, we decided explicitness was probably the better way to go. I'll leave the P/R open incase there are alternative suggestions / opinions from the community. 😄
Thanks again - let us know how it goes!

I had the same concerns, however, I think the benefits of expressiveness outweigh the disadvantages of unintuitive behavior because:

Identical keys in both dictionaries is probably rare
When it happens, it's usually under developer control (the developer defines the key names)
The behavior is documented
If the documentation is ignored and there's a problem, the cause of the problem is easy to determine

Anyway, I see the other side of the coin, so no worries if you decide not to merge it in. Thanks for the great library. :)

@getaaron I'm actually coming around to this. However, I think it may make more sense to prioritize the route parameters over query though?
e.g.
myapp://dlc/products/123?blah=a&id=456

matching:
/products/:id

The route seems like the most important part.

@chrismaddern i am up for this feature :+1:
About prioritisation of route vs query parameters. I would say that if they are two same keys in both dictionaries the route one should have definitely higher priority and the query one should be ignored.


Can't specify version to update to, only the channel.

Please make it possible to specify which version of the software to upgrade / downgrade to via the update manager?
Or if I have missed it the documentation please point me in the right direction.
Great work by the by way! An excellent installer.

Hi there, the intended way to use UpdateManager is to not specify the version or the channel. The channel is auto-detected based on the package metadata when you build it with vpk pack. The version will always be the latest available, which might be lower than the current version if AllowVersionDowngrade is set.
What is your use-case for this? Always helpful to know, so I can consider these things when implementing new features.
I really don't recommend this, but you can implement your own logic by replicating what happens in CheckForUpdate. Since these are "internal" API's, they are likely to change frequently with updates to Velopack.
ILogger logger = null; // create your logger
var locator = VelopackLocator.GetDefault(logger);
var source = new SimpleWebSource("https://the.place/you-uploaded/your/releases");
var feed = await source.GetReleaseFeed(logger, locator.Channel, null, null);
foreach(var a in feed.Assets) {
    System.Console.WriteLine(a.FileName);
}

var targetRelease = feed.Assets.FirstOrDefault(a => a.Version == SemanticVersion.Parse("2.3.4"));
var updateInfo = new UpdateInfo(targetRelease, isDowngrade: false);

// can now carry on with usual update process...
var updateManager = new UpdateManager(source, logger: logger);
await updateManager.DownloadUpdatesAsync(updateInfo);
updateManager.ApplyUpdatesAndRestart(targetRelease);

Hey, thank you for getting back to me.
It's mainly as a fall back system. If a bug makes it to productions that is stopping people from completing a process but this process is known to be ok in an old version of the software. As a quick fix I could have the user downgraded to the specific version that I know works whilst I create a permanent solution for issue.

@caesay unfortunately it is not working as expected, or ever I missing something.
For example, I'm working with VeloWpfSample. One code line was changed:
_um = new UpdateManager(updateUrl, new UpdateOptions { AllowVersionDowngrade = true }, logger: Program.Log);
Also I removed latest version from releases.win.json. I'm able to download it... But it seems like the application is not updated ("Restart" button is not available and after the restart I still getting version .11 instead of expected .10).

Hi, i`m just trying to set the user witch version to update to, but i cant get it to work.
My use case is that we need to make different clients to update only to certain version and not newer than that.
For example: Client 1, which is currently on version 1.1.1, needs to be updated to version 1.1.2, but version 1.1.3 has already been released. However, this client should not update to this version.
I tried this solution

I really don't recommend this, but you can implement your own logic by replicating what happens in CheckForUpdate. Since these are "internal" API's, they are likely to change frequently with updates to Velopack.
ILogger logger = null; // create your logger
var locator = VelopackLocator.GetDefault(logger);
var source = new SimpleWebSource("https://the.place/you-uploaded/your/releases");
var feed = await source.GetReleaseFeed(logger, locator.Channel, null, null);
foreach(var a in feed.Assets) {
    System.Console.WriteLine(a.FileName);
}

var targetRelease = feed.Assets.FirstOrDefault(a => a.Version == SemanticVersion.Parse("2.3.4"));
var updateInfo = new UpdateInfo(targetRelease, isDowngrade: false);

// can now carry on with usual update process...
var updateManager = new UpdateManager(source, logger: logger);
await updateManager.DownloadUpdatesAsync(updateInfo);
updateManager.ApplyUpdatesAndRestart(targetRelease);

Also note that this would prevent delta updates from working. The delta strategy is calculated during CheckForUpdates and stored in the UpdateInfo object.

But i keep getting this error

Could not find asset called 'App-1.2.1072-full.nupkg' in GitHub Release 'Version 1.2.1073'.

The error should point you in the right direction... It's searching for asset App-1.2.1072-full.nupkg inside the github release Version 1.2.1073 which presumably does not have that asset. The GithubSource does not support downloading assets from anything other than the latest release currently. Feel free to write your own IUpdateSource and/or look at the source code for GithubSource.


[v1.2]Updated documentations for enhanced find feature

updated user guide and portfolio to enhanced find feature

Coverage remained the same at 92.144% when pulling e3c0b6669c107178be728031e5dc998ce4018d48 on ALim95:update_about_us into 504fa19869765c339557b7ee0aabf77689798cbf on CS2103AUG2017-T15-B4:master.

Coverage increased (+0.02%) to 92.162% when pulling 00ceed9e7b7490b84511e207bd21c17854336d95 on ALim95:update_about_us into 504fa19869765c339557b7ee0aabf77689798cbf on CS2103AUG2017-T15-B4:master.

Coverage increased (+0.02%) to 92.162% when pulling c28731e9696f8bf8952218dd3578afb64bb42ef0 on ALim95:update_about_us into 504fa19869765c339557b7ee0aabf77689798cbf on CS2103AUG2017-T15-B4:master.


Sundanese: Why Glyphs and Designspace?

We have both Glyphs and DS sources for Sundanese. Which are we using? We should only keep one.

@simoncozens thanks
This is wrong. Knowing the build process I using both but publish one randomly. Will check which one we should keep


Question: Rules on fields in collections

Not an issue, but a question:
Let's assume
class A
{
int a;
int b;
}
class B
{
List myList;
}
I'm trying to do the equivalent of:
foreach (var a in myList)
{
Validate (a.b);
}
If I do:
RuleForEach(_ => _.myList), I don't know how to get access to the members of A.

The usual way is to use RuleFor with SetCollectionValidator passing in an instance of a validator that can work with the child type instead of using RuleForEach.
RuleForEach is used for validating simple collections (eg of strings) where you want to apply a validator to the entire element.

Thanks a lot!


Update Python scripts to support power capping experiments

We need to add plots for node characterization to the plotter.  Also, we need functions in analysis or plotter to display the achieved power from the power_consumption integration test and the plots comparing the governor and the balancer.

This is WIP.  Diana started a script based on the integration tests that will print power statistics for the sockets and DRAM as calculated from the trace files.  I've been extending it a bit to print a data summary (rather than just the individual per-node data), as well as per region power data.
These are just tables for the moment.  No plots are being generated just yet.


Cannot find definition of local_n_nets

state.local_n_nets is used without definition. What's the meaning of it?
If train_nets_type=='known_init', why we want to build several networks with different initialization?
https://github.com/SsnL/dataset-distillation/blob/24e6958cf23bc98de20a7b2714fc76fa7d38b874/main.py#L90-L91
Thanks a lot!

Any Update?

It is set here https://github.com/SsnL/dataset-distillation/blob/24e6958cf23bc98de20a7b2714fc76fa7d38b874/base_options.py#L444
It means the number of training models in a single rank of distributed training. In case of no distributed training, it equals the total number of training models.
With known_init, n_nets controls the number of known initializations, i.e., whether you want to overfit for one model, or for multiple models.


wip: Attempt to solve some pinning errors

Motivation

https://github.com/sorbet/sorbet/issues/649
Test plan

See included automated tests.

We have a policy of testing changes to Sorbet against Stripe's codebase before
merging them. I've kicked off a test run for the current PR.  When the build
finishes, I'll share with you whether or how it failed. Thanks!
Stripe employees can see the build results here:
→ https://go/builds/bui_OYmclpDJZtEOWB
→ https://go/builds/bui_OYmcdpHBfXJQib
→ https://go/builds/bui_OYmc7E5jrreHFw


PLAT-10111: Revamping the BDK services API

Ticket
PLAT-10111
Description
Breaking changes:
ApplicationService#listAppEntitlements -> ApplicationService#listApplicationEntitlements
ApplicationService#updateAppEntitlements -> ApplicationService#updateApplicationEntitlements
DatafeedService -> DatafeedLoop
HealthService#v3HealthCheck -> HealthService#healthCheck
HealthService#v3ExtendedHealthCheck -> HealthService#healthCheckExtended
PresenceService#listPresencesOfAllUsers -> PresenceService#listPresences
SessionService#getSession(AuthSession) -> SessionService#getSession() (OBO enabled)
SignalService#subscribeSignal -> SignalService#subscribeUsersToSignal
SignalService#unsubscribeSignal -> SignalService#unsubscribeUsersFromSignal
UserService#listUsersBySearchQuery -> UserService#searchUsers
UserService#listAllUsersBySearchQuery -> UserService#searchAllUsers
UserService#unfollowUser(Long, List) -> UserService#unfollowUser(List, Long)
UserService#followUser(Long, List) -> UserService#followUser(List, Long)
UserService#addRoleToUser -> UserService#addRole
UserService#removeRoleFromUser -> UserService#removeRole
UserService#getAvatarFromUser -> UserService#getAvatar
UserService#updateAvatarOfUser -> UserService#updateAvatar
UserService#getDisclaimerAssignedToUser -> UserService#getDisclaimer
UserService#unAssignDisclaimerFromUser -> UserService#removeDisclaimer
UserService#assignDisclaimerToUser -> UserService#addDisclaimer
UserService#getDelegatesAssignedToUser -> UserService#getDelegates
UserService#updateDelegatesAssignedToUser -> UserService#updateDelegates
UserService#getFeatureEntitlementsOfUser -> UserService#getFeatureEntitlements
UserService#updateFeatureEntitlementsOfUser -> UserService#updateFeatureEntitlements
UserService#getStatusOfUser -> UserService#getStatus
UserService#updateStatusOfUser -> UserService#updateStatus
Dependencies
List the other pull requests that should be merged before/along this one.
Checklist

[x] Referenced a ticket in the PR title and in the corresponding section
[x] Filled properly the description and dependencies, if any
[x] Unit tests updated or added
[x] Javadoc added or updated
[ ] Updated the documentation in docs folder

Do we have any documentation to update for these changes?

I checked the github documentation of the BDK and it seems ok. Nothing to update there.

Should we plan to update the generator?

Yes I think


Xcode version bump and recommended project changes

Like #353 this results in a broken build; see: https://travis-ci.org/erikdoe/ocmock/builds/397119873?utm_source=github_status&utm_medium=notification


[question]: How did you compile esbuild with plugins support?

I am trying to use this plugin and write some other ones but I can't figure out how to compile esbuild with plugins support. Could you give me a hint? :)

To be honest I haven't gotten a full pipeline working yet. It sounds like Evan is still working on finalizing the plugin API so as that gets closer to being set I will try again.
In theory you can just checkout the plugins branch, run make platform-all (or make platform-<your_platform>) then copy the binary from the npm folder of the esbuild repo to esbuild/bin in node_modules of the project you want plugin support for. It won't survive across npm updates (or maybe even installing new packages?) but should be a start.

@ImedAdel I have updated the README with instructions and fixed an error that would have prevented you from running this plugin. Please give it a shot now!


Add resources to the Gradle all distribution

Add resources dir to source and transitive source, filter to just .java and .kt files in consumers of the source that aren't expecting resource files.
Fixes #26711

@bot-gradle test this

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build

@bot-gradle test this

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build

@bot-gradle test this

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build

@bot-gradle test this

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build

@bot-gradle test this

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build

Makes sense since our -all is advertised as containing docs and sources.
Can we make sure that whatever CI produces does not include unwanted files though?

What do you mean?  Which files are you concerned about?

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

I've triggered a build for you. Click here to see all failures if there's any.

Pre-tested commit build failed. Click here to see all build failures.

Your PR is queued. See the queue page for details.

I've triggered a build for you. Click here to see all failures if there's any.


broken in Edge

so, straight to the point - typing stuff in doesn't give suggestions ( dynamic colors thing works though ), pressing enter after typing doesn't do anything whatsoever, and the clock upon opening the startpage is completely missing, just a white page. Oh and the help screen is aligned vertically instead of, well, the way it is supposed to be

Doesn't work in Safari on a Mac at the moment either.

@lightrow @joshua-russell I just tested on edge & safari (latest versions) and everything seems to work with the latest changes. Feel free to create a new issue if a specific bug persists.


Release/custom fees

Try out this version of the Hiro Wallet - download extension builds.

Contents:

BTC custom fees
mask secret key
fix brc-20 decimals
psbt refactoring

PR to be merged, not rebased


Critical bug: duplicate heartbeats counted

I just came across a critical bug with the time aggregation logic. Apparently, reported total coding time is, in some cases, way too high. The reason for this is the following code block, combined with the fact that heartbeats can be duplicated:
https://github.com/muety/wakapi/blob/e7e5254673b45557f9e54c6372d4979089f244b9/services/duration.go#L81-L88
There is application-side deduplication logic in form of a heartbeat's Hash, however, it doesn't seem to work in many cases. Reasons for this might be the fact that, at least for MySQL, heartbeats' time column only has second-precision, so heartbeats that are considered distinct in Go code might become duplicates once inserted to the database, of their respective time difference is below a second. There might be other reasons for duplicate heartbeats, I need to investigate further.
However, the main reason for this bug is the above code block, for which I unfortunately can't remember why I even put it there in the first place. Logic should be fine by ditching that.
After this fix, expect your long-term coding time to drop significantly. I apologize!!

Apparently, default precision for fractional seconds inside time columns in MySQL is 0, deviating from SQL standard's default value of 6. See https://dev.mysql.com/doc/refman/8.0/en/fractional-seconds.html.


why mAP is too high?

I only use VOC2007 data train the SSD, but after 30000 first traing, the mAP on test dataset is 70%, and when fine-tuning all all parameters with 70000 steps, the mAP on test dataset is 81%, it's too high, and I don't know why?
Sorry, I don't find the evalution process is done on the train and test dataset, the 81% is the train result.

If I understood correctly, you were saying that training mAP you got was 0.81. In that case, that's expected with current implementation. Do share with me if you find ways to push test mAP beyond current 0.65 :).

@lonlonago @LevinJ Hi, all! what the best mAP did you get?

Hi @wuguangbin1230 , the best score I got is 0.65.


Add JDBC Assist for Presto

Add JDBC Assist for Presto

Fixes #799

+1 for above and would it be possible to add a commented example in the hue.ini like https://github.com/cloudera/hue/blob/master/desktop/conf.dist/hue.ini#L839 and also in https://github.com/cloudera/hue/blob/master/desktop/conf/pseudo-distributed.ini.tmpl (that way it documents it better until we have proper documentation)

Could also add a note to https://github.com/cloudera/hue/blob/master/docs/user-guide/user-guide.md#presto in another PR if you want

Thank you for a review! Comments addressed.

Thanks a lot!

Thanks!


Additional docs for the README

Answering some common questions from both our internal slack and from twitter

:tada: This PR is included in version 1.1.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Install ChIPseeker on Test (second attempt)

The first attempt failed when rsyncing changes to the stratum 0, because the Travis console log size grew too large and the build was terminated.

$ cvmfs_server tag test.galaxyproject.org | grep bd3496b
tools-bd3496b                                          │   500839 │ trunk   │ 5 Jun 2019 11:21:08  │        │ Automated tool install for commit bd3496b4c196360a3d93a4410318df0453b64f50

Installed on Test and available immediately upon CVMFS propagation without a restart! Tool panel indexing appears to have worked as well.
It went into the correct section name but the section ID (chip-seq) doesn't match the existing ChIP-seq section (chip_seq). I'm going to fix shed_tool_conf.xml in CVMFS by hand and for the time being, tool installs/updates should use tool_panel_section_id instead of tool_panel_section_label.
Further details on the issue and proposed solutions for usegalaxy.* are in #9.