Add option to disable auto attach

Hi
Thank you for wonderful component!
Could you please add an option to disable auto attach functionality? Just to have draggable widget.
Thank you.

Hi, if you just want to use draggable components, you can implement it directly with Draggable.


Update Set-CsTeamsMeetingPolicy.md

Adding new meetingPolicy setting AllowNetworkConfigurationSettingsLookup to help determines whether network configuration setting lookup can be made for not enterprise voice enabled users. It will be used for enabling network roaming policy

Docs Build status updates of commit bc84f7d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

skype/skype-ps/skype/Set-CsTeamsMeetingPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

@dariomws @tseward @get-itips

@JohanFreelancer9

Docs Build status updates of commit 91a7f64:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

skype/skype-ps/skype/Set-CsTeamsMeetingPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 5e8ab9b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

skype/skype-ps/skype/Set-CsTeamsMeetingPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

@tseward please review

@tiburd Please merge this PR. Thanks

Docs Build status updates of commit 6185a58:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

skype/skype-ps/skype/Set-CsTeamsMeetingPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Set --lib es6, and use Map/Set methods where possible.

PR checklist

[X] Addresses an existing issue: #1971
[ ] New feature, bugfix, or enhancement

[ ] Includes tests

[ ] Documentation update
[X] Enable CircleCI for your fork (https://circleci.com/add-projects)

What changes did you make?
Set the "lib" option to ["es6", "dom"] ("dom" typings used by underscore).
EDIT: Removed underscore; it's only used by 1 function, camelize, that's a one-liner. Their source, mine.
Used Map and Set in cases where their absence appeared to have been compensated for.
Is there anything you'd like reviewers to focus on?
It's more efficient to lookup in a Map only once, so avoid this pattern:
if (map.has(key)) {
	use(map.get(key))
}

Prefer:
const got = map.get(key);
if (got !== undefined) {
	use(got);
}

this is an awesome PR, thanks @andy-hanson


Unable to Install Swift 5.9 on ubuntu-latest

Describe the bug
I'm unable to install Swift 5.9 in an ubuntu-latest GitHub Actions workflow. See here.
Workflow configuration (please complete the following information):

Action version (uses): swift-actions/setup-swift@3aed395c5397f62deb91d8fe7af1418a9ae4d16f
Platform (runs-on): ubuntu-latest
Swift version (swift-version): 5.9

Seeing the same issue

I'm having the same issue with swift 5.10 on ubuntu 24.04
Error log:
Error: Unexpected error, unable to continue. Please report at https://github.com/swift-actions/setup-swift/issues
Version "24.04" of Ubuntu is not supported
Stacktrace:
Error: Version "24.04" of Ubuntu is not supported
    at Object.getSystem (/home/runner/work/_actions/redsun82/setup-swift/b2b6f77ab14f6a9b136b5[20](https://github.com/modiimedia/arri/actions/runs/12484586440/job/34842184987?pr=124#step:8:21)dc53ec8eca27d2b992/dist/index.js:427:15)
    at async run (/home/runner/work/_actions/redsun82/setup-swift/b2b6f77ab14f6a9b136b520dc53ec8eca27d2b992/dist/index.js:335:24)


On browser setting window.OTEL_SERVICE_NAME does not populate service.name

Setting window.OTEL_SERVICE_NAME should populate service.name. Or please document the correct way to populate  service.name in browser request (Not node)

@shanshanzhu What language is this related to?

@theletterf Javascript run on browser (not nodeJS)

@open-telemetry/javascript-approvers Could you have a look?
@shanshanzhu Could you provide the specific doc where this appears?

(Warning: I don't know the Web side of the OTel code very well, so I might get this wrong.)
The automatic handling of OTEL_SERVICE_NAME and some other environment variables is specified only for an OTel "SDK". There currently is not a "WebSDK" for web JS usage -- at least not yet -- so there is no convenient usage that will automatically pick up the "service name".
That means you have to do it manually.
Starting from https://opentelemetry.io/docs/languages/js/getting-started/browser/
the OTel setup code for browser starts with something like:
// ...
import { WebTracerProvider } from '@opentelemetry/sdk-trace-web';

const provider = new WebTracerProvider();
// ...

That WebTracerProvider takes a TracerConfig: https://github.com/open-telemetry/opentelemetry-js/blob/14d086a234416080a111aae7d45682f08b80d92e/packages/opentelemetry-sdk-trace-web/src/WebTracerProvider.ts#L32-L37
TracerConfig takes a resource property: https://github.com/open-telemetry/opentelemetry-js/blob/14d086a234416080a111aae7d45682f08b80d92e/packages/opentelemetry-sdk-trace-base/src/types.ts#L38
The "service name" for OpenTelemetry data is defined in a "resource" object.
Without a convenient "SDK" interface, the @opentelemetry/resources pacakge provides the interfaces to create a resource.
https://github.com/open-telemetry/opentelemetry-js/tree/main/packages/opentelemetry-resources/#readme
Option 1 is to manually create a resource:
// ...
import { WebTracerProvider } from '@opentelemetry/sdk-trace-web';
import { Resource } from '@opentelemetry/resources';

const resource = new Resource({
    'service.name': 'my-service',
});
const provider = new WebTracerProvider({ resource });
// ...

Option 2 is to use the (not documented in its README) detectResourcesSync() and the envDetectorSync export which reads the "environment variables". Something like this (untested):
import { envDetectorSync, detectResourcesSync } from '@opentelemetry/resources';

const resource = detectResourcesSync({detectors: [envDetectorSync]});
// ...

I put "environment variables" in quotes, because the concept isn't really a thing in a browser. However, for now at least, the OTel JS browser code supports reading OTEL_... properties from the globalThis object in the browser (which is the same as the "window" object, I believe).

Yes, this should definitely be documented. I feel a little out of my depth tackling the Web JS docs.
I suppose for a start we could add some reference docs to https://github.com/open-telemetry/opentelemetry-js/blob/main/packages/opentelemetry-sdk-trace-web/README.md mentioning use of the resource to set service name.
Likewise the https://opentelemetry.io/docs/languages/js/getting-started/browser/ page should mention setting service name. Setting service name is so important to using OTel data meaningfully that I feel like I may be missing something in these docs or in the code defaults.

@trentm who would be the best person from the JS SIG to help with that?

@open-telemetry/javascript-approvers PTAL, is this something we can fix?

@svrnm - sorry just saw this:
If I had to recommend a way, I'd recommend Option 1 that @trentm mentioned, doing it manually:
// ...
import { WebTracerProvider } from '@opentelemetry/sdk-trace-web';
import { Resource } from '@opentelemetry/resources';

const resource = new Resource({
    'service.name': 'my-service',
});
const provider = new WebTracerProvider({ resource });
// ...

I talked to the other JS SIG Maintainers recently, and we concluded that the whole window.ENV_VAR_HERE does not really make too much sense and is quite confusing. We concluded that we want to remove that feature at some point as it just offers a second way to do things that has no benefits over the conventional approach from @trentm's Option 1.

thanks, so the best way to solve this is adding these instructions to the getting started guide I guess?

so the best way to solve this is adding these instructions to the getting started guide I guess?

Yes, I think so.
And then, I guess https://opentelemetry.io/docs/languages/js/resources/#adding-resources-with-environment-variables should be made clear that it is talking about Node.js and not JS in the browser.


Is this an evolution to vast-videojs-vpaid? Is there a simple example?

As per the title, is this project an evolution to your previous videojs-vast-vpaid library or is that one still being maintained?
Also is there any simple code examples on how to use this library?

Do you understood how to run it?


add DATEFORMAT as parameter to copy function

It would be helpful if the copy function also would have a DATEFORMAT parameter so that dates in for instance m/d/Y format would be automatically converted to yyyy-mm-dd and imported into your date columns. This would avoid to pre-process the csv file first.
example: copy sales from 'sales2020.csv' (DATEFORMAT '%m/%d/%Y');

I have implemented this now in the strftime branch, following the specifiers used in Python (which I believe are the same ones used in C?).

Merged into master, see here for the documentation.


Add bottom_param option

Hello @himoto !
Here is my idea for the bottom_param option. It is a simple boolean argument, where False provides zero for the bottom parameter, as you described.
A raise catch is also included, which clarifies a potential error that the user might experience (I experienced it) depending upon the applied dataset.

Hello @himoto !
Yes, your edit is correct. Thank you for reviewing it :)
Andrew

@freiburgermsu, I have fixed the code above but we still need to resolve an IndexError

FAILED test/test_fitting.py::test_fitting - IndexError: tuple index out of range

Hello @himoto!
I resolved the bug in my latest push. The curve_fit function was essentially interpreting the bottom_param argument as being one of the parameters that must be calibrated. The location of redefining the bottom parameter was relocated downstream of this process to not interfere with it.
Thank you :)
Andrew

Hi @freiburgermsu, thanks for resolving the issue.
If I understand your code correctly, what you are trying to do is to assign 0 to bottom parameter after parameter fitting.
# here
if not bottom_param:
    popt[1] = 0

This does not look nice to me because there might be more appropriate parameter values for top, nH, or ec50 when bottom is fixed to 0.
In my opinion, the bottom parameter must be fixed to 0 during regression when bottom_param=True.
Below is my idea. Currently, param[1] is assigned to bottom even when bottom_param=True.
def _equation(self, x: np.ndarray, *params) -> np.ndarray:
    self.top = params[0]
    self.bottom = params[1]
    self.ec50 = params[2]
    self.nH = params[3]

    return self.bottom + (self.top - self.bottom) * x ** self.nH / (
        self.ec50 ** self.nH + x ** self.nH
    )

I think this should be the following:
def _equation(self, x: np.ndarray, bottom_param: bool, *params) -> np.ndarray:
    self.top = params[0]
    self.bottom = params[1] if bottom_param else 0
    self.ec50 = params[2]
    self.nH = params[3]

    return self.bottom + (self.top - self.bottom) * x ** self.nH / (
        self.ec50 ** self.nH + x ** self.nH
    )

Then, the estimated parameter values are based on the assumption that bottom=0.
I would be happy if you could give me comments on this plan.
Many thanks,
Hiroaki

@himoto This may cause the same IndexError as before, since the error was originally caused by similar logic
self.bottom = 0
if bottom_param:
   self.bottom = params[1]

to what you propose. Essentially, the additional bottom_param argument in the _equation() function is incorrectly interpreted by Python to be the first entry of the params arguments, and thus an IndexError is launched at self.nH = params[3], since len(params)=3 at this point instead of 4.
The solution may be to simply define a second _equation_bottom() function for where the user desires the bottom parameter, and then remove the bottom parameter from the _equation() function where the user does not want the bottom parameter. The bottom_param argument would be passed through the _get_params() function, and then the proper equation function would be executed to acquire the proper set of parameters.

@freiburgermsu, thanks for the detailed explanation, now I understand the exact cause.
The arguments of _equation(...) must be (self, x: np.ndarray, *params) and we must not add bottom_param here.

The solution may be to simply define a second _equation_bottom() function for where the user desires the bottom parameter, and then remove the bottom parameter from the _equation() function where the user does not want the bottom parameter.

Yes, this can resolve the issue, but what about moving bottom_param argument to __init__()?
Then, we can call bottom_param via self and do not need to create another equation function:
def _equation(self, x: np.ndarray, *params) -> np.ndarray:
    self.top = params[0]
    self.bottom = params[1] if self.bottom_param else 0
    self.ec50 = params[2]
    self.nH = params[3]

    return self.bottom + (self.top - self.bottom) * x ** self.nH / (
        self.ec50 ** self.nH + x ** self.nH
    )

@himoto Yes! This is a brilliant work-around.

@freiburgermsu thank you for your comments and updates!


Problem with RPM Repo listing page?

A problem was reported via Gitter chat that an RPM was not able to be located. The RPMs are being produced and synced to the S3 repo but not showing up on the release listing on the page. The location of that listing code is https://github.com/ngageoint/geowave/blob/master/docs/content/packages.html#L165
@larpar 14:23
@rfecher : We are unable to locate geowave-core.201605201930.noarch.rpm. Could you please point us to that resource?
@rfecher 16:14
@larpar I'm not entirely sure, but @spohnan is looking into it now
larpar @larpar 16:22
@spohnan : BTW, this is for the release, not a SNAPSHOT.
Andy Spohn @spohnan 17:33
Thank you for pointing out the problem. I'm guessing you're looking at the repo listing page and noticed that artifact is missing from the list - http://ngageoint.github.io/geowave/packages.html
Tracing through our build pipeline the rpm seems to have been staged to the S3 bucket I'm going to guess there's something up with my home-grown bucket listing script at the moment. This is not the solution to the problem you're reporting but in answer to the location of that artifact the URL is https://s3.amazonaws.com/geowave-rpms/release/noarch/geowave-core.201605201930.noarch.rpm

Cause of missing artifacts: IsTruncated Listing is true
As seen in the screenshot below there is a limitation on listing an Amazon S3 bucket of 1000 entries which we've exceeded for the total amount of items in both the dev and release repos (they live in the same bucket)

A number of different fix ideas:

Lower the number of development builds we keep around which is currently 5 copies of each unique artifact. This would buy some time and would be quick but isn't a long term fix

(Internal repo link) https://git.geointapps.org/protoglobe/rpm/blob/master/admin-scripts/cull-rpms.sh

Create a snazzier listing page that has logic to paginate
Remove the listing page altogether

I implemented some short term fixes to get this working again but this is just kicking the can down the road a bit and an actual long term solution should be discussed.

Lowered the MAX_COPIES limit from 5 to 3
Manually removed all of the development rpms from 0.8.9 and 0.9.0. This is not done by the  repo maintenance job (internal link) http://jenkins-ui.ec2.internal:8080/jenkins/job/cull-jenkins-latest-rpms-repo/ so after a few more releases this will start to accumulate again
With these two steps the number of keys is the bucket is back down to ~350

So while the specific problem reported is now "working" again I'm going to leave this ticket open for discussion as the existing process needs some attention to ensure it doesn't need manual care and feeding every 6 months.


Add User Profile screen

The app should show similar information to what you'd see on a user's profile page on the website. We would use the profile screen in two locations: viewing yourself, viewing others.
Namely:

Followers / Following count
Social media buttons
Team Role status
Report Button
User profile content

Most of this is done in #17 -- We can open new issues for other specific features.


[Integration] PullApprove

Integration with PullApprove
 The specification repo should be added to Pull Approve HERE by a repo admin (@dzahariev) before merging.
The proposed configuration constitutes that any pull request to the specification repo can only be merged if at least 4 people from the core-team group approve it.
Check the .pullapprove.yml file for more configuration settings.

Probably we should discuss in the working group what the merge policy should be


Ability to Increase All UI Size

Is your feature request related to a problem? Please describe.
In my game, the UI of Panku is really small, I can not read it.
Describe the solution you'd like
I hope there's a setting to adjust  UI scale.

Since OP didn't provide a screenshot, I will.

I'm pretty sure this is a PPI setting or something, but I'm not sure where to find it. Which is how I wound up here.

I'll prioritize fixing UI issues once I have enough free time.

Well, I'll reiterate what many have said. You'd made an awesome plugin. It's exactly what I would've wanted. Thanks for sharing with the world!

Well, I'll reiterate what many have said. You'd made an awesome plugin. It's exactly what I would've wanted. Thanks for sharing with the world!

Thank you! Working on UI related issues now

Well, I'll reiterate what many have said. You'd made an awesome plugin. It's exactly what I would've wanted. Thanks for sharing with the world!

Thank you! It's people likecyou keep me motivated!

Weyooo thanks for tackling this @Ark2000 - stoked to get to actually use the console and add a bunch of debugging commands to it 😎 :shipit:

Scale is working much better for me. I am having to change all my resources because apparently I scaled them wrong for my DPI, but at least everything is manageable now.
This console has been a lifesaver for testing specific scenarios.


Check for merge conflicts before creating SRPM

In addition to merging pull request changes into base branch before
running SRPM build, check for potential merge conflicts and set neutral
status, if detected.
Related to packit/packit#1344

TBD

Bumping priority of refactoring fixtures…

packit/packit#1344 became a hard dependency… introduced PackitMergeException

recheck

recheck

tests are failing because of cached test image with version of packit that doesn't include PackitMergeException, merging manually


modman issue

It looks like there are some email templates in modman that reference files that don't exist.
Am I missing something?
 [ErrorException]
  Source [..]snip[..]/vendor/amzn/amazon-payments-magento-plugin/app/locale/en_US/template/email/amazon_payments_async_decline.html does not exist

Sorry, closing.  Missed #275 merge request.  Any chance of that being merged in any time soon?


This content show up after installed and try to play something on Tidal

Hi There. I am using Macbook Pro with M1 here.
I installed the file completely fine and gave the permission of access.
When I play something on Tidal then it shows me this.
I am so sorry.  This is as far as I can go.
Can you tell me what should i fix?
Thanks for your time.
Uncaught Exception:
SyntaxError: Unexpected token
in JSON at position 7
at JSON.parse ()
at Request._callback (/Applications/tidal-lyrics.app/Contents/Resources/app/src/index.js:377:34)
at Request.self.callback (/Applications/tidal-lyrics.app/Contents/Resources/app/node_modules/request/request.js:185:22)
at Request.emit (events.js:310:20)
at Request. (/Applications/tidal-lyrics.app/Contents/Resources/app/node_modules/request/request.js:1154:10)
at Request.emit (events.js:310:20)

Hello.
Can you provide the log file for when this happened? You can open it from the about menu and then copy paste it ere. If you can't access the about menu, it's located at ~/Library/Logs/tidal-lyrics (~ is your user folder; the Library folder is hidden by default, to temporarily unhide it you can press Command + Shift + . (full stop/period)). After going in that folder, you should see an app.txt file. Just keep in mind that the log resets every time you open the app.
If you have any trouble with all that please let me know.

2021JL26Tidal-lyrics-testrun.log
I shut it down earlier then it works for a little while this time. then it went back crazy again.
and at the same time i am also playing offline mode on my phone with musixmatch running.
so there was only 1 song doesn't have lyric file.
i am trying with japanese songs so the code is more clear for you to read.
btw. i found that i pressed "Toggle Full Screen" then the text changed to "Tidal is not running"
is there anything else you like me to try with??? just ask

Okay. About the fullscreen thing I don't think there is any workaround. The program takes the song data from the "Window Title" (which you don't really see on MacOS but it exists) and I suppose the fullscreen on Mac doesn't have a title (on Windows it works fine. I'll work on a better error handling right now (to deal with the initial problem) and then see what's going on with the fullscreen on Mac. Thanks for the log

I think I fixed the problem but my MacOS Virtual Machine is not working right now so I can't get any MacOS builds. I'll send you a test build here to test when I manage to fix it

Hello, sorry for the delay.
Here is the test build I was talking about. Please test it and see if the problem still persists. I don't know what exactly is causing the problem as I didn't manage to reproduce it, but it should at least give an error instead of crashing. Thanks
https://www.dropbox.com/s/nsu0wywmri54ab8/tidal-lyrics testbuild.dmg?dl=0


prune fails when input is datetime

clouddrift.analysis.prune returns an error if the input is a datetime object and rowsize an ndarray. Concatenation of the outputs seems the issue. See example in screenshot where ds is a xarray.DataArray for which the decode_times=True is used for opening.


Migrate the whole code base to TypeScript

For anyone want to help the migration, I've setup the js + ts mixed project structure successfully at #42


exceeds grid limits error when sheet name ends with a digit

@srobb1 heroically tracked down this bizarre bug: When a sheet (tab) name ends with a digit, we get a exceeds grid limits error:
I am trying to use COGS for the first time. I figured out the credentials!!!! Now I am getting errors when I pull that say I am exceeding limits, but the sheet is not bigger than the limits that the error states. But in this particular spreadsheet I have lots and lots of tabs. Any thoughts or ideas?
Traceback (most recent call last):
  File "/home/smr/anaconda3/envs/cogs/bin/cogs", line 8, in <module>
    sys.exit(main())
  File "/home/smr/anaconda3/envs/cogs/lib/python3.9/site-packages/cogs/cli.py", line 249, in main
    args.func(args)
  File "/home/smr/anaconda3/envs/cogs/lib/python3.9/site-packages/cogs/cli.py", line 408, in run_pull
    fetch(verbose=args.verbose)
  File "/home/smr/anaconda3/envs/cogs/lib/python3.9/site-packages/cogs/fetch.py", line 320, in fetch
    cells = get_cell_data(cogs_dir, sheet)
  File "/home/smr/anaconda3/envs/cogs/lib/python3.9/site-packages/cogs/fetch.py", line 113, in get_cell_data
    resp = request.execute()
  File "/home/smr/anaconda3/envs/cogs/lib/python3.9/site-packages/googleapiclient/_helpers.py", line 134, in positional_wrapper
    return wrapped(*args, **kwargs)
  File "/home/smr/anaconda3/envs/cogs/lib/python3.9/site-packages/googleapiclient/http.py", line 935, in execute
    raise HttpError(resp, content, uri=self.uri)
googleapiclient.errors.HttpError: <HttpError 400 when requesting https://sheets.googleapis.com/v4/spreadsheets/1wd-TXKNa7ohufoI_odjNC-xX8VJ9qUk_BDIGxkqHS1Y?ranges=TSA1&fields=sheets%28data%28rowData%28values%28%2A%29%29%29%29&alt=json returned "Range (All_public_bat_genome!TSA1) exceeds grid limits. Max rows: 1015, max columns: 33". Details: "Range (All_public_bat_genome!TSA1) exceeds grid limits. Max rows: 1015, max columns: 33">

This points to fetch.py 113 where we're calling the official Google Sheets library. I think that if the name of the sheet ending with a digit is quoted then this will work, but I'm annoyed that they don't do the quoting. We should handle it, either by quoting ourselves or warning loudly about this limitation.

This bug is now officially the front runner for Misleading Error Message of the Year.

I'm having some trouble recreating this error. I tried two different things, first adding a sheet with a digit at the end:
$ cogs add test1.tsv -t Test1
$ cogs push

Then, I added a new "Test2" sheet remotely, and pulled it (after adding the ignored sheet):
$ cogs add Test2
$ cogs pull

Looking at the error message, it looks like the problem sheet is "All_public_bat_genome" and it's trying to get column TSA1 which is outside of the range. Maybe the sheet got formatted weird and for some reason Sheets thinks there's something in that cell?

"All_public_bat_genome" was the name of the file.
We also found that COGS worked as expected with a single sheet ending with a digit, but failed when we added a second sheet ending with a digit. Can you try that please?

All_public_bat_genomes was the name of the first sheet in that spreadsheet. I started with an existing sheet that I “connect” to. There needs to be more than one sheet. If I just had one sheet with TSA1, it was okay, as soon as I added another, like ACA1 remotely and pulled, I got the error. I removed all the 1s and it worked. I can share a test spreadsheet with you if you like.

Yes, if you could share a sheet with me that would be great! My email is
@.***
On Mon, May 31, 2021 at 10:24 AM Sofia Robb @.***>
wrote:

All_public_bat_genomes was the name of the first sheet in that
spreadsheet. I started with an existing sheet that I “connect” to. There
needs to be more than one sheet. If I just had one sheet with TSA1, it was
okay, as soon as I added another, like ACA1 remotely and pulled, I got the
error. I removed all the 1s and it worked. I can share a test spreadsheet
with you if you like.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/ontodev/cogs/issues/123#issuecomment-851607082, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AD7ZMRWHOR6A2IXK2J4AX7TTQPA4BANCNFSM45VDWNZA
.

Can you send your email in the obo slack? It is all stared out here. I sent you a direct message in slack.


[develop < T0046-GA] Add order by descending

Description
I added order_by_desc() method in the query builder, so that the user doesn't have to use DESC word as a part of order_by() method.
Pull request type

[x] Feature

######################################
Reviewer checklist (the reviewer checks this part)

[ ] Core feature implementation
[ ] Tests
[ ] Code documentation
[ ] Documentation on memgraph/docs

######################################

Do you plan on making more comments in the code?

@brunos252  I think the plan is to document everything better here: https://github.com/memgraph/gqlalchemy/pull/116


Changes in the picture, how should its coordinate file change?

Hi,A picture corresponds to a txt document with the coordinates of a target, such as antelope.jpg and antelope.txt (antelope 12 45 368 797)。Among them,'antelope' representative category name,' 12 45 368 797' correspond ‘xmin,ymin,xmax,ymax’.

antelope.txt (antelope 12 45 368 797) means the bounding box.

I think you should make some code to implement your target.


NFT number pre reveal not showed

Hi
I post this few days ago. When I have the NFTs in not reveal, I can see the image, all ok, but all my NFTs are same, I mean that the NFTs don't show their identification number #1, #2, #3... I know that the hidden.json is a metadata same for all the collection. How can I do to resolve this? Because all collections in opensea when they are in not reveal, they show their ID.

As far I have seen, the collections not revealed have a hidden metadata specifically for every NFT, using all the same image/gif of course, but to this way you can write every name corresponding to every NFT. Is it possible with this script? I mean, instead of put in the metadata URI a single .json, if my collection has 5k nft so 5k .json for the hidden metadata.

Hi @Salro1g, I'm gonna be quick but feel free to ask on the HashLips' discord server for this, there you might find a dev to hire in order to get what you need.

Because all collections in opensea when they are in not reveal, they show their ID.

This is not true actually, many collections simply have a single hidden file for every token. It's just up to the author to decide what to do here, there is no standard.
To have dedicated names for each token, you would have to change the smart contract in order to return a different token URI for each ID and then you have to generate a full set of metadata files and upload them to IPFS. As I said, I can't provide you with and example for this, but if you can't do that on your own, you may find a dev on the discord server.


TypeError: Cannot read property 'getFrameByName' of null

The game isn't working when trying to build and run it. In the browser's console there's an error with the content TypeError: Cannot read property 'getFrameByName' of null and the game shows basically a black screen. I can see only in the bottom of Phaser Debug (WebGL): with its numbers.
phaser.js:48262 TypeError: Cannot read property 'getFrameByName' of null
    at prepareFontData (ReturnOfGanon.ts:91)
    at new ReturnOfGanon (ReturnOfGanon.ts:16)
    at new Dialog (Dialog.ts:33)
    at Intro._createLoreGroup (Intro.ts:383)
    at Intro.create (Intro.ts:64)
    at Phaser.StateManager.loadComplete (phaser.js:6566)
    at Phaser.Loader.finishedLoading (phaser.js:47818)
    at Phaser.Loader.processLoadQueue (phaser.js:47775)
    at Phaser.Loader.asyncComplete (phaser.js:47848)
    at Phaser.Loader.Loader_jsonLoadComplete [as jsonLoadComplete] (browser.js:191)

Yeah, don't think this has been in a building state for a while as phaser has updated and this hasn't.

Hi.
I have studied your project for more than a month.I tried to run the code but it didn't work.


[BUG] attachdetach-controller  AttachVolume.Attach failed for volume "pvc-xxx-xxx“ : rpc error:

Describe the bug (🐛 if you encounter this issue)
In my use case, I'm creating ReadWriteMany Persistent Volume Claims (PVCs)  as data volume. Multiple task pods in Airflow are simultaneously mounting this single volume as a data volume. Airflow tasks generally running within a few minutes, resulting in frequent mount/unmount operations on the ReadWriteMany PVC. Additionally, there are times when multiple pods mount this volume simultaneously. After running for a period of time, we've encountered the following issues:

Environment

Longhorn version: v1.5.0
K8s: v1.26.6

@ming12713 Can u provide the support bundle? Thanks

@ming12713 Can u provide the support bundle? Thanks

ok

volume status is not ready
volume replica status is running

i also found another RWX pvc on replica is stoped ,but /mnt/disk1 which physical ssd health is ok ,not broken

share volume errors

@ming12713 Can u provide the support bundle? Thanks

@innobead  upload sucess ? i can't get progrocess from webui, bundle manager show progress 100%

cc @c3y1huang

@innobead upload sucess ? i can't get progress from webui, bundle manager pod show progress 100%

@ming12713 Do you mean that you're unable to download the support bundle using the UI?

@innobead upload sucess ? i can't get progress from webui, bundle manager pod show progress 100%

@ming12713 Do you mean that you're unable to download the support bundle using the UI?

@c3y1huang always blocking 40% on webui ,but bundle manager pods has compelete , when i flush webui and try bundle again, output error with increase limit or clean bundle task

@c3y1huang always blocking 40% on webui ,but bundle manager pods has compelete , when i flush webui and try bundle again, output error with increase limit or clean bundle task

Seems like you might be running into https://github.com/longhorn/longhorn/issues/6256.
Please try the below steps to resolve this:

Manually delete the SupportBundle custom resource (CR) by running kubectl -n longhorn-system delete supportbundle <name>.
Update the support bundle manager image to longhornio/support-bundle-kit:v0.0.25.

@c3y1huang thanks ，downloading logs from the web UI is now ok .
Delete the previous bundle task，and try again is ok

@innobead The log files are large, and some of the context  is sensitive. Do you have an FTP or email where I can send them directly to you?

You can send it to<EMAIL_ADDRESS>

You can send it to<EMAIL_ADDRESS>
log has been send thanks;

@innobead
I have two volumes, "airflow-data" and "spark-data," with accessModes set to "rwx."

I think you hit the issue https://github.com/longhorn/longhorn/issues/6287.
You can upgrade to v1.5.1 and check if the issue can be resolved.

I think you hit the issue #6287. You can upgrade to v1.5.1 and check if the issue can be resolved.

@derekbit thanks , i will try v1.5.1


Question on loading keys into a provider

I see two ways of getting an external/opaque key into a provider's keydata and thus into an EVP_PKEY object:
(i)  use the store interface which interacts with keymgmt_load
(ii) use a custom set of params and invoke keymgmt_import through EVP_PKEY_fromdata
Though it seems like the design intent in OpenSSL is to use store for "loading", and import for generating key from fundamental data like n, e, d, I have a situation where the use of store feels and looks awkward. So I use (ii), and the question is whether that's a bad approach. The docs do not seem anticipate such a usage. This is a built-in provider.
The  application has direct access to the certificate, and  signing of messages/digest is done by passing it to a process that also handles user interface, connected via a socket. The application is agnostic about the backend key storage, so there is no store that it "loads" from -- one can cook up a URI for key loading with pointers to an opaque handle and the public key pickled into a string.  Very awkward.
Instead,  I can  define a set of OSSL_PARAMS like: {.key="pubkey", .data_type=octet_ptr, data=pubkey}, {.key="handle", data_type=octet_ptr, data=handle} etc., plus a parameter to indicate that this is custom data, pass it to EVP_PKEY_fromdata and parse the params when it arrives in keymgmt_import.
Is there any reason to shy away from this approach and stick to loading via a "uri" and store interface, even if that looks even more fragile?

Personally, I don't see a problem with loading a key via import. Import is intended exactly for the situation where all the various parameters are coming in as individual fields. Store is generally intended for scenarios where you are loading from some backend storage mechanism. If you don't have such a thing then no need to artificially invent something. Of course the advantage of store is that you can use the URI for things like the command line apps - which you can't really do for the import mechanism. But maybe that isn't important to you and your application.


gulp clean should also remove .module.scss.d.ts files

css-modules-typescript-loader generates .module.scss.d.ts which needs to be deleted on gulp clean command in case if we need a fresh source state.

Implemented in<EMAIL_ADDRESS>& 1.11.4


Implement collision and separation events (a.k.a. Actions).

As observed in the OpenTibia.Server/Data/dat/moveuse.dat file:

implemented in #30


TomCat configuration engine-rest api

My team and I have been developing a front end to run Camunda rest api tasks through. I have setup the keycloak plugin and know I need to modify the authentication filter to accept a token from the front end (in the api call header) to process the request. I have added the following to /camunda/webapps/engine-rest/WEB-INF/web.xml
<!-- KeyCloak OpenID Connect Filter
  <filter>
    <filter-name>KeyCloak OpenID Connect Filter</filter-name>
    <filter-class>org.keycloak.adapters.servlet.KeycloakOIDCFilter</filter-class>
   <async-supported>true</async-supported>
     <init-param>
            <param-name>keycloak.config.file</param-name>
            <param-value>/camunda/config/keycloak.json</param-value>
    </init-param>
 </filter>
  <filter-mapping>
    <filter-name>KeyCloak OpenID Connect Filter</filter-name>
    <url-pattern>/*</url-pattern>
    <dispatcher>REQUEST</dispatcher>
  </filter-mapping> -->

<!-- Camunda Keycloak Authentication Filter
  <filter>
    <filter-name>camunda-auth</filter-name>
    <filter-class>org.camunda.community.auth.keycloak.filter.KeycloakAuthenticationFilter</filter-class>
    <async-supported>true</async-supported>
  </filter>
  <filter-mapping>
    <filter-name>camunda-auth</filter-name>
    <url-pattern>/*</url-pattern>
    <dispatcher>REQUEST</dispatcher>
  </filter-mapping> -->

keycloak.json
{
    "realm": "camunda-id",
    "auth-server-url": "project-flow_jboss.keycloak_1:8080/auth/",
    "ssl-required": "none",
    "resource": "camunda-identity-service",
    "credentials": {
      "secret": "**********"
    },
    "confidential-port": 0
  }

I have also installed the camunda-bpm-auth-keycloak-sso-1.3.jar into the lib folder.
also note i am using two different clients here (same realm). The engine is connected (and properly sharing an identity server) with one client (camunda-identity-service) and the front end is authenticating through another client (camunda-web-app).
I was wondering if I am missing something in config that will allow us to pass a keycloak token from the react app to authorize requests.

Thank you for the reply @VonDerBeck
WIth the sso issues im having with tomcat I'm going to give the spring boot version a shot. Is the dockerfile in sso-kubernetes the container I need? Or is there an image on docker hub.
Thanks.

Hi @JosephVasc,
sorry for coming back to you so late. When talking about Spring Boot with Camunda 7 - in case you have special environment requirements and integrations the best way in most cases is to setup your own Spring Boot project and build it for yourself. The sso-kubernetes example is such a basic example showcasing the integration.

No further info; since the question concerns another project, i am closing this issue now


Updated docs for clear and reset title

Do you plan to resolve the style issues in this PR? If we don't receive a response, we will be closing this PR in the near future.

Closing due to inactivity.


[Feature Request]: Support for pydantic v2

Feature Description
Migrate to pydantic v2. I noticed there is already a feature branch with related name. https://github.com/run-llama/llama_index/tree/support-pydantic-v2 . But I do not know if its alive
Reason
I do not know.
Value of Feature
Pydantic v1 iss absolete

It's technically supported through the backwards compat layer right now (from pydantic.V1 import ...)
Although there might be one or two areas to clean up, I forget 😅
We will likely migrate fully some time early January? It depends on langchain as well right now

Hi @logan-markewich, thanks for ressponse.
Langchain seems to have supported pydantic v2 just recently.
https://github.com/langchain-ai/langchain/issues/6841#issuecomment-1682996914

@ierehon1905 they are using the compatibility layer, same as us. I think there's just one or two bugs to work out


Can't scan to device on the phone that close system log

I notice that the system log is closed in default on some phone like chinese phone meilan s6 . And can't scan to any device on that phone.How could I solve the problem? Thanks.

Hello,
Could you describe what you are facing in a bit more detail way?
I do not have access to the phone you mentioned. Check if no errors are captured during the scan.
I am not sure what you reference by:

I notice that the system log is closed in default

The system log switch button is a switch button in Developer options ,we could print log in android studio when you turn it on.
Once we turn it off, we could not get the log from Log.d("asdf") in android studio cause we turn it off.
In a word, the switch button could control the output of the log.
I see , the function is less common on android phone. so very few people pay attention to this. Some huawei phones also have this function , but the switch button is in Engineering mode.

Is there any problem with turning the logs on for debugging purposes? If so you can set your custom logger to check the library logs.
If something goes really wrong you would get an error in the .subscribe() function callback.

My problem is that some devices cannot be scanned when the log is turned off, and the log switch on some Chinese mobile phones is turned off by default, so I wonder if I can normally scan the device when the log is turned off.

I see no reason why one thing would interfere with another

I can only see one reason for this to happen. If a call to any Log method would throw a RuntimeException but it seems unlikely. Feel free to dig deeper into the issue and report back if you find a culprit.

@xaxasemail Any news?

Closing due to lack of information and inactivity of the original poster.
When more data will be available—feel free to add that information and reopen this issue.
Best Regards


Replace tt_lib in tests/ttnn/benchmarks, python_api_testing, profiling

Tracking Issue : https://github.com/tenstorrent/tt-metal/issues/11242 , Master Issue : https://github.com/tenstorrent/tt-metal/issues/10532

[x] tests/ttnn/benchmarks
[x] tests/ttnn/python_api_testing - tests/ttnn/python_api_testing/sweep_tests/ttnn_ops.py
[x] tests/ttnn/profiling

Status : In Progress

Need to add additional asserts (for WH) and make it a unit test.

Work Done :

Draft PR : #11822
Discussion

Need inputs on this - No CI tests for tests/ttnn/benchmarks/test_benchmark.py as these sweeps are long running

Status : In Progress
Need to add additional asserts (for WH) and make it a unit test.


[Query]  Dialog - box appears for few seconds when user navigates to Phone number screen while creating the account

Frequency: 100%
Repro on build version: Android Internal Release Build V1.34.0, Android Play store Build V1.34.0
Repro on devices: Google Pixel 4a (12.0), Google Pixel XL(10.0)
Does Not Repro on build version: Test Flight Release Build V 1.34.0 , iOS App store Build V1.34.0
Does Not Repro on devices: iPhone 13 mini(15.1.1), iPhone 6+(12.5.4)
Pre-condition:
1.User must have installed the app
2. User must be on “Secure your wallet Step 3 of 4” screen while creating account
Repro Steps:

Tap on “Turn on Fingerprint recognition” button
Scan the correct fingerprint .
Now observe on the phone number screen page

Query: Dialog - box appears for few seconds when user navigated to Phone number screen while creating the account.
Expected Behavior: Dialog - box should not appear for few seconds when user navigated to Phone number screen while creating the account.
Investigation:

For android platform Dialog-box appears for few seconds when user navigates to Phone number screen while restoring the account.

Attachment: Dialog-box.mp4

I'd consider this a bug as the screen flash is unexpected.


Add LabelFilter::should_include_label_by_name with a KeyName argument

On the project I'm working on, we have to filter excessive metric labels added by TracingContext after the fact, but it could be better if we could see metrics' names in the process of TracingContext::enhance_key. And the inner workings of MetricsLayer doesn't allow us to create custom Layer+Recorder to efficiently add only required metric labels.
This PR adds LabelFilter::should_include_label_by_name method with name: KeyName argument to pass from TracingContext::enhance_key, that way custom LabelFilter, that can filter labels by the name of the metric and label key/value, is an option. The default implementation of said method just calls should_include_label, leaving off the name, so it is not a breaking change.

In case this is a concern, we can avoid breaking changes by adding a method with a default impl to LabelFilter trait.
pub trait LabelFilter {
    fn should_include_label(&self, name: &KeyName, label: &Label) -> bool;
    fn should_include_label_by_name(&self, name: &KeyName, label: &Label) -> bool {
        self.should_include_label(label)
    }
 }

@zohnannor

I also replaced Vec<Label> with SmallVec<[Label; 5]> in the Labels to improve performance (ran the benchmarks a couple of times, but buffer of 5 elements is not tested to be the best option).

I'd say these changes better go as a separate PR. Also, it seems that [Label; 3] is more meaningful and much more common use-case.

@ilsiv Totally! I was thinking of writing it like that...
@tyranron Yes, it is unrelated, removed that from this PR.
Changed PR name & description to reflect new changes.

Exciting to see so much activity in a PR before I even open it. 😍
I'll try and take a look at this today.

Released as<EMAIL_ADDRESS>Thanks again for your contribution!


feat(dynamic-field): apply postfix and prefix

To Reviewers

[ ] Skipping review is not a problem (style, chore ONLY)
[ ] Not that difficult

Type of Change

[ ] New feature
[ ] Bug fixes
[x] Feature improvement

[ ] These changes require a usage change

[x] Others (performance improvement, refactoring, CI/CD, etc.)

Checklist

[x] Updated Storybook documents for component
[ ] Wrote test codes for composable, utils and etc.
[ ] Tested with console(if usages are changed)

Description

Add postfix and prefix feature to each dynamic field template

Refactor text type dynamic field from functional to template syntax

Things to Talk About

:tada: This PR is included in version 1.1.0-beta.496 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


@Published and @ObservableObject use

Object gained support for observability with conformance to @ObservedObject and a couple of @Published values, which is great as it opens up for supporting a SwiftUI frontend that displays information about the scene.
But the property wrappers were later changed from @Published to a custom @PublishedDidSet: https://github.com/Hi-Rez/Satin/blob/c29677b9e7217c66511998fff10b72a00204f76a/Sources/Satin/Core/Object.swift#L50
This custom property wrapper defeats the purpose of conforming to ObservableObject, as the whole purpose of this class is:

ObservableObject
A type of object with a publisher that emits before the object has changed.

If an object does not have any @Published properties, there is no longer any effect in making it an ObservableObject (and thus breaking the ability to use directly in SwiftUI).
If possible I'd suggest to revert these back to @Published. If there is need to observe @Published values in another lib (Youi?) after they have changed, you can always use an async dispatch on the next runloop to ensure the value has been set:
    cancellable = object.$position.sink { [weak self] position in
         // Position not yet set here, self?.object.position is still old
         DispatchQueue.main.async {
             // Position has been set here, self?.object.position is new
         }
     }

Hi @OskarGroth, sorry for the long delay, I've been super busy on my end. I thought about this for a while and I think its worth trying out, so I've reverted back to Published... check out this commit. My logic with PublishedDidSet is that sometimes you'll want to get the transform of the object after a position is set, so if you did object.worldMatrix, you'll get the wrong value in the closure. Just something to be aware of, since I'm sure a lot of people will run into this issue. DispatchQueue.main.async works, but I'm not sure what is best practice in this case.


Experiment loop fix

Issue #, if available:
Description of changes:

allow to load experiments in sub-folder and addresses https://github.com/awslabs/syne-tune/issues/191
sync only relevant experiment results when running remote experiments (before all the results of the experiment tag were synced)
use py3.8 in remote launcher and benchmark loops
fix disk space issue with remote launcher when writing to the home partition instead of "/opt/ml"

also small cleanups:

remove unused estimator_factory (which is duplicated in benchmarking/cli/estimator_factory.py)
remove unused scheduler_name method

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@mseeger , I fixed the issue you mentioned (and also the fact that we were writing logs/checkpoints into home partition in SM), can you approve again?


Week 2 Solution

Description
PNG of Graph of Linear vs. Cubic fit, and .py file
Weekly Journal
The Bayesian showed that the Linear was a better fit than the cubic because it had a low BIC value
Questions

I am submitting one extraneous file because I don't know how to remove them.
You can move or remove a file called filename in the Git history by running git mv filename or git rm filename on the command line, respectively.  Unfortunately, this isn't possible using the GitHub desktop client.

I haven't figured out how to edit documents after I submitted the documents (the step right before the pull request).
You can edit your files and do the git add->git commit->git push as many times as you like, and GitHub is smart enough to incorporate any new commits to origin (your fork) into your PR, without closing and re-opening the pull request.

I fixed it. Thanks


Confusing URLs in example gist responses for GitHub Enterprise

What article on docs.github.com is affected?
<EMAIL_ADDRESS>What part(s) of the article would you like to see updated?
The example response for "Create a gist" (and possibly others).
The URL in the example even for GHE is https://api.github.com/gists/aa5a315d61ae9438b18d
However, on a real GHE server, this is something more like: https://github.example.com/api/v3/gists/aa5a315d61ae9438b18d
The html_url is similarly confusing. https://gist.github.com/aa5a315d61ae9438b18d should be something more like https://github.example.com/gist/some-user/aa5a315d61ae9438b18d.
Additional information
N/A

Thanks so much for opening an issue @kpaulisse! This is something that we'll have to look into internally. I'll relay your comments to the team and close this out, thanks so much!


Remove Published and Last updated

#What
Don't display 'published' and 'last updated' in metadata on the Page Data page.
#Why
We don't trust dates to be accurate and are worried we'll mislead our users. We need to investigate the accuracy of dates more thoroughly before we can display them to users.
Trello https://trello.com/c/5xJV5Xrk/870-1-page-data-dont-display-published-and-last-updated-in-metadata
Screenshots
Before

After

Review Checklist

[ ] Changes in scope.
[ ] Added/updated unit tests.
[ ] Added/updated feature tests.
[ ] Added/updated relevant documentation.
[ ] Added to Trello card.

 (1) Page data - don't display 'published' and 'last updated' in metadata


Unexpected indent while importing packages

Issue Report Checklist

[ ] Searched the issues page for similar reports
[ ] Read the relevant sections of the Spyder Troubleshooting Guide and followed its advice
[ ] Reproduced the issue after updating with conda update spyder (or pip, if not using Anaconda)
[ ] Could not reproduce inside jupyter qtconsole (if console-related)
[ ] Tried basic troubleshooting (if a bug/error)

[ ] Restarted Spyder
[ ] Reset preferences with spyder --reset
[ ] Reinstalled the latest version of Anaconda
[ ] Tried the other applicable steps from the Troubleshooting Guide

[ ] Completed the Problem Description, Steps to Reproduce and Version sections below

Problem Description
What steps reproduce the problem?

While importing packages like numpy, pandas, matplotlib

What is the expected output? What do you see instead?
Paste Traceback/Error Below (if applicable)

ImportError: cannot import name 'TYPE_CHECKING' from 'typing' (/usr/local/lib/python3.7/dist-packages/numpy/typing/__init__.py)

Versions

Spyder version:
Python version: 3.7
Qt version:
PyQt version:
Operating System name/version:

Dependencies

PASTE DEPENDENCIES HERE

Hey @S2Sofficial, thanks for reporting. This is not an error with Spyder but Numpy, so we can't help you here.
The only recommendation we can give you is use Anaconda for your user instead of sudo pip, which is what you seem to used to install Numpy.


feat: Push tasks directly to the local runner

Basically implements #22 but in a simpler and less dependency-heavy way.
I'm marking this as draft for now, since the current implementation here actually seems less performant than the master branch implementation. I'll optimize this later.

Superseded by #37


add the traffic_forward:contains two diversion modes: DNAT and DNS

What type of PR is this?

Uncomment only one  /kind <> line, hit enter to put that in a new line, and remove leading whitespace from that line:
/kind feature

What this PR does / why we need it:
DNAT and DNS, as the two diversion modes of yurt-tunnel, have the same function, so I put iptables and dns in the  directory:pkg/yurttunnel/traffic_forward, and I will solve this issue548 in the next PR.
Which issue(s) this PR fixes:
Special notes for your reviewer:
@rambohe-ch
Does this PR introduce a user-facing change?

NONE
other Note

@JcJinChen please fix the golangci lint error.

@JcJinChen how about modify traffic_forward to trafficforward?  because all directory names have no _.

@JcJinChen how about modify traffic_forward to trafficforward? because all directory names have no _.

get it.

@JcJinChen Would you be able to explain that go.sum file updated?


Handling the row of consensus symbols in clustal alignment

When clustal alignment produces a row of consensus symbols below the aligned sequences, msa seems to not handle these properly. Perhaps there is a parameter that I need to add to specify that this is a special row, but I couldn't find what it should be.
Thanks!

Hi @chanulia,
this is most likely an issue with the BioJS clustal parser. I will try to look into this as soon as possible.
All the best
Benedikt

Hello again @chanulia
after some trying today I was not able to reproduce the issue. Could you by any chance provide me  with an example file that cannot be displayed properly in the msa viewer?
Many thanks!

I am attaching an example of the alignment file. It displays fine, - just the last line, the row of consensus symbols seems to not be correct (the program needs to treat that line in a different way, but it seems that it just reads that line like another sequence).
clustalw66638.aln.txt
Thanks!

This should be fixed now (see PR #203). Sorry for the wait!

Not sure if I am getting the updated file. I was using a copy of msa.min.gz.js. I tried this one: cdn.bio.sh/msa/latest/msa.min.gz.js and it doesn't work at all for me. Doesn't seem to load the url correctly at startup.

Ok, this is unrelated to the original problem, but it seems that I have to change where it says:
importProxy:"https://cors-anywhere.herokuapp.com/",importProxyStripHttp:!0,
to:
importProxy:"",importProxyStripHttp:"",
just to load the alignment url correctly
but the original problem remains, - I must not be getting the updated file.


More stable way to detect start of data transmission

Clean solution instead of https://github.com/adafruit/Adafruit_Python_DHT/pull/6/files https://github.com/adafruit/Adafruit_Python_DHT/pull/24/files https://github.com/adafruit/Adafruit_Python_DHT/pull/46/files
I tested it during several days with short and long (10m) cables.

hi this repo is going to be deprecated soon, please try
https://github.com/adafruit/Adafruit_CircuitPython_DHT
which will replace this library


Changed shutdown to exit process

I will change it when i find an lt to it. possibly looking at Async-Kit

you could set a timeout for a second or two with the process kill in it then have a send message right before it


Fixups

As discussed in https://github.com/atom-community/markdown-preview-plus/issues/402, here's a PR fixing some stuff I stumbled upon.
First and foremost, this is a re-write to align better with current Atom ecosystem and API. The API used has been around at least since 1.13.0, so Atom version bounds are not adjusted.
The code has been rewritten in (more or less) pure JavaScript instead of CoffeeScript. Mostly because I haven't written in CS for years, so it was just easier for me, but also because CS is kinda dying in general and within Atom ecosystem in particular.
What this rewrite does differently:

Fixes config subscription disposal -- old code didn't properly dispose of config subscriptions
Fixes extraneous calls to applyFont on activation -- old code called that thrice due to atom.config.observe triggering right away (unlike atom.config.onDidChange)
Adds a new stylesheet via Atom's style manager, instead of setting the style attribute on the body element. I believe it's rather self-evident that setting style attribute on body is not very robust and can potentially lead to conflicts with other packages. Now the package uses the same method Atom itself does and sets the variable on :root instead of body (which is more or less what Atom does). This particular change also fixes https://github.com/atom-community/markdown-preview-plus/issues/402
Simplifies fonts-fixer element creation code
Removes fonts-fixer element on deactivation

Additionally, it moves actual atom-text-editor style into it's own file, main.less, since otherwise it's included twice, once from functions.less, and once from fonts.less, which imports the former. Now functions.less only contains LESS functions and variables.
This is ready for review.

Wow, thanks. I guess I'll have to spin up Atom and check it out.
Would you mind if I add you as a collaborator on the repo? It'll give you full access to make fixes and push releases.

Would you mind if I add you as a collaborator on the repo?

I don't really mind, but here's a disclaimer: I have been roped into maintaining two projects this way already, and as a result, I kinda have my hands full as it is. So don't expect much activity from me.

Well it's not so much about maintaining, but we're merging a bunch of stuff I didn't write and I like the idea that you might be able to go in an push a fix if the need arises. I already maintain enough stuff myself, so I know what it's like ;)

To be clear: are you OK with merging/releasing this, or do you want to test it first?

I did a few quick tests, and everything seems to be in order. But I'll keep using it for now in case I notice something misbehaving. Merging now, will push out a release next time I remember about it, probably after this weekend.

🥇


Welcome to my GitHub profile!

I have crested a new branch, added a file, made a pull request

good work


Lag with large number of fixtures

Hey @ro-ka , first of all, thanks for the great library.
I'm running into what may well be an edge case/something the component wasn't really made to handle. I'm making heavy use of the fixtures property because we have a lot of locations we want to display, and google maps is more of a fallback.
I'm now testing with a dataset that is ~4000 locations passed in as fixtures, and it basically grinds the component to a halt :( After a few characters have been input it works fine again, but the first several freeze things pretty heavily.
I haven't had a chance yet to dig into the code and debug it much, thought I would create an issue first. Is this something you're interested in handling? If possible, I'd be happy to take a stab at it and make a PR, but I'm not even sure my use case will make sense for other users of the component.
Cheers,
David

Should probably look at the code before writing! In any case, I've already got great results doing something as simple as:
var suggests = [],
  regex = new RegExp(escapeRegExp(this.state.userInput), 'gim'),
  skipSuggest = this.props.skipSuggest,
  searchedCount = 0;

this.props.fixtures.forEach(function(suggest) {
  if (searchedCount > 20) {
    return;
  }

  if (!skipSuggest(suggest) && suggest.label.match(regex)) {
    searchedCount++;
    suggest.placeId = suggest.label;
    suggests.push(suggest);
  }
});

(from Geosuggest.js)
That is, only showing max 20 fixtures results. Could make this a configurable prop. Basically the biggest source of slowdown is the regex, and as currently implemented, clicking on an empty search box makes it perform the regex on EVERY fixture, even though no user will realistically scroll through a list of thousands of results.


Fix get_value

As the title.
Here we are implementing approach a in https://github.com/ansys/pymapdl/issues/2772#issuecomment-1941613370.
Close #2772

@pyansys-ci-bot LGTM.


BZ2029418: workflow without VMIO

https://bugzilla.redhat.com/show_bug.cgi?id=2029418
Remove VMIO from workflow.
Previews:
https://deploy-preview-258--forklift-documentation.netlify.app/documentation/doc-migration_toolkit_for_virtualization/master/#virt-migration-workflow_forklift
https://deploy-preview-258--forklift-documentation.netlify.app/documentation/doc-migration_toolkit_for_virtualization/master/#migrating-virtual-machines-cli_forklift

Regarding https://deploy-preview-258--forklift-documentation.netlify.app/documentation/doc-migration_toolkit_for_virtualization/master/#virt-migration-workflow_forklift:

Replace Virtual-Machine-Import with Migration plan in flowchart.
Replace Virtual Machine Import Controller with Migration Controller in flowchart.
Replace the workflow steps:

The workflow describes the following steps:

When you run a migration plan, the Migration Controller creates a VirtualMachine custom resource (CR) for each source virtual machine (VM), and a DataVolume CR for each of the source VM's disks.

For each VM disk:

The Containerized Data Importer (CDI) Controller creates a Persistent Volume Claim (PVC) based on parameters specified in the DataVolume CR. The Persistent Volume (PV) is dynamically provisioned by the StorageClass provisioner.
The CDI controller creates an importer pod.
5  The importer pod streams the VM disk to the PV.

After the VM disks are transferred:

The Migration Controller creates a conversion pod with the PVCs attached to it. The conversion pod runs virt-v2v, which installs and configures device drivers on the PVCs of the target VM.
When the target VM is powered on, the KubeVirt Controller creates a virt-launcher pod and a VirtualMachineInstance CR. The virt-launcher pod runs QEMU-KVM with the PVCs attached as VM disks.

Regarding https://deploy-preview-258--forklift-documentation.netlify.app/documentation/doc-migration_toolkit_for_virtualization/master/#migrating-virtual-machines-cli_forklift

In bullet point 9, Replace "VirtualMachineImport" in "Specify the name of the Plan CR that you are running. The Migration CR creates a VirtualMachineImport CR for each VM that is migrated." with "VirtualMachine"
In bullet point 10, Replace "View the VirtualMachineImport pods to monitor the progress of the migration:" with "View the Plan status to monitor the progress of the migration."
In bullet point 10, Replace oc get pods -n konveyor-forklift with oc get plan konveyor-forklift/<plan-name> -o yaml

@apinnick I've submitted a draft PR with edits to additional sections. I didn't touch the flowcharts, those will still need to be adjusted. The "warm import retry limit reached" message no longer exists, so the "error messages" section is empty in my PR. Hopefully this is helpful in putting together the final documentation.

Closing this PR because the preview build is not working. Changes moved to https://github.com/konveyor/forklift-documentation/pull/281


Add RSMPI_UNWEIGHTED

This is required by https://github.com/linalg-rs/rlst/issues/20
Minimal example that fails with openmpi but runs with mpich on current rsmpi main: https://github.com/mscroggs/mpi-minfail

Converted this to a draft as my failing example is now passing with openmpi but failing with mpich

I've made RSMPI_UNWEIGHTED into a function as this was the only way I could find to make this work with both openmpi and mpich.
@jedbrown perhaps you have an idea for how to better do this?

Can you share a minimal runnable test? The one in mpi-minfail doesn't run, and returns a communicator that is dropped after MPI_Finalize.
MPI_UNWEIGHTED is in the batch of constants that cannot be used in Fortran initialization expressions, but as I understand it, should be link-time constants in C. Looking at the MPICH implementation, it is initialized as a link-time constant, so there must be some interaction I don't yet understand. The function is okay if indeed necessary.

Also note that support for graph communicators would ideally move into rsmpi itself, though I understand that may be a bigger ask.

Can you share a minimal runnable test?

I resorted to running cargo test in rlst/sparse to test this. I'll work on a more minimal test

Can you share a minimal runnable test?

I've added a minimal example to the mpi-minfail repo: https://github.com/mscroggs/mpi-minfail/blob/main/examples/test.rs
From the CI runs, you can see that this fails with openmpi using the current rsmpi (runs tagged "rsmpi"), but passes for both mpich and openmpi with this patch (runs tagged "mscroggs"): https://github.com/mscroggs/mpi-minfail/actions/runs/4979552904

Oh, I understand the issue now and this solution is fine. The original values can be used as initializers inside functions, but not with static storage duration. People don't usually notice this because they're initializing local variables (inside functions, which have "automatic" storage duration).
Looks like we need to bump MSRV to 1.60 or pin a version on a dependency. Can't wait for this MSRV problem to be resolved by Cargo.

Alright, please rebase on main and this should be good to go.


fix some tests for Android

I tested the crate on an Android emulator and quite all the tests passed. Only 6 were failing.

failures:
---- sys::test_pthread::test_pthread_self stdout ----
thread 'sys::test_pthread::test_pthread_self' panicked at 'assertion failed: tid > 0', test/sys/test_pthread.rs:6
note: Run with RUST_BACKTRACE=1 for a backtrace.
---- sys::test_socket::test_getsockname stdout ----
thread 'sys::test_socket::test_getsockname' panicked at 'bind failed: Sys(EACCES)', /checkout/src/libcore/result.rs:859
---- sys::test_socket::test_unixdomain stdout ----
thread 'sys::test_socket::test_unixdomain' panicked at 'bind failed: Sys(EACCES)', /checkout/src/libcore/result.rs:859
---- sys::test_termios::test_local_flags stdout ----
thread 'sys::test_termios::test_local_flags' panicked at 'called Option::unwrap() on a None value', /checkout/src/libcore/option.rs:329
---- test_net::test_if_nametoindex stdout ----
thread 'test_net::test_if_nametoindex' panicked at 'assertion failed: if_nametoindex(&LOOPBACK[..]).is_ok()', test/test_net.rs:11
---- test_unistd::test_mkstemp stdout ----
thread 'test_unistd::test_mkstemp' panicked at 'mkstemp failed: ENOENT: No such file or directory', test/test_unistd.rs:73

This PR fixes 3 of those:

test_pthread_self: pthread_t is unsigned, so it can be negative (and it is often the case)
test_if_nametoindex: constant LOOPBACK is the same than on Linux
test_mkstemp: directory /tmp does not exist in Android

Two are still failing (test_unixdomain and test_getsockname) because we need some special permissions on Android for playing with sockets. On a rooted physical device, they passed. So, if tests for Android are integrated in CI, we should try to embed the tests to run in an APK with requested permissions or find a way to give the permission to a native program.
test_local_flags is still failing also because O_LARGEFILE is hardcoded and is not the right value in Android. Then fcntl::OFlag::from_bits(flags).unwrap() fails because this flag is then not recognised.
I made a PR in libc so that we can rely on libc definitions for all O_* flags. (rust-lang/libc#683)
Do you prefer to wait for this one to be fixed for this PR to merged as well ?

This is a great start, thanks @ndusart!

Do you prefer to wait for this one to be fixed for this PR to merged as well?

While fixing all the tests is ideal, I'd like to limit this PR to those tests which can work with the current release of libc and also don't add any new hard-coded values. We'll wait on the rest once 0.9 is released and we're back to using libc-git. You can either raise a new issue or a separate PR to track that work; your call.
Regarding some points you brought up:

Two are still failing (test_unixdomain and test_getsockname) because we need some special permissions on Android for playing with sockets. On a rooted physical device, they passed. So, if tests for Android are integrated in CI, we should try to embed the tests to run in an APK with requested permissions or find a way to give the permission to a native program.

I get the feeling we'll run into tons of permission issues with Android as we continue to develop nix. So I'm fine with needing to figure out a rooted app to run tests on. It'll be up to the users of nix to figure out if Android supports that functionality or not for their own use cases.

@asomers Wanna take another look at this? Looks like small solid fixes to me!

bors r+


Get Enumeration to FieldTypes

Please add enum to FieldTypes.
When we get fieldTypes(i) we get number id not the description of fieldtype.
thank you.

While I agree it would be better to return a more useful type string, these types can be compared to those exported here: https://github.com/IBM/nodejs-idb-connector/blob/master/lib/db2a.js#L337-L377

Thank tou! Is that I use now. 😊
From: Kevin Adler<EMAIL_ADDRESS>Reply to: IBM/nodejs-idb-connector<EMAIL_ADDRESS>Date: Wednesday, 6 May 2020 at 04:59
To: IBM/nodejs-idb-connector<EMAIL_ADDRESS>Cc: João Mendes<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [IBM/nodejs-idb-connector] Get Enumeration to FieldTypes (#115)
While I agree it would be better to return a more useful type string, these types can be compared to those exported here: https://github.com/IBM/nodejs-idb-connector/blob/master/lib/db2a.js#L337-L377
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/IBM/nodejs-idb-connector/issues/115#issuecomment-624429272, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AC7YHVO6YJ3VOQ4KVE6KXTLRQDOAFANCNFSM4MNPXNLQ.

Actually we have this internal used function getSQLType. But it is only used for debug currently.

I would like to add a new function (maybe called fieldInfo) to provide all the information of the column, including type, width, nullable, etc.

Hello @joaojmendes ,
The PR is drafted at #118 . If no problem, I will merge it later.

Hello, @Xu<EMAIL_ADDRESS>Thank you for the answer.
Yes seem a good solution to have one object with all information about column.
Thank you !
João
From: Xu Meng<EMAIL_ADDRESS>Reply to: IBM/nodejs-idb-connector<EMAIL_ADDRESS>Date: Thursday, 14 May 2020 at 06:45
To: IBM/nodejs-idb-connector<EMAIL_ADDRESS>Cc: João Mendes<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [IBM/nodejs-idb-connector] Get Enumeration to FieldTypes (#115)
I would like to add a new function (maybe called fieldInfo) to provide all the information of the column, including type, width, nullable, etc.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/IBM/nodejs-idb-connector/issues/115#issuecomment-628400893, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AC7YHVLDTEK2HFGAZIH45HLRROARHANCNFSM4MNPXNLQ.

Close this issue since idb-connector v1.2.9 is released now. This new version contains the new API to resolve the issue.


Android notification not show when app closed

For the foreground and background it's working fine.
But , When app complete closed I can not get notification.

Would you share the Push payload you are sending?
And how do you expect to get the notification:

onNotification() or
As a notification in the notification bar?

Hi andrehtissot,
i have the same problem, the plugin works very well for foreground and background. But when the app is closed I noticed that it doesn't arrive
"function setupOnNotification () {
FCM.eventTarget.addEventListener (
"notification",
function (data) "
This is the payload that I intercept when the app is closed but I repeat it is not of type notification in the EventlListiner and therefore does not enter the SetupOnNotification function. Can you help me solve it? Thanks in advance
FCM.getInitialPushPayload()
.then((payload) =>
payload = {google.sent_time:<PHONE_NUMBER>488, google.original_priority: "high", title: "chiusa", collapse_key: "com.***.app", wasTapped: true, …}

Hello, I'm experiencing the same issue here. When I receive the notification, it appears correctly on system notifications. However, when I click on the notification, the app opens but it doesn't receive the data or trigger the event. I would expect it to show 'wastapped=true' and display the incoming data, but it doesn't. It seems like there's an internal communication issue between the system notification and the app's function to receive it. If the app is already open, everything works as expected. Has anyone resolved this or can offer assistance?


[SPARK-12441] [SQL] Fixing missingInput in Generate/MapPartitions/AppendColumns/MapGroups/CoGroup

When explain any plan with Generate, we will see an exclamation mark in the plan. Normally, when we see this mark, it means the plan has an error. This PR is to correct the missingInput in Generate.
For example,
val df = Seq((1, "a b c"), (2, "a b"), (3, "a")).toDF("number", "letters")
val df2 =
  df.explode('letters) {
    case Row(letters: String) => letters.split(" ").map(Tuple1(_)).toSeq
  }

df2.explain(true)

Before the fix, the plan is like
== Parsed Logical Plan ==
'Generate UserDefinedGenerator('letters), true, false, None
+- Project [_1#0 AS number#2,_2#1 AS letters#3]
   +- LocalRelation [_1#0,_2#1], [[1,a b c],[2,a b],[3,a]]

== Analyzed Logical Plan ==
number: int, letters: string, _1: string
Generate UserDefinedGenerator(letters#3), true, false, None, [_1#8]
+- Project [_1#0 AS number#2,_2#1 AS letters#3]
   +- LocalRelation [_1#0,_2#1], [[1,a b c],[2,a b],[3,a]]

== Optimized Logical Plan ==
Generate UserDefinedGenerator(letters#3), true, false, None, [_1#8]
+- LocalRelation [number#2,letters#3], [[1,a b c],[2,a b],[3,a]]

== Physical Plan ==
!Generate UserDefinedGenerator(letters#3), true, false, [number#2,letters#3,_1#8]
+- LocalTableScan [number#2,letters#3], [[1,a b c],[2,a b],[3,a]]

Updates: The same issues are also found in the other four Dataset operators: MapPartitions/AppendColumns/MapGroups/CoGroup. Fixed all these four.

Test build #48055 has started for PR 10393 at commit 21071d1.

Test build #48056 has started for PR 10393 at commit 22d8ba5.

Test build #48055 has finished for PR 10393 at commit 21071d1.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/48055/
Test PASSed.

Merged build finished. Test PASSed.

Test build #48056 has finished for PR 10393 at commit 22d8ba5.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/48056/
Test PASSed.

Merged build finished. Test PASSed.

Test build #48071 has started for PR 10393 at commit b7edd97.

Test build #48071 has finished for PR 10393 at commit b7edd97.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):\n  * public class JavaTwitterHashTagJoinSentiments \n  * case class UnresolvedAlias(child: Expression, aliasName: Option[String] = None)\n

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/48071/
Test PASSed.

Merged build finished. Test PASSed.

Test build #48096 has started for PR 10393 at commit 6b4ba74.

Test build #48098 has started for PR 10393 at commit 63058e3.

Test build #48098 has finished for PR 10393 at commit 63058e3.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/48098/
Test PASSed.

This LGTM.
But I have a question that do we need these resolution related stuff in SparkPlan? cc @marmbrus

Test build #48096 has finished for PR 10393 at commit 6b4ba74.

This patch fails from timeout after a configured wait of `250m`.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/48096/
Test FAILed.

Merged build finished. Test FAILed.

retest this please

Test build #48115 has started for PR 10393 at commit 63058e3.

Test build #48115 has finished for PR 10393 at commit 63058e3.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/48115/
Test PASSed.

I think its fine to have this stuff in physical plans.  I actually added the ! when I was debugging problems with the query planner (i.e. it was producing spark plans with the wrong attributes).

Build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/48217/
Test FAILed.

Build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/48218/
Test FAILed.

Test build #48219 has started for PR 10393 at commit 8c48c35.

Test build #48219 has finished for PR 10393 at commit 8c48c35.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/48219/
Test PASSed.

Merged build finished. Test PASSed.

Thanks, merging to master.

Thank you!


UIRefreshControl-like behavior

Split loading state to one while dragging and after the user releases his finger. Now control starts animating and sends update request just after the user scrolls enough down while he is dragging. Both cases of releasing before finishing and after it.
Fixed loading animation (slight jumping) on iOS 11.
Added .top default value to Position params

hey, @bondareb-evgenii
sorry for a late response and thank you for your contribution!
please resolve conflicts so I can merge your pr


chore(deps): bump pylint plugin

The 3rd-party pre-commit failure is capturing a deprecation. distutils.version will get nuked in v3.12, we'll want to upgrade to package.versions before then. Merging for now as 3.11 isn't even live yet..


Additions

https://github.com/mjcuva/Hidrate

game https://github.com/mjcuva/Chess

sample https://github.com/kylef/Starship

apple watch https://github.com/swilliams/watchkit-connectivity-demo

https://github.com/e-mission/e-mission-phone

https://github.com/mbcrump/TasksForSwiftWithPersistingData

https://github.com/imod/MyAwesomeChecklist

https://github.com/dasdom/KeychainDemo

location https://github.com/eugenpirogoff/pathlogger 11

core data apps #89

misc

[ ] https://github.com/1raynes/speakability 11
[ ] https://github.com/jamiely/ios-balloon-burst 8
[ ] https://github.com/Azoft/CollageMaker-iOS 10
[ ] https://github.com/Mav3r1ck/Tripletz 6
[ ] https://github.com/lucasqiu/Adler-Mobile-App 4
[ ] https://github.com/weiran/Hackers 285 but not maintained
https://github.com/KinveyApps ... ?
[ ] Cepp: Find information on addresses (Brazil)

https://github.com/filipealva/Cepp
https://itunes.apple.com/br/app/cepp-encontre-ceps/id942709971

React Native

[ ] https://github.com/peralmq/ForeignExchangeApp 6
[ ] https://github.com/christopherdro/react-native-sudoku 21
[ ] https://github.com/christopherdro/react-native-sudoku 21

Apple Watch

https://github.com/coolioxlr/watchOS-2-heartrate
[ ] Guess Game :large_orange_diamond:
[ ] Weather :large_orange_diamond:
[ ] https://github.com/jazzychad/FlickrWatch 3
[ ] Stocks App

0ish :star:  :confused:

[ ] https://github.com/andreamazz/Gitorade 1
[ ] https://github.com/danylokostyshyn/AfishaLviv-iOS
[ ] https://github.com/danylokostyshyn/FS-Player
[ ] https://github.com/iOSBeginner/SleepTracker Record and improve your sleep
[ ] https://github.com/AdnanZahid/Math-Quest-iOS
[ ] Coin Tracker: Cryptocurrency coin tracker :large_orange_diamond:
[ ] https://github.com/jenduf/SwiftFanorona
[ ] https://github.com/thomasdegry/Facebook-Pop-Introduction

done

[x] game https://github.com/phpmaple/Stick-Hero-Swift 150
[x] PhoneBattery: Your phone's battery, on your wrist

https://github.com/marcelvoss/PhoneBattery
https://itunes.apple.com/app/phonebattery-your-phones-battery/id1009278300

[x] https://github.com/sonsongithub/AAKeyboard

Add Breadwallet and Doughwallet under finance

Thanks
Did you mean DogeWallet

Nope. Doughwallet. :D

Sent from Deavmi
Keep on hacking...
Contact Deavmi via:
Pjuu: https://pjuu.com/deavmi
GNU Social<EMAIL_ADDRESS>Twitter: https://twitter.com/deavmi
Main mailbox<EMAIL_ADDRESS>
On 08 Jul 2015, at 3:43 PM, dkhamsing<EMAIL_ADDRESS>wrote:
Thanks
Did you mean DogeWallet
—
Reply to this email directly or view it on GitHub.

Ah got it

Thanks :D


Figures related to rankings and heatmap could be transposed.

These are some suggestions regarding visualization defaults.
Working with the ranknings plot i realized that it would be much easier to read if the figure where transposed/rotated so that gene names could be read per line rather than per columns. This would also scale better if one wishes to plot more than 10-30 genes.
There is no inherent information on the y axis that could not be layed out on the x axis.
I also ran in to this format when using the heatmap plotting. Gene names are on the columns rather than rows resulting in a thin matrix and hard to read gene names. As the figure is constructed by using several axis one could transpose the figure having the gene names to the right colorbar on the right and groupings below or ontop of the heatmap. As one is most interested in a dendrogram on the cells (I think?) it could get a nice space on top.

@fidelram should be the expert for this... :smile:

With respect to the heatmap, indeed it is possible to transpose the matrix.
Currently, this option is only available for stacked_violin. I thought
about adding this option to other plots like heatmap, matrixplot and
dotplot but I have not find the time and it is always possible to save the
figure and rotate it so it has low priority for me. The changes are not as
trivial as simply rotating the matrix as all other elements need to be
adjusted.
On Wed, Nov 7, 2018 at 3:03 AM Alex Wolf<EMAIL_ADDRESS>wrote:

@fidelram https://github.com/fidelram should be the expert for this...
😄
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/theislab/scanpy/issues/349#issuecomment-436477272,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEu_1b_dlMN1mihuJIbXg2lPmMJvgqGgks5usj-FgaJpZM4YRQ7g
.

I made a hack solution for this to have something that would work for me. I put it here: https://gist.github.com/Xparx/33026da63dabb1c200b2602bbae0b95c
It's not defensively coded so it might bug out without proper input.

Thanks! Can you post an image to get an idea of your solution?

Sure. Here is one example of my output image

Thanks. I will do a PR soon which allows the heatmap and matrixplot functions to be rotated.

As this issue is not closed I'll add a question here.
Is it possible, or if not could it be added, that the cells in the heatmaps are sorted within the groupby variables. Either by pseudotime if availible or just clustered simply by hierarchical clustering. This could add a more visually and intuitive pleasing ordering of cells.
For example as in my figure above the groupby miss some property of the data with pattern over cells. If the cell ordering was random or default this pattern could not be seen.

Should be possible to do what you want.
sc.pl.heatmap basically plots a pandas DataFrame that has as columns the selected gene names, and as rows the cells with an index based on the categories.
You can modify this dataframe (called obs_tidy) to have the ordering that you want.
This is the line in the current code that returns the dataframe: https://github.com/theislab/scanpy/blob/master/scanpy/plotting/_anndata.py#L1098

Again, as this issue is not closed,
is it possible to do hierarchical clustering on the genes while plotting the heamap?  By grouping the genes according their expression in the clusters would be a lot informative.


'COALESCE' function

Implement the simple 'COALESCE' function which takes arbitrary number of parameters and returns the first non-null.
It can be as substitution of 'IS_NULL(x)? a : x', which is equals to 'COALESCE(x,a)'

Hi @gzsombor,
thanks for your effort. Contributions are always welcome to our project!
Unfortunately, there is already a function NOT_NULL(), which does exactly that:
https://github.com/gzsombor/arangodb/blob/6a5fb92063418d1cd66278752cfaad7de98bb9d6/arangod/Aql/Functions.cpp#L3316
The code is identical to yours, except for some minor style differences. What your implementation lacks is a JavaScript version of the function. It is not strictly needed, but if the surrounding code is a V8 expression, then it is usually cheaper to use a JS implementation of a function instead of converting back and forth between native and V8 data structures.
https://github.com/arangodb/arangodb/blob/55532a9fa5c16aee20340538ebe2fe9356fa0dba/js/server/modules/%40arangodb/aql.js#L4252
I'll add a note to the documentation that NOT_NULL() is what people know as COALESCE() from SQL.

 I've just saw that on the mailing list, a couple of days ago, someone proposed some hack  to use the MAX function, and I thought, that must mean, there is no other, simpler way to do it.
Thanks, for letting me know the working solution


Discord links aren't working

Quick fix with alerts at 666bc9d035aa4d4e0a799b6a8410e731dcb12026 but can try tooltips with bootstrap or popper.js in the future

c6f2e6dcaec06c3706f3b6663e536826403f59a8


Include proposal rationale title and description in JSON block

Description
Rationale title and description are not visible in the "Proposal terms" JSON on the proposal details page. Considering the JSON is a way to see the "raw" proposal submitted, the expected behaviour is that this information would be included there.
Steps to reproduce

Go to a proposal - note the title and description (which is beside the proposal type label)
Scroll down to the "Proposal terms" section and see it's not there

Screenshots

@ckpringle the work on this ticket depends on what you're really after - the rationale doesn't sit inside the proposal's 'terms', it's its own separate value (of which there are many we don't show - party, state, votes, datetime, rationale). We show everything that comes down the pipe within the proposal's terms value. We show the terms because we haven't yet put in the time to make sure we display everything nicely in tables. Barney's recently said we need to (https://vegaprotocol.slack.com/archives/C038EB6V2RE/p1683889039913589?thread_ts=1683825029.555719&cid=C038EB6V2RE) so this ticket may be redundant

So I guess then we'd end up with properly rendering the relevant details in a nice way, and then this part should just become the full raw JSON for any technical types that wanted to review it?  Rather than specifically cherry picking the "terms"?
That would make sense to me if that was the approach.  If we do that I guess we'd need some new tickets for the proper rendering.

Ok capturing as a separate ticket - https://github.com/vegaprotocol/frontend-monorepo/issues/3778

This is already done under https://github.com/vegaprotocol/frontend-monorepo/issues/3778 ... closing


Adopt 1ES Pipeline Templates

This is an internal compliance ask of the vcpkg team.
Example build: https://devdiv.visualstudio.com/DevDiv/_build/results?buildId=9311251&view=logs&j=4cdc4065-f43d-507c-ee4e-5b139a5033dc&t=07e7a3b8-7638-5eac-04cb-963fb3791397

Test build showing the apiscan part now passes: https://devdiv.visualstudio.com/DevDiv/_build/results?buildId=9322240&view=results


Interpreter improvements

The interpreter should interate a list of expressions and/or statements,
and print the results of any expressions that do not return none.
Eval should fail (should eventually be Syntax Error) if it gets a
statement.
Still need to implement continuation lines.
is_none doesn't belong in main, but I'm not sure where to put it (or if there's a better implementation). The is method on vm is private and wouldn't work anyway because None isn't a singleton yet.

I've included my multiline stuff even though the implementation isn't really great. It's too useful not to have!

Nice one!


Enforce CornerExponent of 2 for circles unless explicitly set

There was a regression in rendering of circles that are not CircularContainers (RingPiece, CirclePiece etc.), since they are not explicitly set to use a CornerExponent of 2.
This change will assume a CornerExponent of 2 if the CornerRadius is half of either of the DrawSize dimensions, unless the user has explicitly set a value for CornerExponent.
Are you ok with this change @Tom94 ?
Example of the pointy circles:

@Tom94 What I was thinking as a potential solution is to add a bool CornerContinuous. Then when rendering:

If CornerContinuous is false, always render with a CornerExponent of 2, regardless of the user's setting.
If CornerContinuous is true, and the user has not explicitly set a CornerExponent, always use 2.5f.
If CornerContinuous is true, and the user has explicitly set a CornerExponent, use that.

A CircularContainer should always force CornerContinuous to false, but I am unsure as to what the default should be.
Defaulting to false means the user needs to know about the property get the benefits.
Defaulting to true means that if the user makes circular components without using a CircularContainer (RingPiece, etc.) they will look bad unless the user explicitly sets CornerContinuous to false.

I don't really like that option as it has the same default-dilemma as CornerExponent 2 vs 2.5 has, only with the added complexity of the extra bool.

My $0.02 on this: corner exponent should be 2 by default, and 2.5 only if the user wants it. I don't think CornerExponent is the correct API to expose for this (in which situations would you use something that isn't 2 or 2.5?).

Larger values can look more pleasing on smaller buttons, whereas for larger buttons 2.5 is probably pretty good.


Added test for missing authentication token

Motivation and context

Related #5331
Added test, changed fix because of temporary solutions in #5344
How has this been tested?

Checklist

[ ] I submit my changes into the develop branch
[ ] I have added a description of my changes into CHANGELOG file
[ ] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[ ] I have linked related issues (read github docs)
[ ] I have increased versions of npm packages if it is necessary (cvat-canvas,
cvat-core, cvat-data and cvat-ui)

License

[ ] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.

Test looks fine, but can you look at other tests? Something wrong with them.
And what with test when Authentification by URL? As far as I remember you told we have it .. ?

/check

@klakhov
There is one failed test: actions_users/issue_1810_login_logout.js

@bsekachev Could you look on this once again?


Fixed a UI issue and added back to top button on homepage

Description

I have fixed a UI issue in which the Instagram social icon was colored violet instead of blue. Also, I have added a back to top button on the homepage of the website.

Issue Ticket Number
Fixes #200

Type of change

[X] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist:

[X] I have followed the contributing guidelines of this project as mentioned in CONTRIBUTING.md
[X] I have checked to ensure there aren't other open Pull Requests for the same update/change?
[X] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes needed to the documentation

Screenshots:
1. Fixed UI issue:

2. Back to Top Button:

Congrats @durgeshm01722 🎉, your PR got merged. To be eligible for the swags from ClueLess. Follow this steps.

Star this repository. ⭐

SignIn / SignUp in [ClueLess Website](https://www.clueless.tech/) .

Go to profile section from the top right hand corner.

Comment your profile URL under this comment tagging the person who merged your PR.

That’s it. Follow the leaderboard !
And don’t forget to share your achievement with your friends.

** My Profile URL ** - https://www.clueless.tech/profile/103863839958922066506
** PR merged by ** - @Debajyoti14


Meeting notes may 22nd

Update meeting notes with TSC meeting May 22nd and add cards to the homepage with latest blog posts

This PR is being closed in favor or registering meeting notes on the ASWF confluence:
https://wiki.aswf.io/display/OPENCUE/OpenCue+Home


docs: add references to API docs in a couple commands

This pull request is an example of how we might add these links to the descriptions of the commands that are backed by API calls.
Checklist

[x] I have read the CONTRIBUTING document
[ ] Any required documentation has been added
[x] My code follows the code style of this project (npm run lint produces no warnings/errors)
[ ] I have added tests to cover my changes

This PR now includes all commands under the apps hierarchy.


Password in cleartext was committed into the repo 😮

Checked other resources

[X] I added a very descriptive title to this issue.
[X] I searched the LangChain documentation with the integrated search.
[X] I used the GitHub search to find a similar question and didn't find it.
[X] I am sure that this is a bug in LangChain rather than my code.
[X] The bug is not resolved by updating to the latest stable version of LangChain (or the specific integration package).

Example Code
n/a
Error Message and Stack Trace (if applicable)
No response
Description
Recent commit includes hardcoded username and password for a MongoDB instance.
I was able to connect to it and view the names of the collections.
See CONNECTION_STRING on line 28 here: https://github.com/langchain-ai/langchain/blob/0f45ac4088126746575d40959dabaadd787faeb6/libs/community/tests/integration_tests/vectorstores/test_azure_cosmos_db.py
System Info
n/a

@aayush3011

Thank you @chrisle for the call out. I've updated the master branch, at least.
@aayush3011 I would consider that secret compromised.