variant annotation edge case

Question from user:
Based on sequence ontology stop_gained is a feature_truncation and inframe_insertion is a feature_elognation, could stop_gained and inframe_insertion be together?
o   for example, if we inserted different length of triplets(with a stop codon) at (GAC)(TCT) -INSERTION-(ATT)(TTC)(TGA), some of them ended in a longer peptides while some shorter than the original one. should all of them called stop_gained & inframe_insertion or anything else.
§  chr1       78024346             T              TTAG     DS*
§  chr1       78024346             T              TCATTAG             DSH*
§  chr1       78024346             T              TCATCGTTAG     DSHR*
§  chr1       78024346             T              TCATCGTAGTTAG            DSHRS*
Our definition of stop_gain: A sequence variant whereby at least one base of a codon is changed, resulting in a premature stop codon, leading to a shortened transcript.
This does not cover the case where there is an insertion that includes a stop codon. This seem like we need a new term insertion_stop. I do not think it is a child of stop gained. Any thoughts?

I found this old issue interesting. Is there any update about this topic?
other related questions:

does a frameshift induced stop codon count as stop_gained? for instance, (TCT)(GAC) with an insertion of A between T and C, which turn the sequence to (TAC)(TGA)C
how about this situation? the same (TCT)(GAC) gets an insertion of GATA  between G and A, which turn the sequence to (TCT)(GGA)(TAA)C


Add GitHub links handling

Before you start...

[X] Have you updated the FluentHub? You might be using an old version.
[X] Have you checked if similar issue have already been reported?

Description
This would make the app able to open GitHub links in the app instead of the internet browser
Alternative solutions
No response
Relevant Assets
No response

This feature is one of the most important. I would like to parameterize the URL when I click on a tree or blob item in the repository or when I click on some button and navigate. do so, more confotable when navigate page.

and we need to create hander and parser

We need to discuss how to implement this. I have an idea, but @luandersonn has better than mine, maybe.

We need to discuss how to implement this. I have an idea, but @luandersonn has better than mine, maybe.

UWP has an API for this. https://docs.microsoft.com/en-us/windows/uwp/launch-resume/web-to-app-linking

Yea. good. but problem is how parse the url user gave: when user enter the specific url, we need to discuss how to navigate page

This feature just might conflict with GU11

Yeah, we'll take a look to this in future updates.

Maybe we need a roadmap

In trello or something like that

We can use GH Projects or MS loop but thats still in dev

this feature is developing on multi-account support PR.

@DeveloperWOW64 Why might be conflict?

@DeveloperWOW64 Why might be conflict?

Remember this conversation?

I added it in GU11

Yes. but then how do we implement how to navigate.

Yes. but then how do we implement how to navigate.

Yeah thats the problem

Since the implementation in GU10 is almost complete, would you please stop the implementation of the simplified URL in GU11? I may notice it when implementing that in repository pages.

Since the implementation in GU10 is almost complete, would you please stop the implementation of the simplified URL in GU11? I may notice sth when implementing that in repository pages.

Ooohh thats going to be difficult.
Are you sure GH links handling will fail?
We could make a converter that converts GH links to FH readable links
Url element = actual gh url
displayurl = readable url

There is a great deal of concern about conflicts when General Updates are rife,

Mm yes. What do you meant though when you say 'implememtation in GU10 is almost complete'

Until now, all pages related to the repository code have a ViewModel called RepoContextViewModel that holds the information that the URL has, such as the owner name, repository name, and default branch name, and passes it when navigating.
However, in GU10, I am planning to implement a model called NavigateInfoModel that holds the URL and passes it during navigation.

I thought you abandoned GU10

Yes. but this'll be GU10 https://github.com/FluentHub/FluentHub/pull/127 . that's reason why I asked you to change your branch name.

How to pass arguments when linking or navigating is a very serious problem. I think we need to discuss it very deeply.

How to pass arguments when linking or navigating is a very serious problem. I think we need to discuss it very deeply.

Okay. I can make a discord voice/text/gh discussion for it?

Which one?

text one

text one(cannot speak fluently hahaa)

🙃 https://discord.com/channels/935562861701390336/982985508634366002

On going feature. no end.

On going feature. no end.

@onein528 WDYM?

I created handling methods so completed feature

I created handling methods so completed feature

In GU12?

No, in GU11. please take a look at OnNavigatedTo in DetailsLayoutModePage.xaml.cs

parsing URL.


Can't reach to id attribute

Hi,
I've a problem to reach the id attribute of my model.
We're using an id column to track our records in traditional way. We're also use _id to reach it from the object.
I want to create an object with different _id and id fields but mongoid uses id as _id. I can read it with read_attribute but write is impossible.
For example:
2.0.0p353 :001 > i = Item.last
 => #<Item _id: 528957a99a48ecc1460000fa, id: 122>
2.0.0p353 :002 > i.id
 => "528957a99a48ecc1460000fa"
2.0.0p353 :003 > i._id
 => "528957a99a48ecc1460000fa"
2.0.0p353 :004 > i.read_attribute("id")
 => 122
2.0.0p353 :005 > i.read_attribute("_id")
 => "528957a99a48ecc1460000fa"
2.0.0p353 :006 > i.write_attribute("id", 1)
 => 1
2.0.0p353 :007 > i
 => #<Item _id: 1, id: 122>

How can I set the id attribute?

Since Mongoid is aliasing id to _id it is not possible to have both fields in your document and use Mongoid. This is for compatibility with many Rails based things that expect id but MongoDB uses _id.


[IMPROVEMENT] Reduce replica process's number of allocated ports

Is your improvement request related to a feature? Please describe (👍 if you like this request)
A replica process allocates 15 ports (codes). However, Longhorn doesn't utilize all of these ports even during the replica rebuilding process. Therefore, we can decrease the number of port numbers to 10 or lower.
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

@ChanYiLin Please help verify this. Thanks.

Verified on longhorn master-head 20230612

longhorn master-head 34c07f

follow the test steps
Result
The port range for replica is 10003 - 100012 (which is 10 ports)


[stable/jenkins] Update REQ_URL for jenkins configAutoReload sidecar

What this PR does / why we need it:
The side car container kept giving me this error: POST request sent to http://localhost:8080/reload-configuration-as-code/?casc-reload-token=jenkins-c8d49444f-sg6j4. Response: 404 Not Found
So the configAutoReload wasn't working. When I changed it to the URL in my PR, it worked.
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[X] Chart Version bumped
[ ] Variables are documented in the README.md
[X] Title of the PR starts with chart name (e.g. [stable/mychartname])

@cprovencher can you please sign the DCO?

@cprovencher can you please sign the DCO?

Done!

@cpanato Done!


Compiling build gives an "Class<llua.Lua_helper> has no field sendErrorsToLua" error

Describe your problem here.
I've tried to compile the 0.7.1h source and every time I get this error:
source/psychlua/CallbackHandler.hx:48: characters 18-33 : Class<llua.Lua_helper> has no field sendErrorsToLua
I already did the haxelib remove linc_luajit & haxelib git linc_luajit https://github.com/superpowers04/linc_luajit.git, but It hasn't been solved.
Also, the VS Code shows me the error here:

Is there a solution fo this?
Are you modding a build from source or with Lua?
Source
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
No

Nevermind, I solved it.
I ran again haxelib remove linc_luajit & haxelib git linc_luajit https://github.com/superpowers04/linc_luajit.
Maybe it was a Haxelib artifact

it's a bug with haxelib/git, just do haxelib git linc_luajit https://github.com/superpowers04/linc_luajit master and override the local branch


Color of the form results

For now the results of the form has quite strange Color, brighter would be better for UX.
(of course I mean options which aren't choosen).

Can you edit the screenshot (paint etc) with proposed colors?

I think something like this would be better

Done, voted option will be yellow, other ones bright blue. Will be in release 0.0.15.


Support PostCSS 8

Any update on this PR?

hey, anything i can do to ease / help with the update?

hey, anything i can do to ease / help with the update?

https://github.com/postcss/postcss-custom-media/releases/tag/8.0.0

https://github.com/postcss/postcss-custom-media/releases/tag/8.0.0


Randomly crash after upgrade version from 0.0.6 to 1.0.3 on DictionaryStorageProvider.value.

Describe the bug
According to the crash reports, seems the crash is happen when we use UnleashClientBase.isEnabled or UnleashClientBase.getVariant right after we get the "ready" subscription callback.
public func subscribe(name: String, callback: @escaping () -> Void)
Btw, there is guarantee on our implementation to always use Unleash APIs on main thread.
Steps to reproduce the bug
The reproduce rate is not high, actually, we do not manage to reproduce it by ourselves. However, we are receiving crash reports intermittently after we release our product yesterday.
Expected behavior
No crash.
Logs, error output, etc.
No response
Screenshots

Additional context
No response
Unleash version
1.0.3
Subscription type
None
Hosting type
None
SDK information (language and version)
No response

Thanks. @FredrikOseberg
As mentioned on the issue description, according to the crash reports, seems the crash is happen when we use UnleashClientBase.isEnabled or UnleashClientBase.getVariant right after we get the "ready" subscription callback.
Below is the screenshot which provides an overview about the reports which received so far.

We are now plan to make a small change(see below code snippet) and release it as a patch to see if it helps. If not, we has to rollback to v0.0.6 before you guys have figured out the root cause for the issue.

Thanks. @FredrikOseberg
As mentioned on the issue description, according to the crash reports, seems the crash is happen when we use UnleashClientBase.isEnabled or UnleashClientBase.getVariant right after we get the "ready" subscription callback.
Below is the screenshot which provides an overview about the reports we received so far.

We are now plan to make a small change(see below code snippet) and release it as a patch to see if it helps. If not, we has to rollback to v0.0.6 before you guys figured out the root cause of the issue.

@loongman According to my findings yesterday, this seems to be a concurrency issue that occurs when the SDK has fetched feature toggles from the API and is calling the createFeatureMap function that puts the data into the dictionary at the exact same time as a thread is trying to read from it. I'm looking into possible solutions and should have a fix out this morning. What worked for me yesterday was wrapping the DictionaryProvider with nslock:
public class DictionaryStorageProvider: StorageProvider {
    private var storage: [String: Toggle] = [:]
    private let lock = NSLock()

    public init() {}

    public func set(value: Toggle?, key: String) {
        lock.lock()
        defer { lock.unlock() }
        storage[key] = value
    }

    public func value(key: String) -> Toggle? {
        lock.lock()
        defer { lock.unlock() }
        return storage[key]
    }

    public func clear() {
        lock.lock()
        defer { lock.unlock() }
        storage = [:]
    }
}

It's possible in the SDK to input your own poller implementation and thus pass the poller your own storageprovider, so if you don't have time for me to get the fix out this morning that could be an option. I'm currently evaluating if this is the best approach or if I should just create a new dict when the SDK receives new data and replace the current one after I put all the data into it. This will be my top priority until the fix is out. I'll keep you posted.

@loongman I've released a new version that should resolve this issue. Can you try version 1.1.1?

@loongman Fantastic. I'm glad it's resolved!


Feign doesn't support regexp in request mapping

If I use interface
public interface UserAPI {
  @RequestMapping(method = GET, path =<EMAIL_ADDRESS>  UserDTO getUserByUsername();
}

And want to create client as
@FeignClient("service")
public interface UserClient extends UserAPI{}

it won't work because feign thinks that pattern is part of path. It's even more impressive if you want to use pattern like .+

Any timeframe on this being included in an upcoming release?

@jmgilmour no. PRs welcome.

Thanks for the quick response.  I was under the impression that the referenced commit by @venilnoronha  above would fix this issue.

That commit merely didn't error if there was a regex, it didn't support them.

Good to know :smile:.  I guess I'll take a crack at a fix and PR when I get the chance.  Again, thanks for the quick response.


Remove clusterimagesets clusterrole and crb

Does not require cleanup because it's moving ownership to MCE in https://github.com/stolostron/backplane-operator/pull/366
Issue: https://issues.redhat.com/browse/ACM-2203
Signed-off-by: Jakob Gray<EMAIL_ADDRESS>

/lgtm

/lgtm


[stub] upstream: add fast path wildcard impersonation check for large number of groups and extra fields

This would allow us to skip the inline for loop that checks for impersonation permissions.

Reviewed on 9/17


Dart List doesn't report the correct value for getOwnpropertyDescriptor in dartium 45

Repro:
var list = ["a", "b"];
var descriptor = Object.getOwnPropertyDescriptor(list, 0);
// TODO(jacobr): re-enable th
//expect(descriptor.enumerable, isTrue);
// expect(descriptor.configurable, isTrue);
This is relatively harmless

Using:
var object = context['Object'];
var descriptor = object.callMethod("getOwnPropertyDescriptor", [list, 0]);
descriptor.configurable is True
descriptor.enumerable is False.
descriptor = object.callMethod('getOwnPropertyDescriptor', [descriptor["value"], 0])
descriptor.enumerable is True
descriptor.configurable is False
I don't think we're going to do anything more in Dartium but what should be done in dart:html/dart2js should Object be exposed with static methods e.g., getOwnPropertyDescriptor?


removed LoaderChain stuff

Ticket: #1178

Coverage decreased (-0.009%) to 90.205% when pulling 29c23a8d371a0fbf46e7a17e1ba64124a614d4ed on 1178_gantry_fix into c1b75ce612eabb3e5586fde62762c12eb706785c on master.

Coverage decreased (-0.009%) to 90.205% when pulling 29c23a8d371a0fbf46e7a17e1ba64124a614d4ed on 1178_gantry_fix into c1b75ce612eabb3e5586fde62762c12eb706785c on master.


TASK: Adjustments to Neos 5+

@daniellienert Do you know of any issues with this PR or is it generally good to go?

I tested this change extensively with a Neos 5.1 demo and after my adjustments it seems to work correctly.

@hlubek: Ah completely forgot about that. We are using the package event without your changes in production for quite some time now. So I would just tag it 4.0.0


#11: must_be/2 further errors

Item#11

Sticking with this (as does Scryer)
?- L=[1|L], must_be(chars,L).
error(type_error(list,[1|_1]),must_be/2).
for now.

Right, you have the choice here. 7.12 last par


301 redirect when route ends with slash

It would be nice (if possible) to issue 301 redirect if path ends with forward slash. For example with this router: router.Get("/test", testHandler) request to /test/ results in 404, but it would be nicer if the response code was 301 with path to /test

I would be up for having a function, such as vestigo.RedirectTrailingSlash() where users can opt in.  I have had trouble with gorilla mux in the past with the trailing slash option they have being confusing.  If you want the same handler being called for trailing slash/non trailing slash you could add a function like this to your code:
func TrailingSlashOrNot(method, urlTemplate string, f http.HandlerFunc) {
    vestigo.Add(method, urlTemplate, f)
    vestigo.Add(method, utlTemplate+"/", f)
}

In my experience this type of thing gets very confusing, as many times the "/" at the end has a completely different meaning in an API, as it is usually a different resource.
Would love to hear more about this.

I was thinking of having something like this:

router
request
response

/test
/test
200

/test/
301 /test

router
request
response

/test/
/test
301 /test/

/test/
200

router
request
response

/test
/test
200

/test/
/test/
200

so the redirect would only happen if the difference is only in the ending slash. Of course it would make sense (as you pointed out) to make configurable, something like defaultRedirect = true
I don't think that it would be confusing, even default go http package uses redirects - https://golang.org/pkg/net/http/#ServeMux But this is just my view/opinion.

Seems like this would work well as an option that could be applied to the router itself at the user's discretion.


Fix bug introduced by BDP/2i merge: create_plan/{5,6}

When merging the BDP/2i code I simplified the call to the colocated create_plan from 6-arity to 5, but neglected to make the corresponding change in riak_kv.

+1 9b383d5da49f9883e3278c2ad8d136bff461c213

@borshop merge


travis-ci: Add a specific stable version of rust

Motivation: https://www.reddit.com/r/rust/comments/8ik620/notes_on_impl_trait/dysyg1n/

Wait. I'm making an app, not a library - what's the point of using anything but the latest stable release? Closing.


Use signon for user authentication

Trello: https://trello.com/c/SEYqW8Cg/232-8-move-app-behind-signon
There are a couple of a dependencies, as there are a couple of new secrets that need to be specified as environment variables [1].
1: https://github.com/alphagov/content-performance-manager/blob/use-signon-for-user-authentication/config/initializers/gds-sso.rb#L3-L4
Dependencies:

[x] https://github.digital.cabinet-office.gov.uk/gds/deployment/pull/1326
[x] https://github.com/alphagov/govuk-puppet/pull/5917

I've updated the last commit to use factories for the users in the feature tests @pmanrubia .


Multiple letters propositions

Currently, we have 2 letters combinations for the given name.
If we take my username as example, there is:

forward: Q U EN G
backward: Q U E NG

It should be great to have a page selection, and on each page there will be each letters combinations.
For example (with my username and forward):

Page 1: Q U EN G
Page 2: Q U E N G

That would be a worthy feature as well. Will try to implement that in the upcoming days.

Done in #21 (and fix in #24 )


Allow interface type definition to hint about interface implementors

Now when interface implementor is not referenced anywhere in schema it remains invisible for GraphQL during validation phase.
As a workaround we allow types option on Schema to pass such implementors directly. But a better way for this could be adding implementors option to interface type definition that could act as a hint for schema.
This option will accept an array or thunk of types (similar to fields and interfaces options on ObjectType).
This mechanism will have advantages over current types option on Schema only when type definitions are initialized on demand (vs collected during schema creation as it is done now).
See #38 for more info

After more thinking: this is bad idea. Looking for better ways to solve original issue.


Feature/zero config setup

Still a WIP, opening a PR so this is all tracked.
Updated setup process for boilerplate that should be as simple and intuitive as possible while working on all platforms.
Currently it's 1 script (npm run setup) for setting everything up, after which user can follow the manual WordPress wizard install (by going to website's local url) or try the automatic setup depending on his local dev setup.
Would appreciate any feedback on intuitiveness of this.
Currently done

Reworked the setup.sh script so that it's now done in JS (no more bash)
rename.jsnow works no matter how many times you run it (saves old state to theme-manifest.json)
Added a default theme-manifest.json with initial values we replace using rename.js

To do

Output user's pass when creating him
Add some validation to user input (for example: only allow lowercase and hyphens for package name, etc)
Make the installation pretty ( chalk, some character art, emojis and removing unnecessary output )
Test on Windows (doesn't use bash anymore so it should be all good, also running composer using npx)
Create a Vagrant setup script

Changes in last push

Added the WP install part after the initial setup (wp core install, wp config create and wp theme activate) depending on the user's local dev environment
It's now possible to have a working install (with DB and everything) with just 1 call npm run setup on VVV (still need to implement the wp installation part for other local setup - as it is now it just tells you to do it manually by going to your local website url and you will see the WP's 5-min wizard setup)
Reworked the update / renaming functions / scripts so they're all async(mostly just so they work with ora)
Reworked the rename script completely with new styling

To Do

Add "Create database if it doesn't exist" part before wp config create
Add some basic compatibility check for setups other than VVV (for example check if we're able to connect to mysql using "best guess" credentials for a particular setup - already using this with VVV - If not, prompt user and check again)
Skip setup steps if they're not needed (for example wp core download, as it is now if wp core exists the script will exit with an error - ideally it would just skip to the next part)
Check if search / replace is working with hidden files

Fixed all the issues. Also @iruzevic can you just look at the build.sh/ deploy.shin README (this commit) if I've changed it correctly.
Other than that I've separated the renamescript (npm run rename) so it can be run whenever. Works as long as info in theme-manifest.json (needles) is correct. Added a warning regarding that before prompting.
I think it could be merged , what do you think? @iruzevic @dingo-d
There are still a few things on my to-do regarding the setup (such as automatic wp install setup for more environments other than VVV but these can be implemented later.

Few minor things that are bothering my OCD (please don't hate me 😆)
When you start the setup, the icons are glued to the text, also the cursor is 1 space away when typing which seems kinda odd like I wrote something wrong 😬

Also I tried to create a vagrant install and failed

Now there seems to be some discrepancies between the text in the instructions and what happened.
You wrote Please enter database name (make sure it exists and is empty):
I had a db in the vagrant and i emptied it, but I got an error that said

Can't create database 'wordpress_internal'; database exists

So, something is a miss here 🙂
Also, from a11y point of view this seems a bit unreadable, and could provide a problem to the people who have impaired vision.

Granted, this could be only for my setup, which is totally not a11y friendly. I presume that people who have such problems would have high contrast terminal settings.
This is not a blocker for a release imo, but it would be great if we would make some kind of call to action for users to test, especially if they are visually impaired 🙂
All in all I love this and if these small issues are fixed, I'm for merging this to develop, testing and releasing the v3 :tada:
Awesome work!

Changes since your last review:

Separated the wp-setup part (npm run setup-wp)

Added a Custom local development server setup. Not really tested since I don't have such setup but generally should work. (should be used only for custom setups like @iruzevic has, for example it probably won't work on MAMP due to. Will implement new options for MAMP, XAMPP, WAMP, etc down the line)

Step 1: wp config create ...
Step 2: wp core install  ...
Step 3: npm run build
Step 4: wp theme activate

Renaming @since in rename script

Fixed the wp db create(now it tries doing it, but doesn't throw error if it fails). Also changed the loader text to better indicate what's happening

"Fixed" the renaming of hiden files

public_html is no longer automatically appended to vm_dir in VVV setup so it can be used in other Vagrant boxes (such as Scotch). Also renamed that option label from Varying Vagrant Vagrants to Vagrant (VVV, Scotch, etc)

Changes from feature/in-theme have been successfully integrated with the current setup process and this should be more or less ready to go.
Changelog

Tweaked the current setup script to work when everything is moved to theme
Cut back all unneeded dependencies from project's package.json (in public_html)
Added an export script (zips up the theme without node-modules or vendors folders - npm run export from theme folder)
Added atheme-setup script (npm run theme-setup`) which you should run after uploading the theme to a website (but before activating it). The script does the following:

npm install
It prompts you for your dev-url (for browsersync) and changes the const proxyUrl in webpack.config.js
composer install
npm run build

To Do

Test with other setups (non-vagrant)
Test theme export / import on Windows

To Do as of today:

Test / fix linting - current doesn't seem to work out of the box
Add some more "look why this is awesome" content to documentation
Figure out naming (next meeting)
Prepare the theme for WordPress.org (readme.txt, better description, screenshot, etc)
Issue - Currently when you try to delete theme from WP Admin it takes a long time and then fails with "Undefined" error - need to look into this

OMFG the readme! ❤ ❤ 😄
I'd only change one thing: the aliasing for the phpcs. We should add to composer.json the scripts that look like this (the alias can be changed):
"scripts" : {
  "check-cs": "@php ./vendor/bin/phpcs --extensions=php --ignore=/vendor/",
  "fix-cs": "@php ./vendor/bin/phpcbf --extensions=php --ignore=/vendor/",
}

That way you can just run (from where the composer is setup for development)
composer check-cs .

Or specify the directory, and you'll have linting that's fully working. Aliasing can cause some issues, especially if you have globally installed phpcs- the path won't be pulled from the root of your project but the global installation and in case you are not keeping it up to date that can be an issue, also the phpcs.xml.dist won't be picked up from the root of your project.
Other than that really awesome work on the readme, looks sick 😄 :+1:

Tested it and seems to work great, I'll all for it! Added the scripts and changed the readme to reflect the changes.
Regarding the readme, still need to add the updated Boilerplate graphics when I get them.


How to get coordinates and confidence information from yolo 8 model

The above is the model properties, there should be one float32 output. However, after executing model.runSync([data]). The output is

I have no idea about how to get the detection result from this model. Anyone can help?

Sorry but this is not really related to the issue tracker here. I don't know that model, I don't use that model, so I can't help you.
Ask in the yolo support channel

I have the same question, if anyone has figured it out, please write.


✨ Release memory modifier

I wonder where should we put this.
modifier releaseMemory {
    uint256 m;
    /// @solidity memory-safe-assembly
    assembly {
        m := mload(0x40)
    }
    _;
    /// @solidity memory-safe-assembly
    assembly {
        mstore(0x40, m)
    }
}

I think we put this where we use 0x40 memory for calculation purpose. Is that gas optimized?

This is useful for those functions where you do a lot of abi.encode to compute a hash.

I check this with ECDSA lib. Unfortunately it will increase gas cost and codesize.
Result: forge s --diff

It’s not for everything. Only very specific functions.

It’s not for everything. Only very specific functions.

Yup. But it will increase slightly gas cost and codesize.

I'll make an example to demonstrate when I have more energy, motivation, and time.

I think this operator may be accidentally abused to cause issues. Even amongst experts.


HSEARCH-4255 Test Hibernate Search against JDK 18

https://hibernate.atlassian.net/browse/HSEARCH-4255

Build on JDK 18: https://ci.hibernate.org/blue/organizations/jenkins/hibernate-search/detail/PR-2610/2/pipeline

Looks good.
I run a full-build: https://ci.hibernate.org/blue/organizations/jenkins/hibernate-search/detail/PR-2610/3/pipeline.
Feel free to merge it when it passes.

I run a full-build: https://ci.hibernate.org/blue/organizations/jenkins/hibernate-search/detail/PR-2610/3/pipeline.

That's not necessary, as explained above I already ran a build on JDK 18: https://ci.hibernate.org/blue/organizations/jenkins/hibernate-search/detail/PR-2610/2/pipeline/ And other environments are not affected.
Merging, thanks.


Only apply tax to some products

Is it possible to only apply tax to some products in the catalog?  Which modifier should i look at using for that?
GST in Australia does not apply to food items for eg.

You can write your own Modifier for that. I guess you'd have to add some sort of Type field to a product to determine whether or not tax (or which rate) should be applied.

oh. ok thanks.

Hi @asecondwill
I have made a gist here doing somewhat like that: https://gist.github.com/hpeide/fee33737ad5efa5806396e5f6a91b87c
It do not work that well combined with the Discount module, so if you stumble on a good solution for that, please post! :-)

hi @hpeide  - thanks for this.  very helpful.
I think the discount module issue is because you are going back to the product and calculating the price.
Maybe if we can somehow use the attribute on the product to influence the $incoming value directly just for tax it won't mess up anything else.

I don't think you need to rely on the $incoming value. Just ignore it and calculate tax based on order contents (accessible via $this->Order())

ok.  does $this->Order() contain shipping and discount stuff, or just product line items?

answer: no.  so how to avoid losing the shipping and discount calculations.  tricky.

actually.  we need to consider what the discount has been applied to.

There is some abandonware that has some crossover.
https://github.com/burnbright/silverstripe-shop-taxframework

@jedateach  - do you have any advice on this topic?

https://github.com/burnbright/silverstripe-shop-taxframework/pull/7
made an attempt at getting tax Framework to work in 3.5

That module you linked to doesn't do things differently than I suggested. It doesn't take discounts into consideration either…
The architecture with Order-Modifiers only works well for certain scenarios. Ideally, an order-modifier is a self-contained entity that can perform its task without knowledge of other modifiers. As soon as you have modifiers that operate on the same items as others do, without knowledge of the changes made by another modifier, then you'll run into problems.
So the scenario where you have a tax per product-type combined with coupons can get problematic (because your tax doesn't operate on an incoming total, but calculates from base-prices instead).
If you have a good idea for a better architecture, I'm all ears. I think this should be some sort of RFC-style document/issue where the new architecture can be discussed.

That module you linked to doesn't do things differently than I suggested. It doesn't take discounts into consideration either…

Yep.  i realise that.  good base though to build on.

As soon as you have modifiers that operate on the same items as others do, without knowledge of the changes made by another modifier, then you'll run into problems.

Agreed.  That's discounts for you.  i'm not even sure i can get my head round exactly what i'd want to happen.  possibly it changes depending on the law / type of discount etc.  it's probably epicly hard.

If you have a good idea for a better architecture, I'm all ears

i'm not that clever.  I didn't mean to imply i thought it was all wrong.  sorry if my ticket came off as rude.

I'm reopening this, so that this discussion isn't getting lost and can be considered when there are architectural changes about to happen.
@asecondwill You might still be able to make it work by rolling your own Tax-Modifier. If you install the discounts-module, there are several ways to figure out the discounted amount (either per product, or total). You would then have to incorporate these amounts into your tax-calculation. I guess in the end it really depends on what kind of discounts you need.

there are several ways to figure out the discounted amount (either per product, or total)

Any further hints there? this is the bit im struggling with.


Trying to request profile/getUserInfo asynchronously in v10

Here is my current setup after a couple days of determining auth0-lock 10.11.0 requires auth0.js:
      lock.on('authenticated', function (authResult) {
            store.set('id_token', authResult.idToken);
            lock.getProfile(authResult.idToken, function (error, profile) {
                if (error) {
                    return console.log(error);
                }
                store.set('profile', profile);
            });
            authManager.authenticate();
            ...
        });

Since version 9, both token and profile were transmitted in the same request.

Hi @alobban
The getProfile method from previous versions of Lock has been replaced with getUserInfo. When passed on access_token, it will return information for the user. This is the implementation in the 03-User-Profile sample:
var accessToken = localStorage.getItem('access_token');
lock.getUserInfo(accessToken, function(err, profile) {
  // handle profile
});

You will of course need an access_token to begin with. It comes in the authResult when you authenticate a user.
lock.on('authenticated', function(authResult) {
  console.log(authResult); // accessToken will be a key here
  localStorage.setItem('access_token', authResult.accessToken);
}

From our other conversation, it sounds like you're looking for a way to delay rendering your profile view until the profile call completes and you have an object to work with. Can you outline the behavior you're looking for in more detail so I can provide the best answer?
Thanks

@chenkie,
So the behavior that I am exhibiting is that upon login, I receive the token and my home page is updated as an authenticated session but the profile information isn't rendered because the controller did not receive the profile information. Hence no name or profile image was rendered.

Versus my workaround to get the designed outcome:

Instead of a $timeout you can use $q to set up a promise. This sample shows how to do that. Essentially you want to set up a promise for your profile in your AuthService and then later get it out using the promise's then method in the spot that you need it. In this case it looks like that spot is the main route.
Have a look specifically at the AuthService and the home controller in the above sample to see how this is implemented.

Awesome, I was just researching Promises on youtube and how to handle them in Async Callbacks.
Thanks @chenkie

awesome :)
I'll close this for now but please let me know how it goes with your implementation.


Create a team cargo account

For the github action, we're currently using a personal token to release krustlet to crates.io. We should create a team account and use that instead.

This can be done with cargo owner --add github:deislabs:<teamname> once we publish the 0.2 crate

Oh, if we want to use a bot for it (instead of someone's credentials), we'd need to create a GH bot that we can log in to crates with and add to the team

Added the krustlet-maintainers group as an owner. We can add a bot down the line


Docs: Replace DocCardLists with DocLinkLists

This PR utilizes the DocLinkLists added in https://github.com/grafana/plugin-tools/pull/493 to the site's index pages. Add two index pages for develop-a-plugin and extend-a-plugin.

how does it look for /migration-guides which has a mixture of folders and pages?

Right now it looks like the screenshot below which is a little misleading as it suggests the update create-plugin versions and migrate from toolkit are angular related. I can separate them in someway, just not entirely sure how off the top of my head. 😄

Figured that might be the case 😄 hmm, personally I would just go the titles and use of custom metadata for now. I'm not convinced its worth worrying too much over.

link to first item in category

To work around having to pick we can always just add the _index file with DocLinkList for the ones which lack it:

upgrade from grafana versions
develop a plugin
extend a plugin

hmm, personally I would just go the titles and use of custom metadata to indicate its a collection of content for now. I'm not convinced its worth worrying too much over.

Just to confirm, we'd be happy to add the customProps.description to each _category_.json and for it to look like this:

I'm happy with that

we'd be happy to add the customProps.description to each _category_.json and for it to look like this:

looks good

@josmperez I've pushed a commit to this branch that updates the DocLinkList component as we've discussed above. To add descriptions in category links you'll need to update the _category_.json files with customProps.description. Example below:
{
  "position": 1,
  "label": "Develop a plugin",
+  "customProps": {
+    "description": "Category description text."
+  },
  "collapsible": true,
  "collapsed": true
}


Using an orphan instance of GoogleMapsAPIWrapper should raise an exception

Issue description
Took me a solid day (as an admitted angular noob) to figure out I need a custom component inside the sebm-google-maps component, without specifying the GoogleMapsAPIWrapper as a provider on the component itself. There's more than a few issues that reference this, and it describes the solution, but it's pretty hard for a newbie to work out. Also this seems to undercut the basic idea of the promise returned by this method, since neither resolve or reject are ever called.
Steps to reproduce and a minimal demo of the problem
Sorry no details here .. just attempt to inject and use a GoogleMapsAPIWrapper outside the context of a sebm-google-maps component, or inside one but having overridden the injected provider.
Current behavior
All methods on the GoogleMapsAPIWrapper never resolve/reject the promise - nothing is called.
Expected/desired behavior
It should throw an exception immediately, or at least reject the promise.

Hey @fearlsgroove can you give a simple example of what you did to solve this as I am having the same issue?


added a 'timer-set-startTime' listener

added a watcher to listen on 'timer-set-startTime' events. Before there was the problem that when you used the start-time attribute the value could not be changed after compilation. you had to compile the whole timer object again. this method should fix this problem by setting the startTimeAttr manually
#88 is a similar problem. you postet the plunkr using $compile but that is not realy the nice way is it? ;)
#44 is a similiar problem as well. in that case the problem was the countdown value.
if you find another way to handle this problem please tell me

Sorry to bring an old issue back - but I'm having trouble implementing the start time via a $scope variable.   Can you please further explain what code needs to be used here to set the timers start time and the continue to count up from that number.
Any help is appreciated!


Fix searchTags and bad dimension name

This commit resolves https://github.com/nerdswords/yet-another-cloudwatch-exporter/issues/365.
In cases in which the name of the dimensions for a given metric didn't
match a dimension in our searchTags list, we would pass a default global query
for all metrics under a given namespace into our list, returning metrics
we meant to filter. A test case was added to cover.
In addition, I decorated a debug log entry that was useful in finding
this bug.

hello again @cristiangreco! no rush on this, but I wanted to get a sense of when you'd be free to review and ship a release! we're waiting on the fix internally so I wanted to get a gauge on if I should ship the patch on a forked copy until there's more time.


more distributed architecture

see unity's ngo and unreal's iris. the reason is server is too expensive(heavy), and the game experience is broken even on the fast client(gaming pc) if the server is slow.

not for now


jsbin editor / script runner fails if you have a string containing '' text inside

Code execution fails if you have a javascript code with a string containing ""
Example
https://jsbin.com/tukumopose/edit?html,css,console,output
I found similar problem with jQuery replaceWith function and was trying to create a test case in jsbin - but it appears jsbin has similar problem

maybe it's not a bug, just the incorrect use of unescaped html text inside a string?


deps: update to mocha@10

This migrates library to mocha@10 and exposes mjs and cjs bundles explicitly.

Merged via https://github.com/bpmn-io/element-templates-validator/pull/24


Contact Page

The contact page is not properly aligned it can have better spacing and have spaces

@tushargupta1504 Please use the proper issue templates to raise your issue. I am closing this issue as #178 is already present there


User authentication in NextJs 13.

I created a blog using sanity.
Now I want to create a user registration and like system (user can like a post and it will be saved). Can I do it with sanity?
I thought to create a schema for user, and then just add the liked posts to the array.

Hey @Php93, this is possible to do with Sanity and I suggest you ask this in our Community Slack: https://slack.sanity.io/ so that one of our Community Engineers can guide you through it :relaxedL
The team that monitors issues on this repo don't have capacity to handle general questions and focuses on feature requests and bug reports directly related to the next-sanity library itself. Our community Slack have a dedicated #next channel that provide you with a ton of resources and an active community that can help you build just about anything with the Next.js + Sanity stack 😉


Package management: Working with Nuget Packages

Issue tracking new topic at http://dotnet.github.io/docs/packaging/working-with-nuget-packages.html

Same, so can we just link to http://docs.nuget.org/? The only special thing would/could be the versions.

So we just agreed to have this one topic, including the lilnk to nuget docs. Other Nuget work items can be postponed (post Build).

Closing this issue, will use #306 to track the overall NuGet doc mentioned in #306.


Filter by location is broken

Describe the bug
Probably related to #526
Tested locally and on QA (on prod it's ok)
Tried to get gtfs feeds from Canada:
curl -X 'GET' \
  'https://api-qa.mobilitydatabase.org/v1/gtfs_feeds?limit=10&offset=0&country_code=CA' \
  -H 'accept: application/json' \
  -H 'Authorization: Bearer your_token_here'

Result from QA (clearly has nothing to do with Canada):
[
  {
    "id": "mdb-195",
    "data_type": "gtfs",
    "status": "active",
    "external_ids": [
      {
        "external_id": "195",
        "source": "mdb"
      }
    ],
    "provider": "10-15 Transit",
    "feed_name": "",
    "note": "",
    "feed_contact_email"<EMAIL_ADDRESS>    "source_info": {
      "producer_url": "http://data.trilliumtransit.com/gtfs/10-15transit-ia-us/10-15transit-ia-us.zip",
      "authentication_type": 0,
      "authentication_info_url": "",
      "api_key_parameter_name": "",
      "license_url": ""
    },
    "redirects": [],
    "locations": [
      {
        "country_code": "US",
        "subdivision_name": "Iowa",
        "municipality": "Ottumwa"
      }
    ],
    "latest_dataset": {
      "id": "mdb-195-202402071841",
      "hosted_url": "https://storage.googleapis.com/mobilitydata-datasets-qa/mdb-195/mdb-195-202402071841/mdb-195-202402071841.zip",
      "bounding_box": null,
      "downloaded_at": "2024-02-07T18:08:42.045274Z",
      "hash": "ca38f60b18c9fae64ecc4ec6a2e5df8a687dd9985a87d75184d6f2ec90a4b7a0",
      "validation_report": {
        "total_error": 0,
        "total_warning": 10,
        "total_info": 47,
        "unique_error_count": 0,
        "unique_warning_count": 4,
        "unique_info_count": 2
      }
    }
  }
]

Just to make sure, I get 10 gtfs feeds without filtering:
curl -X 'GET' \
  'https://api-qa.mobilitydatabase.org/v1/gtfs_feeds?limit=10&offset=0' \
  -H 'accept: application/json' \
  -H 'Authorization: Bearer your_token_here'

And it got me 10 feeds:

Steps/Code to Reproduce
See above
Not that is also happens with subdivision_name. Did not try municipality.
Expected Results
Up to 10 gtfs feed with CA as a country code.
Actual Results
See above
Screenshots
No response
Files used
No response
Additional notes
No response

@jcpitre @cka-y is the plan with this to basically re-test and see if the bug is present now that #545 has been fixed?

For the described query, I get this response, which includes 10 feeds with "CA" as the country code.
I think we can close this issue if @jcpitre's subdivision_name tests work as well.

I will close this one once tests are added in #479


Fix shortcut to Contact

add space above and below
fix position of sent message

https://github.com/reallyluis/reallyluis/pull/33


Implement jnp.tril_indices, jnp.triu_indices etc. in JAX.

Implement jnp.tril_indices etc. using XLA primitives instead of wrapping numpy implementations, so we don't materialise their output while staging.

@jakevdp jnp.tril_indices currently has the wrong signature, as k defaults to 0. You seem to miss my comment here.


add 'create_before_destroy' and name_prefix to the launch config

This will allow changes to the LC to create a new LC and attach it to the ASG prior to deleting the old LC (since LCs can't be updated). This will leave the currently running instances intact

@mikeurbanski1 good catch. Are there any other resource types where you're aware this is an issue?

Late now, but yeah, these were the only typos I saw. It was noticeable when I copied in the example.


A11y_Azure Data Studio_Query Plan (Preview)_Keyboard: Tooltip provided at each not is not accessible with keyboard.

“Check out Accessibility Insights! - Identify accessibility bugs before check-in and make bug fixing faster and easier.”
GitHubTags:
#A11yMAS; #A11yTCS; #A11ySev2; #SQL Azure Data Studio; #SH-SQL Azure Data Studio-Win32-OCT2022; #DesktopApp; #Win11 #Keyboard; #WCAG2.1.1; #FTP;
Environment Details:
Application Name: Azure Data Studio
Windows Version: Windows 11
Repro Steps:

Launch the "Azure data studio" application.
Connect with the provided SQL server.
TAB to "New Query" button and hit ENTER key.
Enter the SQL query and TAB to "Estimated Plan" button and hit ENTER key.
"Query Plan (Preview) screen appear.
Observe that tooltip provided at each not is not accessible with keyboard.

Actual:
Tooltip provided at each not is not accessible with keyboard.
NOTE: The issue is reproducing throughout the application wherever tooltip is present.
Expected Result:
Tooltip provided at each node should be accessible with keyboard
Attachment:

@poojapatil36
dupe of: https://github.com/microsoft/azuredatastudio/issues/19566
we've already have a sev3 issue for AzureDataStudio.
please close the issue.

Issue is fixed in latest insider build. Please find below attachment for reference.

Github Tags: #Closed;


Question about running slalom app

Hi!
When I run the "run_all.sh" it keeps saying the error below
tensorflow.python.framework.errors_impl.NotFoundError: ../imagenet; No such file or directory
It seems like a file called "imagenet" should have been in the "python/slalom" directory
Could you please help me figure out what that file is?
Thanks in advance!

That's the path to the imagenet dataset which you have to download first.

Thanks for the response!

I am sorry to ask you more questions but can I ask you what Imagenet data did you use exactly?
What would be the data type (is it a bunch of raw image files)?
Thanks!

Never mind! I figured it out.


Toolbar navigation should also be possible with up / down

@isidorn
More of a suggestion:
When navigating within toolbars, left / right move among items. However, if the item is a text input (like a filter box), arrow keys suddenly navigate within that item's text as they should. This makes context a bit difficult to discern and how to get back to the toolbar in which you were navigating.
Perhaps up / down should navigate toolbars since they have no special meaning within text input fields (unless the text input is multiline which is probably not too often encountered in a toolbar context).
This may confuse some users, so perhaps make it a setting?
Also might want to consider a separate screen reader or a11y or accessibility settings category?

Using up/down to navigate toolbars was already discussed, however there were some issues since some toolbars already use up/down for other functionality, so we could not do it in a consistent way.
Regarding left/right that gets trapped in input fields, that was also discussed and we landed on the tab has to be used to focus out.
So for now I would not change anything here, however keeping this open for discussion.
We try not to introduce any a11y settings unless they are really needed, in case we introduce enough we will definitely have a category for this.
For reference here are accessibility docs for vscode https://code.visualstudio.com/docs/editor/accessibility


Add Reference to Ansible "degoss" Role

This project is an Ansible Role which installs, runs, and removes Goss and test cases.

Curious on how this differs from goss-ansible?

Basically, goss-ansible is a simple Ansible module that allows you to intelligently run Goss from Ansible.
degoss is an Ansible role which includes goss-ansible and does the following in order:

create a temporary directory
install a version (or the latest version) of Goss into that temporary directory
copy a Goss spec file or series of spec files into that temporary directory
run Goss on all of those spec files
if an error occurs, pretty print the output
remove Goss and any traces that anything was run

My personal use-case which prompted the development of this was needing to test an AMI without leaving traces on it during the build. I have Packer run Ansible to install and configure the machine, then run degoss to validate it passes all tests. Once finished, there is no trace of Goss or the spec files left on the machine anywhere.

Makes sense, thanks!


Rename file module: file move

[ ] rename file module (this is a big pain and the reason why we added ///ts:import transform in the first place)

Completely agree. Is this in the plans?

Completely agree. Is this in the plans?

We already have the refactoring code working (https://github.com/TypeStrong/atom-typescript/blob/ac11fe7323051f94bfac95a14805ead897a4efda/lib/main/atom/commands/commands.ts#L181-L213) The only issue is getting notified from atom tree view -> https://github.com/atom/tree-view/issues/433

This would be sweet. Can we keep this open?

Please open a new issue lest I miss this in the backlog.


huge long time to generate trainval GT

Thanks for your work.
But I find that it takes huge long time to generate trainval GT even with parallel generating. Also, it takes almost all CPU threads.
Is that same with your past practice?
Thanks!

Yes, non-keyframe point clouds lack labels, requiring the use of nearest neighbor methods to assign labels to unlabeled points, which can be a time-consuming task.

Thanks for your reply. Yes, it takes 96 CPU servers days.   BTW, what’s the difference with CVPR 2023 occupancy GT ? Is that the same method to generate GT as yours?
Also, the recent OpenOccupancy repo releases it’s code as well for nuscenes.  What’s the difference with your GT?
Thanks!

Yes, this speed is normal. The specific content of OpenOccupancy can refer to their papers.

hello, could you please release the generated occupancy data?


Sumatory problem solved with JavaScript.

Hacktoberfest! 🎊🎈
🎉 Have you read the Contributing Guidelines? 🤔
Yeah, read it.
📝 Description
The problem is how to solve a sumatory from 10 to 5, using a for loop. This is the way I was taught how to work with for loops.
The solution is a simple:
let sum = 0;
for (let i = 10; i <= 5; i++) {
sum += i;
}
console.log(sum);
Fixes #issue_no 🔧

✅ Checklist

[ y] I've read the contribution guidelines. 📚
[ y] I've checked the issue list before deciding what to submit. 🔍
[y ] I've edited the README.md and linked to my code. 📄

🔗 Related Issues or Pull Requests
(Write your answer here.)

Hi there!
Although this repo is not part of Hacktoberfest 2024, we have some great alternatives for you. Welcome to SkillShow, a repository focused on skill enhancement!
Guidelines:

Please adhere to the existing guidelines.
No additional rules, thanks!
Check it out here: SkillShow.

Note: Start by creating an issue. Once it’s assigned to you, feel free to open a pull request (PR). Directly opening a PR will lead to an invalid label.
Looking forward to your contributions!


React Native modal does not work with snackbar

I am using react native modal with the react-native-snack-bar https://github.com/cooperka/react-native-snackbar, i have a scenario in which when user perform any action in the modal i need to hide the modal and show the snackbar, problem which i am facing is that when i hide the modal snackbar also hides. i have also tried to show the snackbar in onModalHide event but still modal and snackbar both hide
onModalHide(){     Snackbar.show({       title: 'Hello world',       duration: Snackbar.LENGTH_SHORT     });  }
only workaround which is working for me is to show the snackbar after 1-2 seconds with time out
setTimeout(this.showSnackbar.bind(null, index),1000);
Is there any way i can show the snackbar after closing the modal with using timeout

Hey @FahadRS ! Could you try swapping replacing react-native-modal with react-native's own Modal component and see if the issue is still there?


[Bug][Web]: Cannot build web when package_info_plus is in pubspec.yaml

Platform
Web
Plugin
package_info_plus
Version
1.4.3
Flutter SDK
3.0.5
Steps to reproduce

Add package_info_plus: ^1.4.3 to dependencies in pubspec.yaml
flutter build web

Code Sample
name: testpip
description: A new Flutter project.

publish_to: 'none' # Remove this line if you wish to publish to pub.dev

version: 1.0.0+1

environment:
  sdk: ">=2.17.6 <3.0.0"

dependencies:
  flutter:
    sdk: flutter
  package_info_plus: ^1.4.3
  cupertino_icons: ^1.0.2

dev_dependencies:
  flutter_test:
    sdk: flutter

  flutter_lints: ^2.0.0

flutter:

  uses-material-design: true

Logs
Building with sound null safety 💪

Target dart2js failed: Exception: Warning: The 'dart2js' entrypoint script is deprecated, please use 'dart compile js' instead.
/opt/flutter/.pub-cache/hosted/pub.dartlang.org/http-0.13.4/lib/src/browser_client.dart:31:18:
Error: 'HttpRequest' isn't a type.
  final _xhrs = <HttpRequest>{};
                 ^^^^^^^^^^^
/opt/flutter/.pub-cache/hosted/pub.dartlang.org/http-0.13.4/lib/src/browser_client.dart:43:15:
Error: The method 'HttpRequest' isn't defined for the class 'BrowserClient'.
 - 'BrowserClient' is from 'package:http/src/browser_client.dart' ('/opt/flutter/.pub-cache/hosted/pub.dartlang.org/http-0.13.4/lib/src/browser_client.dart').
    var xhr = HttpRequest();
              ^^^^^^^^^^^
Error: Compilation failed.

Compiling lib/main.dart for the Web...                             12.9s
Exception: Failed to compile application for the Web.

Flutter Doctor
[✓] Flutter (Channel stable, 3.0.5, on Ubuntu 20.04.4 LTS 5.4.0-122-generic, locale en_GB.UTF-8)
    • Flutter version 3.0.5 at /opt/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision f1875d570e (2 weeks ago), 2022-07-13 11:24:16 -0700
    • Engine revision e85ea0e79c
    • Dart version 2.17.6
    • DevTools version 2.12.2

[✓] Android toolchain - develop for Android devices (Android SDK version 32.1.0-rc1)
    • Android SDK at /media/kolmanp/Data/opt/android-sdk-linux
    • Platform android-32, build-tools 32.1.0-rc1
    • ANDROID_HOME = /media/kolmanp/Data/opt/android-sdk-linux
    • Java binary at: /opt/android-studio/jre/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • CHROME_EXECUTABLE = /snap/bin/chromium

[✓] Linux toolchain - develop for Linux desktop
    • clang version 10.0.0-4ubuntu1
    • cmake version 3.16.3
    • ninja version 1.10.0
    • pkg-config version 0.29.1

[✓] Android Studio (version 2021.2)
    • Android Studio at /opt/android-studio
    • Flutter plugin version 67.1.2
    • Dart plugin version 212.5744
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)

[✓] VS Code (version 1.69.2)
    • VS Code at /usr/share/code
    • Flutter extension version 3.42.0

[✓] Connected device (2 available)
    • Linux (desktop) • linux  • linux-x64      • Ubuntu 20.04.4 LTS 5.4.0-122-generic
    • Chrome (web)    • chrome • web-javascript • Chromium 103.0.5060.134 snap

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

mhh the example inside package_info_plus/examples runs on Chrome for me.
How are you building for web, which command do you use?

I tried the command flutter build web or click to start in Android Studio Chipmunk | 2021.2.1.

yep, no problems for me:
$ flutter build web

💪 Building with sound null safety 💪

Compiling lib/main.dart for the Web...                             12.3s

Can you check if you have the same error with the example project?

Closing due to inactivity


Sampling details for FID evaluation

While it's mentioned that the full training set was used as reference for FID, I couldn't find details about evaluation so I wonder if I have missed them.  Noticed the 10K samples (instead of 50K) used in ablation, but no other specifications about sampling.  Besides, for guided sampling, specifics like samples per class would be useful too.  Any insight is appreciated, either from author or readers.

Found in evaluation code that it's 50K but #samples/class is random

Found in evaluation code that it's 50K but #samples/class is random

@zhihongp I also have the same question. Do you know why they use random int to represent the output class they desire to generate in classifier-guided sampling?
classes = th.randint(             low=0, high=NUM_CLASSES, size=(args.batch_size,), device=dist_util.dev()         )         model_kwargs["y"] = classes

Found in evaluation code that it's 50K but #samples/class is random

@zhihongp I also have the same question. Do you know why they use random int to represent the output class they desire to generate in classifier-guided sampling? I believe this "y" is the class you want to see in the outputs, right? classes = th.randint( low=0, high=NUM_CLASSES, size=(args.batch_size,), device=dist_util.dev() ) model_kwargs["y"] = classes
@zhihongp @xcharxlie
hi,have you solved this question?I tried to look into image_sample.py script,and in muti-class situation,I want to know how the pics matching with the lables. According to (model_kwargs["y"] = classes) , classes are something like these:682 380 174 125 533  27 133 421 581 985(tensor),AND I can't figure out the numbers and the relationship with the labels.Thanks for your reply.


Node tip=plus forward backward optim

For testing only

Investigating the issue with the node tip weights:
Run the following:
ts = msprime.simulate(sample_size=10, length=2e6, Ne=10000, mutation_rate=1e-8,                           recombination_rate=1e-8, random_seed=11)
Look at node 703. It has a value of nan in mn_post, forwards, backwards etc.
We see that the issue is that it is listed as having zero tips:
span_data.weights(703)
{10: Weights(descendant_tips=array([10,  0]), weight=array([0.48817349, 0.51182651]))}
The actual value should be this (the old code returns the right value):
{10: Weights(descendant_tips=array([10]), weight=array([1.]))}

Investigating the issue with the node tip weights: ...
Look at node 703. It has a value of nan in mn_post, forwards, backwards etc.
Let me know if you can reproduce this @hyanwong

Hi - I've now fixed this in the latest commit in this branch @awohns, and added the example to the test cases (it was when root in the first tree dropped out in the second - I was forgetting to initialise the number of children for all parent nodes at the start of the iteration loop). Let me know if you find any other bugs.

Tested a little, closing now that https://github.com/awohns/tsdate/pull/54 is merged


Provide default message for exceptions

Hi,
This PR provides default messages for those two exceptions. Especially in REST context when you throw this excetpino, it's annoying to always write the same message.
ping @danizord

Coverage remained the same at 94.09% when pulling 7f024fa4a1474e3c17dc501b9f0cf45231e08469 on default-messages into c3dcc0dccd7183bb401de9870e711b5291b6b194 on master.


Clicking a link to source file in sidebar opens file in a new VS Code tab

Is this a bug?

Is this a bug?

I don’t think so. What’s unexpected?
On Thu, Jan 7, 2021 at 4:04 PM Dustin Byrne<EMAIL_ADDRESS>wrote:

Is this a bug?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/applandinc/vscode-appland/issues/70#issuecomment-756383058,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAVC66G7435Q2DYX4UCYBLSYYOVJANCNFSM4VZKRIEA
.

I don’t think so. What’s unexpected?
On Thu, Jan 7, 2021 at 4:04 PM Dustin Byrne<EMAIL_ADDRESS>wrote:

Is this a bug?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/applandinc/vscode-appland/issues/70#issuecomment-756383058,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAVC66G7435Q2DYX4UCYBLSYYOVJANCNFSM4VZKRIEA
.

Hijacking this issue because this behavior needs to be re-implemented for the Vue frontend

Hijacking this issue because this behavior needs to be re-implemented for the Vue frontend


Remove mocha jsdom

Fixes [GHE]/CFGOV/platform/issues/1455
mocha-jsdom is not being maintained with the current version of jsdom. We can run jsdom directly with some tweaks. This should be the preferable option so that we don't create a series of tests that depend on proprietary mocha-jsdom syntax.
Additions

jsdom-global

Removals

mocha (I don't think this is needed, it's not referenced?)
mocha-jsdom

Changes

Updates data-set-spec, since jsdom now supports dataset attribute.
Moved HTML_SNIPPET constant to above describe block for consistency.
Minor indent cleanup.
Uppercase two constants.
Wraps tests in a describe block that weren't inside one root describe block.
Updates sinon to version 4.

Testing

npm install && gulp test:unit

Todos

Some tests were xit'd because they relied on form submission. These need to be refactored to not require form submission since jsdom does not support it https://github.com/tmpvar/jsdom/blob/master/lib/jsdom/living/nodes/HTMLFormElement-impl.js#L70 @jimmynotjim had some ideas.
GlobalBanner is unnecessarily complex and requires updates for the latest jsdom that aren't worth it in my opinion. Aiming to resolve this in https://github.cfpb.gov/CFGOV/platform/issues/1899
In theory this will allow updating of gulp-mocha, but at the moment it's not compatible with istanbul, see https://github.com/SBoudrias/gulp-istanbul/issues/115

Checklist

[ ] PR has an informative and human-readable title
[ ] Changes are limited to a single goal (no scope creep)
[ ] Code can be automatically merged (no conflicts)
[ ] Code follows the standards laid out in the development playbook
[ ] Passes all existing automated tests
[ ] Any change in functionality is tested
[ ] New functions are documented (with a description, list of inputs, and expected output)
[ ] Placeholder code is flagged
[ ] Visually tested in supported browsers and devices
[ ] Project documentation has been updated
[ ] Reviewers requested with the Reviewer tool :arrow_right:

@sebworks please chuck your local and re-pull. I rebased with master and did some cleanup.

yeah, I think mocha is a requirement of gulp-mocha, so we should be good.


Trailing slashes break URLs

Not working: (https://dziennik.netlify.com)

/pl/
/en/
/logged-in/
/zalogowany/

The problem is that Google serves links with trailing slashes. Instead of providing a log in prompt it serves a blank page. Normally, e.g.: /pl works, but with the trailing slash the urls turn from this
https://dziennik.netlify.com/css/app.7cb7cb60.css

to this:
https://dziennik.netlify.com/pl/css/app.7cb7cb60.css

which does not exist.
Resources:
- Possible fixes:

https://github.com/vuejs/vue-router/issues/1753#issuecomment-386180402

https://jsfiddle.net/L7hscd8h/9377/

https://github.com/JakubKoralewski/dziennik_php_frontend/commit/ee894de969dcb16d02ebfe32bb9b08604a24fb4d almost does it.
Just the hogwarts.jpg is loaded incorrectly.

Fixed via https://github.com/JakubKoralewski/dziennik_php_frontend/commit/60c6e48183938940a31b36dbd7490bb51e2b6712.


channel modifiers issue

I have some problem with Star-Lights-mini-Wash-Moving-Head.qxf:
appling invert channel inverter on RGB channels some machines during show do not give light.
In forum Jano and Massimo had different ideas about.....
https://sourceforge.net/p/qlcplus/discussion/general/thread/87f529f6/?limit=25#236f
Jano sayd "The modifiers are applied too soon in the code. They are applied before HTP check and before Grand Master, but they should be applied at the very end of the processing chain"
and Massimo sayd "Channel modifiers are applied before HTP check and Grand Master on purpose.
Please DO NOT change this logic in the code until this discussion explains why it is necessary."
Please, what we can do to resolve that issue?

Tolmino, the forum is frozen cause I'm migrating it somewhere else. (it's written)
Please do not post here requests for support. This is mostly a development area.
Be patient and post this on the new forum once it's public.
Thanks

you are right, excuse me, i'll use forum in the future. but Jano define it that as a bug!

Closing this then


npm install displays vulnerabilities. Ppckages are to be updated

Doing a npm install is displaying at least 12 vulnerabilities.

If you're okay I can try and fix it.

PR #265


How to find the people who are not wearing helmet in the picture

Hello everyone.
I know how to using image detection to find whether there is helmet or not in the picture,
but I don't know how to find whether there is someone without wearing helmet,  especially when there is another  wearing helmet in the picture.
Would you please tell me how to solve the problem?
Thanks in advance.
Ryu
Update
ーーーーーーーーーーーーーーーーーーーーーー
I have tried to train a model with my picture with and without helmet(about 100 images each), but the recognition rate is not good when using the trained model actually, sometimes the model can recognize well and sometimes it can NOT recognize correctly when my head moves. Are my images for training too few? Can the model be use to recognize the people without wearing helmet but another wearing helmet?

This issue is closed due to lack of activity in the last 90 days. Feel free to ping me to reopen if this is still an active issue. Thanks!
Also, do please check out our forum (and Chinese version) for general "how-to" questions.


chore: update docs for jib-core after 0.24.0 release

Addresses post-release tasks in https://github.com/GoogleContainerTools/jib/issues/4006.

Closing in favour of https://github.com/GoogleContainerTools/jib/pull/4024.


VS project.json migration shouldn't delete global.json before calling dotnet migrate

See https://twitter.com/blaken/status/824827246091309057
Repro steps:

git clone https://github.com/exceptionless/foundatio
Open the .sln file in VS 2017 RC3
Migrate the project.json based projects to .csproj.

Expected Results
The migration should succeed.
Actual Results
UpgradeLog.htm.txt
All the "test" and "sample" projects fail to migrate.
Notes
I believe this is because VS deletes the global.json file before calling dotent migrate.  See https://github.com/dotnet/roslyn-project-system/blob/ea445f5cd6e567196c285e4f254695858158165b/src/Microsoft.VisualStudio.ProjectSystem.CSharp.VS/ProjectSystem/VS/Xproj/MigrateXprojProjectFactory.cs#L55.
This is an issue because dotnet migrate needs the projects section of the global.json file in order to resolve P2P references across directories - which is what is happening here.  There are P2P references from the "test" directory to the "src" directory.
/cc @livarcocc

This was designed to work this way: https://github.com/dotnet/roslyn-project-system/issues/1184. The issue is if there's a global.json that specified a lower version of the cli, it probably won't call the correct version of dotnet migrate.

@333fred I wonder if we are hitting the issue fixed by https://github.com/dotnet/cli/pull/5352. When I talked to @eerhardt, his version of the CLI was older and did not have that fix in.

@333fred I believe we should try this scenario in a VS instance that definitely has a newer version of the CLI.

I'll update to the latest d15prerel and try locally.

@livarcocc no dice, failed with test\Foundatio.Benchmarks\Foundatio.Benchmarks.xproj: MIGRATE1014::Unresolved Dependency: Unresolved project dependency (Foundatio) on ~1/2 of projects. CLI version:
C:\Users\frsilb>dotnet --info
.NET Command Line Tools (1.0.0-rc4-004689)

Product Information:
 Version:            1.0.0-rc4-004689
 Commit SHA-1 hash:  c740fa8f43

Runtime Environment:
 OS Name:     Windows
 OS Version:  10.0.14986
 OS Platform: Windows
 RID:         win10-x64
 Base Path:   C:\Program Files\dotnet\sdk\1.0.0-rc4-004689

All right. I will take a look.

This was designed to work this way: #1184. The issue is if there's a global.json that specified a lower version of the cli, it probably won't call the correct version of dotnet migrate.

Agreed having the old "sdk" version is a problem.  A solution may be to delete the "sdk" property out of the global.json instead of removing the whole thing.

I think @eerhardt is right after all. As usual! I don't see how this scenario ever worked. When we try to migrate a project under test/project, we only search for dependencies under the test folder, because there is nothing telling us to search for projects under src/ (that would be in the projects of the global.json).
Now, we could do a fix in the CLI to not validate these projects, but the problem here would be that we could end up migrating project dependencies as package dependencies. Only references marked with target: project would be migrated as project, the other ones, because we didn't find them, would migrate as package references.
@333fred How feasible would it be to only move the global.json at the end of the process? A problem because of the SDK property? Otherwise, I don't see how migration can get this right for all cases.

@livarcocc and I talked offline. I'll prep a change that will remove the sdk element from the global.json and delete the global.json after migration is completed.


Increase font size of the exercises

The text describing the exercise and hints on e.g.
http://tekvideo.sdu.dk/t/henrikmidtiby/CK-FKO1/2017/0/Blok02/6
is too small.
Can you make it larger?

I see no reason it shouldn't be possible


While Adding Working flow conditions

/usr/local/lib/python3.6/site-packages/sklearn/metrics/classification.py:1143: UndefinedMetricWarning: F-score is ill-defined and being set to 0.0 in labels with no predicted samples.
'precision', 'predicted', average, warn_for)
[Parallel(n_jobs=1)]: Done  12 out of  12 | elapsed:    0.1s finished
[Parallel(n_jobs=1)]: Using backend SequentialBackend with 1 concurrent workers.
/usr/local/lib/python3.6/site-packages/sklearn/metrics/classification.py:1143: UndefinedMetricWarning: F-score is ill-defined and being set to 0.0 in labels with no predicted samples.
'precision', 'predicted', average, warn_for)

resolve the error to reinstalled the packaged . do you have any guide for all functionality configurations. like slot,workflow.,.etc please share to<EMAIL_ADDRESS>

Glad to hear the error is resolved.
I've written up some of the details on the wiki page but just haven't had the time to add all the information just yet.
Slots will basically try and exact any found slot from the sentence (you can test it by sending the sentence to Rasa and see the returned slots) then save them in a variable which you can then use in webhooks / responses / next replies.
The workflow works like a waterfall, when you enter the top on in that session it is then possible to trigger the ones below it and so on. This will reset on a session change.,


Use test name to derive a file name for the svg

Fails

:no_entry_sign:
Please add a description to your PR.

  Generated by :no_entry_sign: dangerJS


How to specify video quality (CRF) when "--download-sections --force-keyframes-at-cuts" is used

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm asking a question and not reporting a bug or requesting a feature
[X] I've looked through the README
[X] I've verified that I have updated yt-dlp to nightly or master (update instructions)
[X] I've searched known issues and the bugtracker for similar questions including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Please make sure the question is worded well enough to be understood
--force-keyframes-at-cuts re-encodes with too low quality by default and i can't find a way to increase quality
Provide verbose output that clearly demonstrates the problem

[ ] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] If using API, add 'verbose': True to YoutubeDL params instead
[ ] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
No response

you pass arguments to ffmpeg using yt-dlp's --downloader-options option


Add a git repo definition

Relevant information
Add a git repo entity definition to allow the Entity Community team to execute on the steel thread.
Checklist

[ ] I've read the guidelines and understand the acceptance criteria.
[ ] The value of the attribute marked as identifier will be unique and valid.
[ ] I've confirmed that my entity type wasn't already defined. If it is I'm providing an
explanation above.

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

It looks like the CLA check got lost. @jerelmiller can you re-push this so it triggers? (I can't do it since it's on your fork)

Merging this one, I asked @jerelmiller if we want to have synthesis rules also, but they can be added in another PR

@srvaroa I signed the CLA, and just did it again, but not sure why the status is not changing. You can clearly see the comment above is green. Let me reach out to our OSS team.

@jerelmiller the CLA bot seems to be broken. We will need to ping other teams to look into that one


Permission denied

Hi,
when I run sudo from Termux I get:
-bash: /data/data/com.termux/files/usr/bin/sudo: Permission denied. I've changed the permission and verified that the file is there from adb (as root).

figured it out by myself - the commands need to be run from Termux (I didn't read the documentation that focused and it wasn't too obvious like enter those commands IN TERMUX).


Ubuntu snap for drive

Hi odeke-em
i wanted to make it easier for people to install drive on different types of Linux distros, so i went ahead an setup an Ubuntu snap for drive, you can install drive by doing the following command. luckily snaps can be deployed on other distros as well such as Redhat, CentOS, Debian etc. currently i have my launchpad account pulling from my fork automatically using git and building an new snap each time i change something. so once you add my snapcraft.yaml file to the main repo for drive, i can change my launchpad settings to grab any changes automatically and push an new snap.
https://github.com/Dedsec1/drive/
https://github.com/Dedsec1/drive/blob/master/snapcraft.yaml
sudo snap install drive --classic
Regards
Dedsec1

@odeke-em is there anything that blocks this PR from merging?

My apologies for the very late reply @Dedsec1, I have been swamped for the past year and more.
Awesome, I'll kindly page some Linux folks to help me review this change @shaggytwodope @m-svo

My apologies for the very late reply @Dedsec1, I have been swamped for the past year and more.
Awesome, I'll kindly page some Linux folks to help me review this change @shaggytwodope @m-svo.
Also @LyzardKing posted some feedback PTAL.
@idvoretskyi nothing blocking it, I was just away from here for a while :)

@odeke-em The only thing I'd add to the current PR are the following lines:
version: 'master'
version-script: |
    grep "Version =" src/about.go | cut -d'"' -f2
base: core18

First we set the version to a string, then we override it with the actual drive version.
Then in the 3rd line the base is updated to the latest ubuntu LTS

@odeke-em sorry, not my area of expertise, so not able to review

@odeke-em There seems to be an outdated version already uploaded to the snap store.
Once the snapcraft.yaml file is included in the repo you can ask on the snapcraft forum for the ownership of the snap to be transferred to you. And the snap would build automatically on every upstream commit.

the ownership of the snap to be transferred to you

+1.

@odeke-em What can we do to get this merged? I can open another PR with my changes..
A part from that I can tell that I have it built with my snap changes and it seems to work properly. Although there might be something else that needs testing..


Posts on v. 0.7 issue for the word "first"

The word "first" never shows the first char so the post became "irst"

relates to https://github.com/TryGhost/Ghost/issues/6375

Thx

:smile:


Warn users about the insecurity of SHA-1

I believe that a warning should be in place on this project for any given user that SHA-1 should not be used unless there is a very good reason for it. I know that the majority of people who manage to make it to this library will be here for good reason and not using this for security in production, but it is vital that we still have information, even in the basic form of a README header.

added in README


Fix blurry text on tile hover

Replaces transform: scale (1.1) with width: calc(100% + #{$iui-xl}); to fix the blur issue in #44.  This change ensures that the thumbnail's width & height are always a whole number.  When the thumbnail expands to a non-whole number it introduces sub-pixelation which seems to be creating the blur effect on Windows.
Thumbnail width in our demo = 288px
Old method of using 1.1 scale = 316.8px & blurry
New method of using 100% + iui-xl = 320px & not blurry

The location tile using an iframe is still having a hover issue, I am unsure how to solve this at the moment.

Isn't @mayank99  working already on that? https://github.com/iTwin/iTwinUI/pull/37
Two people shouldn't work on the same task. @FlyersPh9 if you find something, better send it to Mayank that he could investigate further.

Agree with @bentleyvk. The amount of PRs got really insane and it gets worse (now we have duplicates over same bug).
Cant we just remove on hover effect of changing sizes?
Lets have something simple.. We can better change some colors (apply overlay on image, showed iModel Manager example to @mayank99), add shadows, etc..
I really do not find the advantage of having this effect versus bugs it provides...
Also, is not showing menu without hover a good UI? Isnt there "visibility of interactive items" kinda important? Or its ok that users only accidentally can find the menu when they hover over tile...

My PR was not a proper fix so I kinda gave up and had assigned #44 to @FlyersPh9. I didn't think it would become top priority (unlike tooltip) 😄

I needed a break from #43 & since this was also a blurry text issue, I thought they might have similar causes.  It sounds like this PR is not the actual fix, so I might abandon this PR & submit a new one to replace the hover effect temporarily.
@veekeys, in response to the More Menu only appearing on hover: it's a delicate balance of what do we show VS what do we show thru progressive disclosure. The user will need to interact with the tile regardless to access these more menu options.  Our tiles are already pretty busy, if we always showed the more menu icon, it becomes unnecessary visual noise.
Here's a screenshot of Microsoft Teams doing something similar to what we have now:

Google Drive doesn't even show a more menu on the tile / table row, they just hope that you think to either:

Select a tile / row & then select the more menu from the top right of the UI.
Right click on the tile / row.


Tabbed form with differend table in every tab

Hallo all
Im new in laravel, but this backpack help me a alot.
And can i use different table in every tab, a little code for explain please..
Thanks so much

As far as I can tell, this isn't easily possible.

ok thanks, ill  try to use "native" laravel instead, and btw thanks for the respon


[nfs-provisioner] Stale file handles on GKE with regional persistent disks

Hi,
we are running the nfs-provisioner on a GKE regional cluster. We are also using a regional persistent disk as a storage backend for the NFS provisioner pod. According to the google documentation [1] this provides a persistent disk replicated in 2 zones. In an emergency case the nfs-provisioner pod can just be started on a node in the replicated zone and all should be fine (as the replicated disk can be mounted on the new node). We are using NFS v4.1 which should help a lot in cases where the state of clients has to be recovered.
When we tested the failover scenario we sometimes noticed some 'NFS: stale file handle' errors on connected NFS clients. The clients are writing constantly to a log file. As this was not happening all the time, we started to investigate more. What we have seen so far is:

when the new nfs-provisioner pod gets scheduled on the same node as before, the clients can just continue to work (there is only a short interruption)
when the nfs-provisioner pod gets scheduled onto another node (with access to the replicated persistent disk) it does start normally, but clients are experiencing 'NFS: stale file handle' issues. After we restart the client pods everything works normal again.

As this does not provide a real HA solution, we are searching for explanations/solutions here :-) Does anyone have an idea why this happens?
We are using the gke version 1.12.7-gke.25.
[1] https://cloud.google.com/blog/products/gcp/get-higher-availability-with-regional-persistent-disks-on-google-kubernetes-engine

/assign
I'll try to find some answers, but I suspect this is related to how the NFS client handles connections to the server.  I saw similar reports using NFS backed by EmptyDir: https://github.com/kubernetes/kubernetes/issues/75918
Have you tried regular PD to back the NFS share? Does it also see the same issue?

Thx for your answer. That actually inspired me to do some more debugging on the topic :-)
So I tested to restart the nfs provisioner on a different node in the same zone with the same google persistent disk and all clients got into the 'stale file handle' state again. This lead to the assumption that whenever the underlying host changes we run into the issue. I enabled debug logs in ganesha (MID_DEBUG level) and found the following line in the case of stale file handles:
26/08/2019 08:32:39 : epoch 5d639925 : nfs-server-provisioner-nfs-r9fhsa-0 : nfs-ganesha-19[svc_12] vfs_extract_fsid :FSAL :M_DBG :Handle len 18 0x45: fsid=0x0000003000000008.0x0000000000000030, type 0x01, opaque: (8:0x010004005f00b991)
26/08/2019 08:32:39 : epoch 5d639925 : nfs-server-provisioner-nfs-r9fhsa-0 : nfs-ganesha-19[svc_12] vfs_check_handle :FSAL :INFO :Could not map fsid=0x0000000000000008.0x0000000000000030 to filesystem
26/08/2019 08:32:39 : epoch 5d639925 : nfs-server-provisioner-nfs-r9fhsa-0 : nfs-ganesha-19[svc_12] complete_op :NFS4 :DEBUG :Status of OP_PUTFH in position 0 = NFS4ERR_STALE, op response size is 4 total response size is 40

I read the corresponding part of the NFS4.1 RFC [1] and it looks like that whatever handle the client got from the server needs to be valid for the lifetime of the filesystem object (even after restarts of the NFS server).
After that I found that 'fsid_device' is always set to true in the ganesha configuration [2]. This leads to server generated file handles based on the major/minor device ID of the mounted storage block device (the one which backs '/export'). As these device IDs differ between different gke nodes, the server expects a different file handle when being restarted on a different node and returns NFS4ERR_STALE. As there is already a file system ID generated per export configuration I would be interested in the reason of setting fsid_device to 'true'?
I created a new docker image with 'fsid_device' set to false and also with the most current ganesha version (2.8.2) as 2.7.x will not get any fixes back ported [3]. With that a restart of the nfs provisioner was possible. Clients only experienced a wait time of 90s (the grace period).
Would it be possible to make the 'fsid_device' setting customizable?
[1] https://tools.ietf.org/html/rfc3010#section-4.2.2
[2] https://github.com/kubernetes-incubator/external-storage/blob/350c374b1331b4b636f30f3d41c87d76844dc8d3/nfs/pkg/server/server.go#L114
[3] https://github.com/nfs-ganesha/nfs-ganesha/wiki#Release_Status

We are using the nfs-provisioner in a regional google GKE cluster. As far as I know docker is not used as the container runtime, but I might be wrong (did not find documentation about what is used underneath). Having the possibility to set 'fsid_device' would solve the issue for us. We just need to get the change as well into the upstream helm chart. After that we are fine. In the end the combination of nfs provisioner + google regional persistent disk provides a nice HA solution (we might need to lower the grace period a bit though).
On a different note: Could you tell me something about the update strategy with the Ganesha version in use? If I understand the wiki page of ganesha correctly, then the 2.7.x version will not get fixes and 2.8.x should be used instead. Should this be a separate issue? There is also a minimal version of fedora available which could be used with a multistage build. In my tests I got a 100MB smaller image in the end.

All of this is solved and merged. Will close this issue.


Make build task .NET Core compatible

Currently we have two separate build process, especially all the build tasks that are being used.
Making all the task .net core will help reduce various scripts that are being used to accommodate unix build.

Seems related to https://github.com/Azure/azure-sdk-for-net/issues/4692
I tried investigating the issue above but haven't found the build task source code.  Could you advise?

The repo is building cross-plat.


[BLOCKER] When generating with a shared enum, it's not imported from the api.types files in .base file

Describe the bug
I have a shared enum object user in a collection. When I generate then start the admin microservice, in the MyCollectionEditForm.base.tsx file, the enum initial value is correctly set to initialValue: MyCollectionPossibleStatus.SOMETHING, but MyCollectionPossibleStatus is not imported by default from the api.types file.

@Fen747 please show me the code how the enum looks, how you've added it in blueprint, you've set a default value for it, but be careful, values that come from GraphQL are not always values you'd want.
Maybe we should generate enum files for frontend as well?

The enum in Blueprint :
export enum ClientRequestPossibleStatusesEnum {
  CLIENT_REQUEST = "CLIENT_REQUEST", // DEFAULT
  QUOTATION_TO_DO = "QUOTATION_TO_DO",
  QUOTATION_IN_PROGRESS_TECH = "QUOTATION_IN_PROGRESS_TECH",
  QUOTATION_DONE_TECH = "QUOTATION_DONE_TECH",
  QUOTATION_VALIDATED = "QUOTATION_VALIDATED",
  QUOTATION_TO_CORRECT_TECH = "QUOTATION_TO_CORRECT_TECH",
  QUOTATION_SENT = "QUOTATION_SENT",
  QUOTATION_SIGNED = "QUOTATION_SIGNED",
  PREPARATION = "PREPARATION",
  SITE = "SITE",
  BILLING_TO_DO = "BILLING_TO_DO",
  BILLING_CREATED = "BILLING_CREATED",
  BILLING_SENT = "BILLING_SENT",
  BILLING_TO_CORRECT = "BILLING_TO_CORRECT",
  BILLING_TO_BE_RELAUNCHED = "BILLING_TO_BE_RELAUNCHED",
  BILLING_PAID = "BILLING_PAID",
  BILLING_COMPLICATED = "BILLING_COMPLICATED",
  COMPLETED = "COMPLETED",
}

export const ClientRequestPossibleStatus = Studio.sharedModel({
  id: "ClientRequestPossibleStatus",
  description: "Client request status",
  enumValues: Object.values(ClientRequestPossibleStatusesEnum),
});

const application = app({
  id: "belett",
  sharedModels: [
    //  ...
    ClientRequestPossibleStatus,
  ],

The problem is that  after a generation, the type is correctly generated in api.types.ts, but is not imported in one of the .base file client side.
ERROR in src/bundles/UIAppBundle/pages/ClientRequestsManagement/config/ClientRequestEditForm.base.tsx:111:23
Cannot find name 'ClientRequestPossibleStatus'.

The fix is simply to import it manually / with the VSCode suggestion, from api.root.ts, but why ?

This one starts to be very annoying, having to manually imports after each generation :P

@theodorDiaconu Hi ! :)
Have you been able to reproduce maybe ? This is quite problematic as it forces us to manually fix a bunch of enums at each generation :/

@y-moufakkr please prioritise this one as it's a blocker for David, thanks! @Fen747 we'll look into it ASAP

Hello @Fen747 , I ve tried to reproduce the bug but I couldn't, can you please share how you did you use the sharedModel enum ClientRequestPossibleStatus  in the collection ClientRequest?
also from what I m having right now, we are simply not importing the type from api.types.ts because we are not using it when generating the .base form code,
PS: I m not 100% sure about this last statement thats why I asked to see how you using the sharedmodel
if you are the one editing the .base code and need the type in it, thats another situation

HI @y-moufakkr ,
I can assure you do use the enum in the file generation, as it is used to populate the options of the enum's selector, but it is not imported :) (liekly, the code writer service does not handle it).
Maybe the beste solution is to schedule a quick screen-share so I may make it clear easily for you :)

I am available right now, and tomorrow anytime, just hit me up in slack, in the meantime gonna try to reproduce the type import in the .base


Fix(ios): Improve image orientation detection

Problem
Platform: iOS
Image manipulator is not detecting orientation correctly for images that were captured on iPhone's camera
https://user-images.githubusercontent.com/44551710/183207466-058a3720-1aae-41fd-b175-8922acf0fced.mp4
Solution
Add a function that fixes orientation before applying any manipulations. (Inspired by expo-image-manipulator)
Result
https://user-images.githubusercontent.com/44551710/183206404-088cdd9a-7e25-4462-9086-b74a1d524a85.mp4

I'd also appreciate if we could get this fix merged 🙏


The --preserve-posix-properties only works copying to BLOB stroage

Which version of the AzCopy was used?
Note: The version is visible when running AzCopy without any argument
AzCopy 10.22.0
Which platform are you using? (ex: Windows, Mac, Linux)
Linux
What command did you run?
Note: Please remove the SAS to avoid exposing your credentials. If you cannot remember the exact command, please retrieve it from the beginning of the log file.
# copy file to storage (this works - properties show in metadata)
azcopy cp --preserve-posix-properties testfile https://${account}.blob.core.windows.net/${container}

# copy back from storage (properties are not set on the local file)
azcopy cp --preserve-posix-properties https://${account}.blob.core.windows.net/${container}/testfile .

What problem was encountered?
The  --preserve-posix-properties flag only works copying to BLOB stroage
How can we reproduce the problem in the simplest way?
See above
Have you found a mitigation/solution?
No

@edwardsp Could you please share debug level logs for the AzCopy command when you are trying to download?

@gapra-msft Here is the log:
2023/12/13 14:40:33 AzcopyVersion  10.22.0
2023/12/13 14:40:33 OS-Environment  linux
2023/12/13 14:40:33 OS-Architecture  amd64
2023/12/13 14:40:33 Log times are in UTC. Local time is 13 Dec 2023 14:40:33
2023/12/13 14:40:33 ISO 8601 START TIME: to copy files that changed before or after this job started, use the parameter --include-before=2023-12-13T14:40:28Z or --include-after=2023-12-13T14:40:28Z
2023/12/13 14:40:33 Any empty folders will not be processed, because --recursive was not specified
2023/12/13 14:40:33 Job-Command --log-level DEBUG cp --preserve-posix-properties https://ACCOUNTNAME.blob.core.windows.net/test/testfile?SASKEY .
2023/12/13 14:40:33 Number of CPUs: 8
2023/12/13 14:40:33 Max file buffer RAM 4.000 GB
2023/12/13 14:40:33 Max concurrent network operations: 128 (Based on number of CPUs. Set AZCOPY_CONCURRENCY_VALUE environment variable to override)
2023/12/13 14:40:33 Check CPU usage when dynamically tuning concurrency: true (Based on hard-coded default. Set AZCOPY_TUNE_TO_CPU environment variable to true or false override)
2023/12/13 14:40:33 Max concurrent transfer initiation routines: 64 (Based on hard-coded default. Set AZCOPY_CONCURRENT_FILES environment variable to override)
2023/12/13 14:40:33 Max enumeration routines: 16 (Based on hard-coded default.  So the downloaded data cannot be MD5-validated. Dst: /home/paul/testfile
2023/12/13 14:40:34 INFO: [P#0-T#0] DOWNLOADSUCCESSFUL: /home/paul/testfile
2023/12/13 14:40:34 DBG: [P#0-T#0] Finalizing Transfer
2023/12/13 14:40:34 JobID=4c1e280f-e215-e646-64a6-3f28089aa129, Part#=0, TransfersDone=1 of 1
2023/12/13 14:40:34 all parts of entire Job 4c1e280f-e215-e646-64a6-3f28089aa129 successfully completed, cancelled or paused
2023/12/13 14:40:34 is part of Job which 1 total number of parts done
2023/12/13 14:40:35 PERF: primary performance constraint is Unknown. States: C:  0, R:  0, W:  0, F:  0, H:  0, B:  0, S:  0, P:  0, Q:  0, D:  0, E:  0, T:  0, GRs: 128
2023/12/13 14:40:35 100.0 %, 1 Done, 0 Failed, 0 Pending, 0 Skipped, 1 Total, 2-sec Throughput (Mb/s): 108.0933
2023/12/13 14:40:35

Diagnostic stats:
IOPS: 0
End-to-end ms per request: 0
Network Errors: 0.00%
Server Busy: 0.00%

Job 4c1e280f-e215-e646-64a6-3f28089aa129 summary
Elapsed Time (Minutes): 0.0334
Number of File Transfers: 1
Number of Folder Property Transfers: 0
Number of Symlink Transfers: 0
Total Number of Transfers: 1
Number of File Transfers Completed: 1
Number of Folder Transfers Completed: 0
Number of File Transfers Failed: 0
Number of Folder Transfers Failed: 0
Number of File Transfers Skipped: 0
Number of Folder Transfers Skipped: 0
TotalBytesTransferred: 27040670
Final Job Status: Completed

2023/12/13 14:40:35 Closing Log

@edwardsp Thanks for sharing the log, I think I have an idea as to what is going on here. We will work on getting a reproduction and fix as soon as we can.
Future dev notes: common.unixStatAdapter.ReadStatFromMetadata does not take into account that the metadata could be returned with an initial capital letter, and I believe that is where the issue is for this bug.

Closing this issue as a PR was merged that will be released as part of 10.23.0 which will resolve this issue.


No more free option?

Hi, is there no more free option for Cursor Pro? I saw a free option for 2 hours before.

Hello, simply close the window and reopen it, it was that it was integrated for the payment on the same terminal.

So it's not free anymore? I followed your method but it only has 1 option which is pay via Paypal

Exact same here, reopening don't work.

Yes, the free entrance and backend have been closed


Is transposed convolution supported ?

If not, do you have a plan to support in the future?

I have found it. By using convolution op, and set the direction parameter of DML_CONVOLUTION_OPERATOR_DESC to DML_CONVOLUTION_DIRECTION_BACKWARD


Switching from opening settings to the applicant view after making an opening public has the opening color label as gray

I think this is because of the way SWR caches things:

Desired outcome:

Closing as too much FE stuff has changed since this issue was made


The nginx base image look for config in conf.d not in site-availables

The default nginx configuration doesn't works like ubuntu ones.
You have to replace the /etc/nginx/conf.d/default.conf file with your default one.

Thanks @achretien
First moment when I tested it did not work and I was confused. Followed your suggestion. Now it works.
@jamessharp please fix the container with above suggestion

balls - fixed in #7 I hope
Sorry everyone


Fix out of bounds error when folding last line of code

Fixes #52385
Issue ;
Folding a block of code at the end of a script file causes an out of bounds error in the console.
Identified cause :
Error in folding loop. i should never be equal to endline. Valid line index goes from 0 to get_line_count() -1
https://github.com/godotengine/godot/blob/18c0f0b3ba46caf3f6dda401362bc5993eaf7f8d/scene/gui/code_edit.cpp#L1433-L1435
Fix proposal
Simply replace <= compare operator by < .
Result
No more error. Folding OK.

If I understand, you prefer keeping <= in the loop but changing line 1403 to
int end_line = get_line_count();
I can do that but I don't really understand why. Other loops ends with test < get_line_count() not <= get_line_count() -1 (l. 1411, l 1420, ...)
That's why I choose this way.

Built & tested, confirmed to fix the issue. :) Thank you!

Even if approved, I'm closing this PR cause changes are included my the other PR about code folding : #52339