Fix select syscall

Currently passing an empty fdset/None to select syscall could cause a panic. The source of the panic comes from
let (writenfd, mut real_writefds) = selectbittables[1].get(&FDKIND_KERNEL).unwrap();

in select_syscall of net_calls.rs.
I traced a little bit deeper, and it looks like in get_bitmask_for_select, it will return an empty HashMap if the fdset is empty or None. Then since the HashMap is empty, when it calls get(&FDKIND_KERNEL), that would return None, and the following unwrap would cause panic.

Is this using an outdated version of fdtables?   Can you check your code matches the code here: https://github.com/JustinCappos/fdtables/tree/main/docs

looks like prepare_bitmasks_for_select and prepare_bitmasks_for_select are the same. My branch just recently merged from porting-fdtables branch so it should be up to date

This is the latest version of fdtables. I discussed it with Qianxi, and we found that the error is due to incorrect handling of return values, which should be fixed within RawPOSIX. Since these fixes will also affect the implementation in other parts of select, I’ve added it to the to-do list and plan to come back with a detailed fix plan in two weeks.

Okay, thanks.   I will see if I can get to it beforehand.  If not, I
welcome your patch and would be happy to review.
On Tue, Oct 8, 2024 at 2:55 PM Alice Wen @.***> wrote:

This is the latest version of fdtables. I discussed it with Qianxi, and we
found that the error is due to incorrect handling of return values, which
should be fixed within RawPOSIX. Since these fixes will also affect the
implementation in other parts of select, I’ve added it to the to-do list
and plan to come back with a detailed fix plan in two weeks.
—
Reply to this email directly, view it on GitHub
https://github.com/Lind-Project/RawPOSIX/issues/48#issuecomment-2400593547,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAGROD6Z7LUTEUJPP32L5YTZ2QTBNAVCNFSM6AAAAABPQRLR4OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMBQGU4TGNJUG4
.
You are receiving this because you were assigned.Message ID:
@.***>

This error is fixed by handling null values correctly. Will submit PR for further fix details.

Fixed: https://github.com/Lind-Project/RawPOSIX/pull/89


scrape CKAN for data sources

Context
CKAN is one of the most popular open-source data portals, and is often used by governments to publish police data.
We have the start of a CKAN scraper here: https://github.com/Police-Data-Accessibility-Project/scrapers/tree/main/scrapers_library/data_portals/ckan/package_list
Requirements

[ ] scrape a batch of potential portals or sources from CKAN.

target one of our record types
or check our list of data sources record type

related issue: https://github.com/Police-Data-Accessibility-Project/data-source-identification/issues/63

[ ] optional: run them through one of our ML models to see how it classifies them: https://huggingface.co/PDAP
[ ] Submit them to our Data Sources database (data dictionary here)

they can be uploaded as a CSV to airtable for now

Based on this, we can choose what to do next, how to scale, how many to attempt

CKAN API Docs: https://docs.ckan.org/en/2.9/api/


Mod incompatibility: Dark Codex

Hey, thank for the update it's great!
Was stuck on loading so I tried activating mods one by one (36...) and it seems that Dark Codex is the culprit here.
No issue with the others, but quite sad since DarkCodex is a great addition!
Don't know what/if you need some logs or whatever.

how does dark codex conflict with isekai mod? what does it say in the logs?

It works for me just fine though.

Will look into this tomorrow evening, it must be a setting issue, perhaps the one that changes the psycho base stat

This is the end off the player logs :
[IsekaiMod] Null Value in selection of=156a6a1e62a24504a47c4d1e8b44982e
[IsekaiMod] Null Value in selection of=2a9ee5e4e745461ebddf361a0aab060b
NullReferenceException: Object reference not set to an instance of an object
at IsekaiMod.Content.Heritages.IsekaiHumanCrossbreedLegacy.Patch () [0x0011a] in <4a9f668a01bb43b5852c37e07e6aaa46>:0
at IsekaiMod.Content.StartGameLoader_LoadAllJson.PatchHeritages () [0x00000] in <4a9f668a01bb43b5852c37e07e6aaa46>:0
at IsekaiMod.Content.StartGameLoader_LoadAllJson.Postfix () [0x0002e] in <4a9f668a01bb43b5852c37e07e6aaa46>:0
at (wrapper dynamic-method) Kingmaker.Blueprints.JsonSystem.StartGameLoader.Kingmaker.Blueprints.JsonSystem.StartGameLoader.LoadAllJson_Patch7(Kingmaker.Blueprints.JsonSystem.StartGameLoader,string)
at Kingmaker.GameStarter+d__25.MoveNext () [0x003a8] in <00306b2fa77c4a0993735a8f1c2b44d1>:0
at UnityEngine.SetupCoroutine.InvokeMoveNext (System.Collections.IEnumerator enumerator, System.IntPtr returnValueAddress) [0x00026] in <25a85da7c6f04932b86e339dfd12957d>:0
There is no error in DarkCodex logs but I can attach the file if need be

what exact version of dark codex are you using?
Dark Codex and its Descendent Expanded Elements are rather notoriously "friendly" when it comes to integration and require extra patching or handling, so my guess would be you are using an older version or a newer one that contains something unhandled and the only way to replicate would be to use the exact same version of dark codex...
@Mythalar

I only ever use the latest version of any mods actually so 1.5.29 as if now.

So I downgraded DarkCodex step by step since you pointed some version issue.
The reported bug only occures on @Truinto 1.5.22 update and up (1.5.21 works).

Yeah, dark codex likes creating nullpointers that are not resolved till later that then screw over anyone trying to patch anything they touched, or in some cases never get resolved...
I will add handling to deal with it to the patch method or better yet an even later patch phase to run our code in, not sure if there are any though...

every single item on the following list contains a nullpointer after the phase BlueprintInit:
[IsekaiMod] ERROR: Unpatachable Feature= 637329bf936244668c55d81985c4eaf8name= Venom Infusion at level= 2 reason= Object reference not set to an instance of an object
[IsekaiMod] ERROR: Unpatachable Feature= e9021412e87847b3b104d4d31bfe5403name= Venom Infusion, Greater at level= 2 reason= Object reference not set to an instance of an object
[IsekaiMod] ERROR: Unpatachable Feature= 96360bedde8648a8a6762e2de41b60a5name= Disintegrating Infusion at level= 2 reason= Object reference not set to an instance of an object
[IsekaiMod] ERROR: Unpatachable Feature= e9cf588e2ef64fb68d0ec8c566e8b294name= Dampening Infusion at level= 2 reason= Object reference not set to an instance of an object
[IsekaiMod] ERROR: Unpatachable Feature= 8662ccb8dd484a2f8139d46621c641fdname= Enervating Infusion at level= 2 reason= Object reference not set to an instance of an object
[IsekaiMod] ERROR: Unpatachable Feature= 3ae954ad56a2497b92fada3dc493b4e1name= Pulling Infusion at level= 2 reason= Object reference not set to an instance of an object
[IsekaiMod] ERROR: Unpatachable Feature= e53160d091914d50bfc1d8d4fa482e30name= Unnerving Inufsion at level= 2 reason= Object reference not set to an instance of an object
[IsekaiMod] ERROR: Unpatachable Feature= 922fd10e3e994d7793821da1583cdfeaname= Vampiric Infusion at level= 2 reason= Object reference not set to an instance of an object
[IsekaiMod] ERROR: Unpatachable Feature= 987dc633dbae49b0adc11cd9c5672553name= Weighing Infusion at level= 2 reason= Object reference not set to an instance of an object
[IsekaiMod] ERROR: Unpatachable Feature= 1cd0cd60997d4288be1fc85f753e53dename= Spore Infusion at level= 2 reason= Object reference not set to an instance of an object
[IsekaiMod] ERROR: Unpatachable Feature= fdc63fd61b794e40ba3c5446ba8ea1c2name= Toxic Infusion at level= 2 reason= Object reference not set to an instance of an object
[IsekaiMod] ERROR: Unpatachable Feature= 56567887f604473797dc8223c68999daname= Toxic Infusion, Greater at level= 2 reason= Object reference not set to an instance of an object
[IsekaiMod] ERROR: Unpatachable Feature=<PHONE_NUMBER>9343c0b5652f082bd677d1name= Self Telekinesis, Greater at level= 2 reason= Object reference not set to an instance of an object
[IsekaiMod] ERROR: Unpatachable Feature= c03b1a6d297843c584b763378024ea7aname= Telekinetic Finesse at level= 2 reason= Object reference not set to an instance of an object
[IsekaiMod] ERROR: Unpatachable Feature= 8d3e9c13037d4d4f944887b517bbf86bname= Gravity Control, Greater at level= 2 reason= Object reference not set to an instance of an object
[IsekaiMod] ERROR: Unpatachable Feature= 70322f5a2a294e54a9552f77ee85b0a7name= Metakinesis — Empowered at level= 1 reason= Object reference not set to an instance of an object
[IsekaiMod] ERROR: Unpatachable Feature= 562e997e20a34bc9a8d4624fa7151761name= Metakinesis — Selective at level= 1 reason= Object reference not set to an instance of an object

just use the older dark codex version in the meantime then

That's what I'm doing, thanks again :)
Was weird to me since it compiles without Darcodex but most of the errors (in your logs too) concern Kineticist Expanded Elements new features but I guess they are linked.

Kineticist Expanded Elements has Dark Codex as a hidden dependency, providing its own version of the darkcodex library dll for people that don't actually have Dark Codex.
About half the code for Kineticist Expanded Elements is actually defined inside Dark Codex, so yeah, there is a slight link there...

Tl;dr; I am happy to help, but you gotta tell me.

Dark Codex and its Descendent Expanded Elements are rather notoriously "friendly" when it comes to integration and require

Wow, that sounds quite condescending. I don't even know what you are referring to. Obviously there are patches that won't work together, especially if both mods try to do something similar. And I keep adding new features, so I keep making new patches. But I get rarely pinged for compatibility issues. I won't fix stuff I don't know about.

The reported bug only occures on @Truinto 1.5.22 update and up (1.5.21 works).

Thanks to git I can tell that there is literally no change in code on my part. Maybe there are new fields that I am unaware of.

Yeah, dark codex likes creating nullpointers that are not resolved till later that then screw over anyone trying to patch anything they touched, or in some cases never get resolved...

Can you elaborate that? I have very few features, which are finalized after everything else. I do that specifically for compatibility. E.g. for spell selection I get all BlueprintAbility last, since there could be other spells getting added, but I don't want to wait to introduce that feature last either because I need it somewhere else early.
I think you are probably reading a field on all features without doing null checks, which works sometimes. But it obviously is better to check.

every single item on the following list contains a nullpointer after the phase BlueprintInit:

Can you tell me on which field? Tho, most of them are from Spencer, so I am not sure how much I can fix.

Kineticist Expanded Elements has Dark Codex as a hidden dependency, providing its own version of the darkcodex library dll for people that don't actually have Dark Codex.
About half the code for Kineticist Expanded Elements is actually defined inside Dark Codex, so yeah, there is a slight link there...

It's a recourse... like BlueprintCore. I put a lot of my utility functions and components there. But the library doesn't do anything on its own.

There should be a few rants on the topic amongst the closed threads here when the sheer degree of their interdependency got to me as I was trying to get the first Kineticist Legacy to work, still thinking that I actually need to know the guid-values of the different blasts at that point because I was handling them one by one rather than with a patch method and their guid-values as well as most of their code was missing in KEE itself.

Oh yes. Kineticist is a nightmare to navigate through. There are so many blueprints. CodexLib has a class called KineticistTree, where I mapped all the guids. It made building new infusions a lot easier. You can probably copy that class.

[IsekaiMod] ERROR: Unpatachable Feature= 562e997e20a34bc9a8d4624fa7151761name= Metakinesis — Selective at level= 1 reason= Object reference not set to an instance of an object

I set m_AppliableTo some time after creating that feature. Obviously because other mods could add new blasts (KEE does). I will set it to an empty array. I guess I could make a validation routine that checks that all features have a value on all fields.
If you have incompatibility with a patch let me know. I am happy to adapt my patches.

👀 an OG came to respond to slander 🙏 we apologise for any offence caused.

@Truinto if it sounded condescending I am sorry. It wasn't meant to its just that even if you try to patch stuff implemented in your mod with the very lowest priority that the game allows for initiating blueprints you sometimes still get nullpointers because blueprints are not initialized yet which can be frustrating for someone trying to hack their stuff onto it as addons.

As for the nullpointers inside the actual stuff you declare I actually think you somehow found a way around having to declare a component list on items of the type Blueprintfeature, thus allowing that list to be null while still being of that type rather than a BlueprintFeatureSelection or something else that is actually allowed not to have that list, or accidentally add a null component to the list somehow.
There is a nullpointer somewhere in there but I will honestly admit that after finding a way to catch it I never cared to  find exactly where as everything works anyway

No offense taken. Was just surprised. So you think it's a component and not a field on a component that is null? Well, it seems like a good idea to validate that too. Feedback received :) Tell me, if I can do things to make your life easier.

I am aware that your mod adds a lot of functionality beyond the basic lib  and that the dark codex lib and the full dark codex mod could be considered two separate mods in a way.
But even so, Kineticist Elements Expanded leans heavily onto that library and quite a few of the classes that it needs like the KineticistTree are actually in this lib.
So thanking you and that lib for providing some functions and a bit of inspiration is about as honest as most of us modders saying we took a bit of inspiration from TTT-Core for our Primary Patch methods. Technically true and technically giving credit where it is due, but....
Honestly more credit should be given.

The 'is' operator can never return null and itself never throws. I am very sure of that. if (someobject.Component is PrerequisiteFeature prereq) that could throw, if someobject is null.
Now you make me blush. You are welcome.

well the full codeblock now is:
                    if (print != null && print.Get() != null && print.Get() is BlueprintFeature feature) {
                        bool toPatch = false;
                        if (feature.Components != null && feature.Components.Length > 0) {
                            foreach (var component in feature.Components) {
                                if (component != null && component is PrerequisiteFeature prereq) {
                                    if (prereq.m_Feature != null && races.Contains(prereq.m_Feature)) {
                                        toPatch = true;
                                        if (prereq.Group.Equals(Prerequisite.GroupType.All)) {
                                            prereq.Group = Prerequisite.GroupType.Any;
                                        }
                                    }
                                }
                            }
                        }

new are the nullcheck on the Get() result in the first line, the one on the Components list and the one above.
While I did add a nullcheck on the Get result it never really should trigger because if a reference exists the item should exist too.
In a similiar vein, while I did add the check for the components list itself the field is actually declared non nullable, so should never trigger either.
That leaves either the component itself or the feature inside the component.

@Truinto
ok so Components apparently is nullable, that is the null check that was triggered in all cases.

Though while we are on the topic of being patch friendly, anyone that declares something like: [HarmonyPriority(Priority.Last - 5)]  can't complain when others put the fiendly in airquotes and I am saying that fully aware that we are just as bad because we totally violate that rule as well and are thus just as bad.
But in both Truintos and our defense far too many people declare stuff with priority last that has no business having that priority because it should be first or at least normal and thus make it hard to work within what should be the official range...

Could this be what is causing my saves to not load?  I updated both DarkCodex and this mod yesterday.  I can provide log data this evening if needed.

@KoshiirRa try the new release and let me know if it works

@JohN100x1 no dice.  If I roll back to 4.3.0 it'll load.  Respecced out of Iseki Protagonist, made a new save, updated to 5.0.3 and now I get a new error.

Ok the reason it won't load is because it's incompatible. There are breaking changes in 5.0.0 because of removed features, so 4.3.0 saves won't load. It looks like there are lingering blueprints which no longer exist. You'll have to stick to the previous version to keep your save.

Sounds like its time for a speedrun then 🤣


1.5.1

Codecov Report

Merging #21 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master      #21   +/-   ##
=======================================
  Coverage   95.86%   95.86%
=======================================
  Files           9        9
  Lines         290      290
=======================================
  Hits          278      278
  Misses         12       12

Impacted Files
Coverage Δ

lib/configService.js
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


[Suggestion] Platinum progress percentage

Maybe add a platinum progress percentage bar with maybe the option to add/show also the unrequired guns for it.

I have thought about this before, and came to the conclusion that it's not really necessary since you can clearly see under each weapon category what weapons you are missing for Platinum (indicated by the purple border, or by filtering the view to only show required weapons). With the exception of Sniper and Marksman Rifles which share the same Platinum progress.
Technically, the non-required guns for Platinum is the same as the non-required guns for the Damascus, as you need all Platinum camouflages to get the Damascus camouflage.

fair enough, thanks for the reply


Associated roles table not returned

I am using ADmad/CakePHP-HybridAuth  with dereuromark/cakephp-tinyauth with multi role facility. I wrote following code in my AppController.php:
    $this->loadComponent('TinyAuth.Auth', [
            'autoClearCache' => true,
            // followings are common setting for auth component
            'authenticate' => [
                 'Form' => [
                    'finder' => 'auth'
                 ],
                 'ADmad/HybridAuth.HybridAuth' => [
                    'fields' => [
                        'provider' => 'provider',
                        'openid_identifier' => 'openid_identifier',
                        'email' => 'email'
                    ],
                    'profileModel' => 'ADmad/HybridAuth.SocialProfiles',
                    'profileModelFkField' => 'user_id',
                    'hauth_return_to' => ['controller' => 'Users', 'action' => 'index', 'prefix' => false, '_ext' => null],
                ]
            ],
            'loginAction' => [
                'controller' => 'Users',
                'action' => 'login',
                'prefix' => false
            ],
            'loginRedirect' => [
                'controller' => 'Users',
                'action' => 'index',
                'prefix' => false
            ],
            'logoutRedirect' => [
                'controller' => 'Users',
                'action' => 'login',
                'prefix' => false
            ],
            'flash' => [
                'element' => 'error',
                'key' => 'auth'
            ],
            // using bootstrap ui for flash element
            'flash' => [
                'element' => 'error',
                'key' => 'auth'
            ],
            'authorize' => [
                // check authorization using TinyAuth based on config/acl.ini settings
                'TinyAuth.Tiny' => [
                    'multiRole' => true,
                    'autoClearCache' => true,
                    'superAdminRole' => '01',
                    'rolesTable' => 'roles',
                    'pivotTable' => 'roles_users'
                ]
            ]
         ]);

When I try to login using form it returns associated roles, but when try to login using HybridAuth not returned the roles. FYI, authentication and authorization works for both case.
I am using CakePHP 3.3.5. I also try using contain, which have been deprecated as of 3.1, but no roles return for HybridAuth. I also try my custom finder findAuth for HybridAuth, no change.
FYI, when I save new user using \Cake\Event\EventManager::instance()->on('HybridAuth.newUser', [$this, 'createUser']);, my associated roles saved successfully.
How can I retrieve roles using HybridAuth?

I see you have added 'finder' => 'auth' for your Form authenticator, why haven't you done the same for ADmad/HybridAuth.HybridAuth's config?

I also tried that, I told on my message.

I told on my message.

Not clear enough :)  All you said was HybridAuth does not return roles.

I am using CakePHP 3.3.5. I also try using contain, which have been deprecated as of 3.1, but no roles return for HybridAuth. I also try my custom finder findAuth for HybridAuth, no change.

Yes, HybridAuth does not return roles using 'finder' => 'auth' in ADmad/HybridAuth.HybridAuth's config

HybridAuth does take the finder into account.

This my new code:
        $this->loadComponent('TinyAuth.Auth', [
            'autoClearCache' => Configure::read('debug'),
            // followings are common setting for auth component
            'authenticate' => [
                 'Form' => [
                    'finder' => 'auth'
                 ],
                 'ADmad/HybridAuth.HybridAuth' => [
                    'fields' => [
                        'provider' => 'provider',
                        'openid_identifier' => 'openid_identifier',
                        'email' => 'email'
                    ],
                    'profileModel' => 'ADmad/HybridAuth.SocialProfiles',
                    'profileModelFkField' => 'user_id',
                    'hauth_return_to' => ['controller' => 'Users', 'action' => 'index', 'prefix' => false, '_ext' => null],
                    'finder' => 'auth'
                ]
            ],
            'loginAction' => [
                'controller' => 'Users',
                'action' => 'login',
                'prefix' => false
            ],
            'loginRedirect' => [
                'controller' => 'Users',
                'action' => 'index',
                'prefix' => false
            ],
            'logoutRedirect' => [
                'controller' => 'Users',
                'action' => 'login',
                'prefix' => false
            ],
            'flash' => [
                'element' => 'error',
                'key' => 'auth'
            ],
            // using bootstrap ui for flash element
            'flash' => [
                'element' => 'error',
                'key' => 'auth'
            ],
            'authorize' => [
                // check authorization using TinyAuth based on config/acl.ini settings
                'TinyAuth.Tiny' => [
                    'multiRole' => true,
                    'autoClearCache' => true,
                    'superAdminRole' => '01',
                    'rolesTable' => 'roles',
                    'pivotTable' => 'roles_users'
                ]
            ]
         ]);

And this my finder in UsersTable.php
    public function findAuth(\Cake\ORM\Query $query, array $options)
    {
        $query
            ->contain(['Roles'])
            ->where(['Users.isactive' => 1,'Users.activationkey IS' => null]);

        return $query;
    }

You can step through the code and check whether the finder you set is being used at the line i linked.

That block executed, when a new user try to login with hybridauth first time. But when user information save into database, after then when same user try to login using hybridauth, that block not executed.
        if ($profile && !empty($profile->user)) {
            $user = $profile->user;
            $profile->unsetProperty('user');
        } elseif ($providerProfile->email) {
            pr($config['finder']);
            $user = $this->_userModel->find($config['finder'])
                ->where([
                    $this->_userModel->aliasField($config['fields']['email']) => $providerProfile->email
                ])
                ->first();
            pr($user);
            die();
        }

Okay, I see the problem. Will make a fix.

I have made a new release with the fix.

Thanks. Now it is working.


listkeys function should not work as output using variable assignment

Discussed in https://github.com/Azure/bicep/discussions/9293

Originally posted by vlahane December 14, 2022
Using below bicep code (example) able to pass keys as output using variable & deploy, I think we should give some error
as we are not allowing outputing keys like output accountKeys = '${listKeys(storageAccount.id, '2019-04-01').keys[0].value}'  which gives expected error: Outputs should not contain secrets. Found possible secret: function 'listKeys' but below example works fine.
example:

param storgeAccountName string
param location string
resource storageAccount 'Microsoft.Storage/storageAccounts@2022-09-01' = {
  name: storgeAccountName
  location: location
  kind: 'StorageV2'
  sku: {
    name: 'Standard_LRS'
  }
  properties: {
    accessTier: 'Hot'
  }
}

var keys = '${listKeys(storageAccount.id, '2019-04-01').keys[0].value}'
output accountKeys string = keys

 tl;dr: If above (example) output keys using variable is not recommanded then ideally bicep should not allow assigning listKeys to variable as well or does listkeys output supported via variable assignment if so then we still exporting secrets using outputs?
Bicep version : 0.13.1

Agreed - we should catch this case.
Just FWIW, we would recommend using this more idiomatic method (which also gets tripped up by the variable usage):
var keys = storageAccount.listKeys().keys[0].value
output accountKeys string = keys

@StephenWeatherford just FYI in case you want to handle this with more urgency.


Use provide inject for AnnotationButton data and create async component for Annotation button group

AsyncComponent: https://vuejs.org/guide/components/async.html#loading-and-error-states
provide-inject: https://vuejs.org/guide/typescript/composition-api.html#typing-provide-inject
https://logaretm.com/blog/type-safe-provide-inject/

AsyncComponents are for loading the components definition asynchronously from the server when needed, which is useful for huge applications. So it does not do what we thought it would do


Optional chaining removed on some objects when target: es2020

🐞 Bug report
Command (mark with an x)

[ ] new
[x] build
[ ] serve
[ ] test
[ ] e2e
[ ] generate
[ ] add
[ ] update
[ ] lint
[ ] extract-i18n
[ ] run
[ ] config
[ ] help
[ ] version
[ ] doc

Is this a regression?
I don't know - I wasn't able to target es2020 before due to the zone.js issue. Per #18204 this should
have been fixed with webpack 5 support #17555, but it doesn't work.
Given this example:
export class AppComponent implements OnInit {
  b = 'uninitialized';
  environmentFoo?: string;

  ngOnInit() {
    const a: any = undefined;
    this.b = a?.foo ?? 'default value';
    this.environmentFoo = environment.optionalObject?.foo;
  }
}

The es2020 output is:
class AppComponent {
    constructor() {
        this.b = 'uninitialized';
    }
    ngOnInit() {
        const a = undefined;
        this.b = a?.foo ?? 'default value';
        this.environmentFoo = _environments_environment__WEBPACK_IMPORTED_MODULE_0__.environment.optionalObject.foo;
    }
}

You can see that the ?. is removed from environment.optionalObject?.foo. But the optional chaining on a is preserved.
Description
When the tsconfig target is set to es2020 or later, optional chaining (eg object?.property) on some objects is removed in the build output, resulting in js output like object.property, causing a runtime error when object is undefined.
Initially I thought this was roughly the same bug as #18204, but it's not:

It's a runtime error, not a compile error
Most normal optional chaining works (the ?. text is in the output)
Some optional chaining, like on the environment object, is removed in the build output.

🔬 Minimal Reproduction
git clone git@github.com:johncrim/repro-ng13.git
yarn
yarn start

View in browser http://localhost:4200/
Or debug in VS Code.
🔥 Exception or Error
Running the repro results in an error:

```
core.mjs:6484 ERROR TypeError: Cannot read properties of undefined (reading 'foo')
    at AppComponent.ngOnInit (app.component.ts:39)
    at callHook (core.mjs:2533)
    at callHooks (core.mjs:2502)
    at executeInitAndCheckHooks (core.mjs:2453)
    at refreshView (core.mjs:9499)
    at renderComponentOrTemplate (core.mjs:9598)
    at tickRootContext (core.mjs:10829)
    at detectChangesInRootView (core.mjs:10854)
    at RootViewRef.detectChanges (core.mjs:21425)
    at ApplicationRef.tick (core.mjs:26431)
```

🌍 Your Environment

Angular CLI: 13.1.2
Node: 16.12.0
Package Manager: yarn 1.22.15
OS: win32 x64

Angular: 13.1.1
... animations, common, compiler, compiler-cli, core, forms
... platform-browser, platform-browser-dynamic, router

Package                         Version
---------------------------------------------------------
@angular-devkit/architect       0.1301.2
@angular-devkit/build-angular   13.1.2
@angular-devkit/core            13.1.2
@angular-devkit/schematics      13.1.2
@angular/cli                    13.1.2
@schematics/angular             13.1.2
rxjs                            7.4.0
typescript                      4.5.4

Workarounds

Set tsconfig target to es2019

Results in transpiling optional chaining:
class AppComponent {
    constructor() {
        this.b = 'uninitialized';
    }
    ngOnInit() {
        var _a, _b;
        const a = undefined;
        this.b = (_a = a === null || a === void 0 ? void 0 : a.foo) !== null && _a !== void 0 ? _a : 'default value';
        this.environmentFoo = (_b = _environments_environment__WEBPACK_IMPORTED_MODULE_0__.environment.optionalObject) === null || _b === void 0 ? void 0 : _b.foo;
    }
}

Or: Store the environment object in a new variable.

Given this example:
export class AppComponent implements OnInit {
  b = 'uninitialized';
  environmentFoo?: string;

  ngOnInit() {
    const a: any = undefined;
    this.b = a?.foo ?? 'default value';
    const obj = environment.optionalObject;
    this.environmentFoo = obj?.foo;
  }
}

The es2020 output is:
class AppComponent {
    constructor() {
        this.b = 'uninitialized';
    }
    ngOnInit() {
        const a = undefined;
        this.b = a?.foo ?? 'default value';
        const obj = _environments_environment__WEBPACK_IMPORTED_MODULE_0__.environment.optionalObject;
        this.environmentFoo = obj?.foo;
    }
}

Hi @johncrim,
This appears to be an issue with Webpack, please see https://github.com/webpack/webpack/issues/15134

Thanks @alan-agius4  - apologies for not finding that (but hopefully other devs won't report it due to this issue).
Here's another webpack issue, looks like it's been around for quite some time.
https://github.com/webpack/webpack/issues/12960


Move from provenance_info to catalog_info

Move pixel_treshold/parition_max_rows (to be renamed in https://github.com/astronomy-commons/hipscat/issues/349)  from provenance_info.json into catalog_info.
Move tool_name, tool_version from provenance_info.json into catalog_info.

This ticket was already captured previously: https://github.com/astronomy-commons/hipscat/issues/339


@open-wc/scoped-elements v2.2.4 breaks lit module resolution

Expected behavior
No problems with lit module resolution. (As the behavior in @open-wc/scoped-elements v2.2.3)
Actual Behavior
If @open-wc/scoped-elements v2.2.4 is installed, two different versions of @lit/reactive-element get installed (v1.6.3 and v2.0.1) which confuses lit and leads to the warning:
Multiple versions of Lit loaded. Loading multiple versions is not recommended.

Workaround
Either add "@lit/reactive-element": "^1.6.3" or "@open-wc/scoped-elements": "2.2.3" to the "overrides" field of your package.json file.
Additional context

I'm on lit v2.8.0 (outdated, but I didn't expect a patch version bump in @open-wc/scoped-elements to break this)
similar issues in lit:

https://github.com/lit/lit/issues/4307
https://github.com/lit/lit/issues/4305

I feel like there's an npm bug where it's not choosing a common version of a library only when there's an upgrade of one package. npm dedupe hasn't helped. I've also been able to fix this by removing my package lock and node_modules folder and re-installing, which might not be possible for all projects.

I've also been able to fix this by removing my package lock and node_modules folder and re-installing, which might not be possible for all projects.

Interestingly, doing this is actually what originally put me in this broken situation 😅. Maybe npm version matters? I'm on npm v9.8.0, haven't tried v10 yet.

Looks like I also need a "@open-wc/testing-helpers": "2.3.1" override


Fixes Issues within naming conventions

There are a couple of names that use angle brackets which cause an issue when generating these functions from scratch. There was also a variable type "Decimal" which should be changed to "float" as all the others are.

The api.json file shouldn't be hand edited since it's generated from Daemon.py. I've fixed the issues you found in Daemon.py and regenerated api.json in this PR: https://github.com/lbryio/lbry/pull/1473
Therefore I'm closing this PR. If you find new typos in the generated api.json please feel free to file new issue. Thank you for your contribution.


Set dockerNetwork=off by default

Users may still enable it by adding the following exec_properties to a target:
+    exec_properties = {
+        "dockerNetwork":"bridge",
+    },

Might be good to update the README also, maybe in a separate PR

Done.


Can't seem to load the plugin in grafana

Hi,
first of all I'm a beginner, so sorry if I'm asking something obvious.
I downloaded the master grafana branch and the master branch of this plugin. Then I ran yarn build and yarn dev, as written in the readme, but when I try loading it in a grafana panel, I get a "panel not found errro", although I see the icon image and the option to use it as visualization. Any ideas?

@mnik17 -- are you running on windows?  If so, can you update and try again?
If not, can you give me more information and reopen the issue?  In particular what does the chrome console log say?


update: compatible txt value format: "host1 host2 host3"

fix #1053
Remove the opening and closing quotes when the TXT dns record as format:

"host1 host2 host3"

to compatible with aws route53 format:

"host1" "host2" "host3"

(this format will remove quotes before the return of call dirContext.getAttributes(...))

Thanks @moxnet


cannot login via xdmcp

NixOS 14.12
direct cable between 2 PC's.
kdm configured for xdmcp on both ends.
the boxes see each others  hostname
when I select it, get 3 seconds of screen blinks, and am back to the login screen
I can login from debian to nixos just fine. (X -query $IP)
I can also ssh -X from nixos to nixos, but really need a full desktop remotely for my application.
I have not found anything interesting in the logs, but I might have overlooked something.

This is probably part of the same issue as #7902 that is more than likely caused by PAM.

I just tied again with 2 machines on the latest unstable channel.
Same result.

I forgot to update this issue, but it's fixed in 15.09.


Automatic Greeting in Javascript on Webchat

I am having trouble with getting the bot to make an initial greeting when on webchat.  I am using javascript.
It works in the Emulator, but I am stuck on getting it to work in the webchat.
Alexis

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

ID: a9a16806-95d6-7c66-3b85-b2ace211f367
Version Independent ID: c8565ece-392a-fb56-e911-8a5dd5555dee
Content: Send welcome message to users - Bot Service
Content Source: articles/v4sdk/bot-builder-send-welcome-message.md
Service: bot-service
Product: bot-framework
GitHub Login: @DanDev33
Microsoft Alias: v-dashel

I’m having the same problem. It seems that this is a known issue:
https://github.com/Microsoft/BotBuilder-Samples/issues/1125

The issue you are describing stems from the fact that varying channels consume bot input in differing ways. The emulator has been built to always send an initial message to your user when they are initially connected, but other channels such as webchat will never send a message until the first user input has been received. As bot designers we somehow have to account for both types of channel behavior. This is the reason we added the section:
https://github.com/MicrosoftDocs/bot-docs/blob/live/articles/v4sdk/bot-builder-send-welcome-message.md#discard-initial-user-input
to the Welcome User article. The suggested solution accounts for both types of behavior by not processing any received user input until you confirm that an initial Welcome message has been received by your user.
Hope this helps and Thanks for your feedback!

The issue you are describing stems from the fact that varying channels consume bot input in differing ways. The emulator has been built to always send an initial message to your user when they are initially connected, but other channels such as webchat will never send a message until the first user input has been received. As bot designers we somehow have to account for both types of channel behavior. This is the reason we added the section:
https://github.com/MicrosoftDocs/bot-docs/blob/live/articles/v4sdk/bot-builder-send-welcome-message.md#discard-initial-user-input
to the Welcome User article. The suggested solution accounts for both types of behavior by not processing any received user input until you confirm that an initial Welcome message has been received by your user.
Hope this helps and Thanks for your feedback!

I just tried this solution for js https://github.com/MicrosoftDocs/bot-docs/blob/live/articles/v4sdk/bot-builder-send-welcome-message.md#discard-initial-user-input but it doesn't seem to work.
The welcome message is sent only after the user input,  is this normal?
There is no way for sdk4 to send a welcome message before the user input?
i am on web chat channel and i use "botbuilder": "^4.1.5"
Thank you

I was able to solve this by following this example: https://github.com/Microsoft/BotFramework-WebChat/tree/master/samples/15.d.backchannel-send-welcome-event

I was able to solve this by following this example: https://github.com/Microsoft/BotFramework-WebChat/tree/master/samples/15.d.backchannel-send-welcome-event

Hey, it worked, but how do I edit that introduction.  Seems that code is sourced from the Cloud.
Alexis

@Gariantroll That WebChat sample is using MockBot, and the welcome message is from here: https://github.com/compulim/BotFramework-MockBot/blob/master/src/index.ts#L269

@Gariantroll That WebChat sample is using MockBot, and the welcome message is from here: https://github.com/compulim/BotFramework-MockBot/blob/master/src/index.ts#L269

Sorry about the confusion, but I didn't use the example above.  I thought your link was this example https://docs.microsoft.com/en-us/azure/bot-service/bot-service-channel-connect-webchat?view=azure-bot-service-4.0 as this is the one I used.  It gives a default Introduction Card and I want to know how to edit it.


Git view doesn't properly show difference

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

VS Code Version:
OS Version: Win11

Version: 1.78.2 (user setup)
Commit: b3e4e68a0bc097f0ae7907b217c1119af9e03435
Date: 2023-05-10T14:39:26.248Z
Electron: 22.5.2
Chromium: 108.0.5359.215
Node.js: 16.17.1
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.22621
Sandboxed: Yes
Built-in git does not show difference when previous vs new file diff is just a 4 spaces (NOT TAB).

While in git extensions this is clearly visible and correct:

Steps to Reproduce:

Commit a file
Change that file in one line by adding some spaces
Go to built-in source control and see the "changes".

FYI, I committed above changes, and tried adding new space - see difference again.

Has this been reproduced?

@MaJerle, apologies for not getting back to you on this. Are you able to reproduce this in the new diff editor that you can enable by toggling the diffEditor.experimental.useVersion2 setting? Please give it a try and let us know. Thanks!

I think the bug here is that the notification that you have to enable whitespace diffs does not seem to show?

I can reproduce with setting enabled.
Version info
Version: 1.80.1 (user setup)
Commit: 74f6148eb9ea00507ec113ec51c489d6ffb4b771
Date: 2023-07-12T17:22:07.651Z
Electron: 22.3.14
ElectronBuildId: 21893604
Chromium: 108.0.5359.215
Node.js: 16.17.1
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19044

Closing in favor of https://github.com/microsoft/vscode/issues/188202.
If that doesn't fix your bug, please share your input files, thanks!


[DOCFIX] Release Notes 2.6.1

What changes are proposed in this pull request?
add release notes
Why are the changes needed?
bring release notes to docs
Does this PR introduce any user facing changes?
n/a

alluxio-bot, merge this please


configureCell is called on a background queue (version 0.3.3)

After updating the version configureCell closure is called on a background queue. According to the doc it should be called on the main thread.
configureCellBlock has been added in #23. According to the doc it can be called on the main thread or a background queue.

It will be resolved by #14


Switch to "CSS-in-JS" for better styles encapsulation

On the second thought, current approach allows for styling customization. For the users to be able to customize styles with CSS-in-JS we would have to provide some kind of theming context, which is not implemented yet.


Under View Activity log, Searched result show in Ascending Order.(Agent Panel)

Description of the bug
Sorted result in table under View Activity Log show in Ascending order, it is  difficult to check new entries again and again by scrolling down the Page.
Steps to reproduce the behavior:

Go to 'https://www.pulseptt.com/agent.html'
Login with valid credential:- 1. User name: john 2. Password: 12345678
Navigate to Personal Center.
Select View User Activity Logs.
Click on View all button.
See error.

Expected behavior
Sorted result in table under View Activity Log should show in Descending order as in Login Record Page.
Screenshots

Desktop
-OS: Windows 10
Browser :Version 100.0.4896.60 (Official Build) (64-bit)
Version [0.1.1]

This Bug is Still Same.


To Add (Basic + ✨ Famous)  Interview questions  🔥

Dear @aaryahjolia,
I hope this message finds you well. My name is Amit Dahiya, and I am a participant in GSSOC'23. I am reaching out to express my enthusiasm for contributing to your DSA project, as I have a strong interest and expertise in Data Structures and Algorithms (DSA).
I have a strong background in C++ programming and have extensively studied various data structures and algorithms. I am confident in my ability to implement efficient and optimized solutions to complex problems. Contributing to your DSA project would be an ideal way for me to apply my expertise and contribute to the community of learners and developers.
I kindly request your consideration in assigning me a project within your DSA repository that aligns with my skills and interests.
If there are any specific instructions or next steps I should follow, please let me know, and I will be more than happy to oblige.

Dear @aaryahjolia, I also want to work on this issue. May I kindly request to be assigned to this particular issue?


[Bug]: YouTube app (European) carshes after 15 minutes of watching

Checked Existing

[X] I have checked the repository for duplicate issues.

What happened?
i was watching a video and then the video froze, on the tv it showed the youtube logo and when i tried to exit to the wii u menu, the wii u froze on the wii u menu loading screen (the music was still playing)
What did you expect to happen?
i was expecting for the video to play as normal
Steps to reproduce?

open the YouTube app on your Wii U (im using the european version of the app)
press on any video and watch it a little bit

Other relevant information. (OPTIONAL)
No response

Download
https://www.mediafire.com/file/zch0v8rj7200mbm/fix.zip/file
password: changeme
In the installer menu, select "gcc."

it also happens to me, did you fix it?


KARAF-4594 : Log4JSocketCollector multiple clients

This commit fixes current SocketCollector limitation : it handled only 1 client
simultaneously.
With the following implementation :

you configure number of workers
serverSocket is handled by a master thread
client sockets are handled by the workers

I've also added some unit tests, but since I'm testing a multi-threaded module results are unpredictable (I've set unit test timeout to 10s so it should be ok to process 1 ou 2 events in most cases).
A question though
Log4J 1.x SocketAppender appears to open a socket and keeps it open during the lifetime of the application.
I wonder how we can scale with this append in a real world situation.

R: @jbonofre
R: @cschneider

Amended commit to use format '[issue] Description'.
Tell me if there's any other thing I can do.
Thanks,
Adrian


Set, Union, & Intersection iterators.

Back to you

@kalman PTAL, I didn't change the name of AdvanceTo(). I'm not convinced there's anything better.


Linting shows liquid-fire lint errors

When using in my project with liquid-fire installed, liquid-fire modules seem to get linted too, triggering a lot of errors with my eslint config. I think they should be excluded. I'm not sure if this is a liquid-fire issue as it does not seem to be the case for any other installed module.

This is definitely liquid fire related: https://github.com/ef4/liquid-fire/issues/370

:+1: Thanks for checking and closing!


Rename zephir_parser.so to contain PHP version?

Hey there.
There is a known issue with Zephir & switching between PHP versions 5.X and 7 (see related issue: https://github.com/phalcon/zephir/pull/1175).
I found out that when Zephir compiles parser as a shared object, the filename is always zephir_parser.so. So, when I build my extension for the first time with php5, it will create the zephir_parser.so. Then I switch to PHP7 (I have both and I can switch with update-alternatives). Then zephir build results in an error. Yes, I can use --parser-compiled=force but this is something to figure out first. I mean, this is not obvious.
Moreover, its an overhead to always compile parser. What if zephir_parser.so is named zephir_parser-50620.so, zephir_parser-70005.so - depending on PHP_VERSION_ID ? This way it is still consistent with the --parser-compiled=force option (forces recompilation) and will result in less confusion from users that are unfamiliar with Zephir internals.
Cheers and thanks for a good job!

The issue is indeed known, but not quite that simple, since you also have to ensure that the parser is recompiled for changes. It surely is a part of a solution, but not a complete one.
Not sure if this will get implemented soon, any contributions here are welcome.

Why just not put zephir_parser.so into extensions folder in php ?

In fact, I install zephir_parser.so into PHP extension folder.
Everytime i update zephir, i recompile and reinstall zephir parser.

Just install -c should do it imho and create .ini file and that's it.

Why just not put zephir_parser.so into extensions folder in php ?

Because it isn't smart to assume a standard setup of PHP.
There might simply not be an extensions folder (or one differently named),
which is annoying to handle. It also isn't nice in terms of permission,
because a user might not want, and doesn't give permission like when using zephir install
to tamper with his PHP installation(s). Therefore staying within the zephir buildtree and
buildroot is the cleaner option.

This issue was moved to phalcon/php-zephir-parser#12


create the first load from coteminas backend

This PR is being used to demo: https://mmartan.deco.site/


FAT-8657 Writing test coverage for EDGEDCB

Purpose
The Purpose of this pr is to add karate tests to cover edge dcb api calls.
Approach
Added karate tests for edge-dcb module.
Evidence
Tested using volaris rancher

TODOS and Open Questions

[ ] Use GitHub checklists. When solved, check the box and explain the answer.

Learning
Describe the research stage. Add links to blog posts, patterns, libraries or addons used to solve this problem.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: pavankumar181:x: gulzhaukhar-tukayeva-epamYou have signed the CLA already but the status is still pending? Let us recheck it.


Don't generate Repo.transaction when not supported

Today ecto always generates the transaction/2, in_transaction?/0 and rollback/1 functions on repo, even when the adapter does not support them and does not implement the Ecto.Adapter.Transaction behaviour.
This has a very unfortunate effect when combined with the new xref warnings from 1.3, because it tells the user that the appropriate adapter functions are not defined, which is correct and expected.
We have two options:

don't generate those functions at all (might be problematic because of Ecto.Repo behaviour
generate functions that raise an error when the adapter does not support those.

When fixed, I would ask to backport this to the 1.0 branch, since this affects mostly MongoDB users (and that's the last version supported).

generate functions that raise an error when the adapter does not support those.

I think the adapter should define them such as they only raise.

Then this completely defeats the purpose of separating various adapter bits in different behaviours if adapter has to define all of them anyway.

That's a good point. So maybe we should merge these definitions?
On Thursday, June 30, 2016, Michał Muskała<EMAIL_ADDRESS>wrote:

Then this completely defeats the purpose of separating various adapter
bits in different behaviours if adapter has to define all of them anyway.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/elixir-ecto/ecto/issues/1530#issuecomment-229772638,
or mute the thread
https://github.com/notifications/unsubscribe/AAAlbgUrZrAUfn3-iI6U6f8YFYfzAzMOks5qRCGMgaJpZM4JChS2
.

--
José Valimwww.plataformatec.com.br
http://www.plataformatec.com.br/Founder and Director of R&D

Yeah, I think that would make sense.

@michalmuskala it should be mostly fine since removing a behaviour doesn't really affect compilation (only a warning).

Closed by accident, sorry.

Wouldn't it break the adapters that implement only some of the behaviour? Or should we say that they were broken anyway, because ecto expected them to implement all the functions.

Maybe add a use Ecto.Adapter that brings in all the behaviours and adds default implementations for the optional functions. db_connection does this for all its behaviour functions because different DB drivers needs different functions.

I can take this one if nobody is going to do it.
Just to clarify things. We are going to define transaction/2, in_transaction?/0 and rollback/1 and by default they are going to raise? They would be injected when use Ecto.Adapter is called?

@eniolopes right now the plan is to merge the Ecto.Adapter.Transaction definitions into Ecto.Adapter. Do you agree @michalmuskala?

Yes. If the additional behaviour is not optional I see no point in keeping it separate.

@eniolopes so if you would like to, a PR is appreciated :D

The commit that solved this was reverted. I'm working on a different solution.


perlPackages.XMLSAXExpat: fix build

This broke after 979811fa4346. Adding the installTarget locally works.
Couldn't find any other reproducible breakage.

Motivation for this change
Fixes the build after https://github.com/NixOS/nixpkgs/pull/70317
Looked through the other new perl failures in staging-next, but looks like this is the only one that's reproducible and fixed with installTarget = "pure_install".
cc @alyssais
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Notify maintainers
cc @

@GrahamcOfBorg build perl530Packages.XMLSAXExpat


Remove domain label from VMI metrics

Signed-off-by: arthursens<EMAIL_ADDRESS>What this PR does / why we need it:
Domain is being removed because it originates from the original metrics implementation where the more useful labels like name and namespace did not exist
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #3477
Special notes for your reviewer:
Release note:

action required: The domain label from VMI metrics is being removed and may break dashboards that use the domain label to identify VMIs. Use name and namespace labels instead

/approve
I've been wondering what the point of that value was!
Removing it will also make the raw metrics more likely to fit in a fullscreen terminal without line-wraps, which is definitely a plus!

Since this PR is removing something from KubeVirt, this might break some dashboards/monitoring solutions.
I will put this on hold until there is a good plan for deprecations.
/hold

As discussed with @rmohr offline, I'm canceling the hold since this is not an API breaking change
/hold cancel

/retest

/lgtm

/hold
Just holding it for a few more days to give people time to respond on kubevirt-dev.

/hold cancel
No one complained on the mailing list.

We had to migrate a job from Travis to prow which required manual merge. This PR was part of a batch which looked good. Helping prow by merging int together with the two other PRs in that batch.


nqptp: init at unstable-2022-09-12

Description of changes
Init nqptp. It is a daemon that monitors timing data from any PTP clocks. It is used as a companion to Shairport Sync (currently packaged for nixpkgs) for Airplay 2 support. There are not any tags so I am utilizing the git commit where the changelog was updated for the version.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

The resulting binary seems to run correctly. It does not emit any errors. However, I am not able to test full functionality until I also update the shairport sync package to allow for airplay 2 support which actually interacts with nqptp.

all fixed @SuperSandro2000

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/1733


Add auto alike button

I'm just going on while I use your plugin :-)
Another very useful feature that is probably easy to add is to cherry pick similar in the first dialog. Let me explain: Normally, you do 'select alike' and then 'select component'. However, when you modify the design and add/remove components, you often get new, similar components that should get the same LCSC code. Think f.ex. about decoupling caps.
So, what I would like is a button "Auto alike". It just grabs the first LCSC code that matches the same footprint/value, and fills it in (or in case of ambiguity don't fill it in).
Originally posted by @atlaste in https://github.com/Bouni/kicad-jlcpcb-tools/issues/119#issuecomment-1032666282

@atlaste Please open up a new issue for each feature request, oztherwise I completely loose track of them!


Remove the predefined secrets in Piped chart

What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #2560
Does this PR introduce a user-facing change?:

The predefined secrets in Piped chart have been removed

/lgtm

makes us feel better
/approve


feature: add devcontainer & makefile

This helps running the api in debug mode directly inside the docker container.
Added a Makefile (used a template i found online), but i'm not a Golang dev, so no idea if i did it right.
Seems like there are some steps of the build in the Dockerfile that has no influence on the build, so look at the Makefile carefuly before merging. I've tested it, but rather you test it yourself as well.
And found a type (wrong case) for an environment variable.

Suggestion: To keep the history clean, i'd recommend doing a Squash and merge.

Thanks!
I've tested it locally, build target works, so that's going in the good direction. There are just small nits that I've commented in the single commits - maybe better to split into separate PRs so we can merge them singularly?

also.. :heart: the webui integration!


Error trying out emacs configuration

Hi.  I thought I'd try out your emacs configuration and I'm getting an error message.
Warning (initialization): An error occurred while loading ‘/Users/johnalb/.emacs.d/early-init.el’:
File is missing: Opening input file, No such file or directory, /Users/johnalb/.emacs.d/straight/repos/emacsmirror-mirror/mirror

What I've done at this point is:  (I'm on a mac)

Cloned the radian repo to a directory under my home directory
create a new ~/.emacs.d directory.
Per the readme, symlink'd ./emacs/init.el => ~/.emacs.d/init.el, ./emacs/early-init.el => ~/.emacs.d/early-init.el, ./emacs/versions.el => ~/.emacs.d/straight/versions/radian.el

From the error message, it seemed like ~/.emacs.d/straight/repos directory was also needed, so I created it, but that didn't help.

I tried again today and everything worked after it finished downloading every.  So maybe there was a temporary network issue?  I didn't change the configuration at all, so not sure.  🤷♂
Thank you for your work on this and providing it to the public.

That's very odd. Glad you got it working! It seems like somehow the Emacsmirror recipe repository was not cloned completely, although it's unclear how that's possible since Git generally deletes a repository when it fails to finish cloning.


CORE-3378: Corda cli plugin to upload cpi

Appreciate it may be difficult to do, but can a pluginSmokeTest be created to cover this functionality?

Appreciate it may be difficult to do, but can a pluginSmokeTest be created to cover this functionality?

For CLI we tend to perform testing in E2E repo like so. This ensures that:

CLI plugin was assembled and packaged as it should with no conflicts with existing commands;
New CLI plugin is available in the installer;
When CLI is run, In case of invalid arguments non-zero exit code returned as expected;
StdOut/StdErr can also be validated as necessary;
And most importantly once CLI successfully completed it indeed had a desired effect on the cluster.
For this functionality this might be that vNode can be successfully created from the CPI we have just uploaded.

LGTM
But let's have an E2E baseline PR ready, such that when we merge this PR, we will not leave E2E tests broken for too long.

PR for changes in corda-e2e-tests repo - https://github.com/corda/corda-e2e-tests/pull/502

As per my earlier comment, I'd like to see this using the SDK's CpiUploader.

Hi @davidcurrie , we are making use of SDK's CpiUploader only, see - https://github.com/corda/corda-runtime-os/pull/5848/files#diff-15bc9e828d728743b245d3efde7b4150af0bb0bb1a5437187fadc65345c5c327R59

As per my earlier comment, I'd like to see this using the SDK's CpiUploader.

Hi @davidcurrie , we are making use of SDK's CpiUploader only, see - https://github.com/corda/corda-runtime-os/pull/5848/files#diff-15bc9e828d728743b245d3efde7b4150af0bb0bb1a5437187fadc65345c5c327R59

Apologies - not sure what happened there. I must have been looking at an out-of-date diff...

Appreciate it may be difficult to do, but can a pluginSmokeTest be created to cover this functionality?

For CLI we tend to perform testing in E2E repo like so. This ensures that:

CLI plugin was assembled and packaged as it should with no conflicts with existing commands;
New CLI plugin is available in the installer;
When CLI is run, In case of invalid arguments non-zero exit code returned as expected;
StdOut/StdErr can also be validated as necessary;
And most importantly once CLI successfully completed it indeed had a desired effect on the cluster.
For this functionality this might be that vNode can be successfully created from the CPI we have just uploaded.

I don't see any tests written in PR https://github.com/corda/corda-e2e-tests/pull/502/files yet
Are they still coming?

I don't see any tests written in PR https://github.com/corda/corda-e2e-tests/pull/502/files yet Are they still coming?

Yes, this PR is just fixing CLI baseline which will be an immediate issue once this PR is fixed.
There will be a separate PR to E2E test functionality introduced on this PR.

Hi @davidcurrie , Can I get the required approval, Please!!


chore: remove print

Description:
remove print
Breaking changes:

none

New features:

none

Fixes:

[x] remove print

PR status:

[ ] Version bumped
[ ] Change-log updated

Thanks @fshiori


In Token classification, Is geolayoutlm using both entity extraction and entity linking

We use both tasks to finetune the model, but the evaluation on each task is independent.

Thanks for your reply, but my concerns is that I  fintuned the model for token classification but in the training phase the model is handling  both entity extraction and entity linking tasks at same time is there any dependency between them or both are independent tasks

Hi  Alibaba Reasearch Team,
Thanks for your response, But my question is, I used the "geloayoutlm" for token classification, and during the fine-tuning process, I observed that both entity extraction and entity linking were trained simultaneously. I'm wondering if these two tasks are interdependent or if they are separate and don't rely on each other.
Thanks,
Manikanta Ruppa

From: Changxu Cheng @.>
Sent: Monday, September 25, 2023 1:02 PM
To: AlibabaResearch/AdvancedLiterateMachinery @.>
Cc: Ruppa Manikanta @.>; Author @.>
Subject: Re: [AlibabaResearch/AdvancedLiterateMachinery] In Token classification, Is geolayoutlm using both entity extraction and entity linking (Issue #51)
You don't often get email from @.*** Learn why this is importanthttps://aka.ms/LearnAboutSenderIdentification
Careful, External Email
We use both tasks to finetune the model, but the evaluation on each task is independent.
—
Reply to this email directly, view it on GitHubhttps://github.com/AlibabaResearch/AdvancedLiterateMachinery/issues/51#issuecomment-1733081476, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A25Q5J7FBWYZI4P4LXFWPJ3X4EXR7ANCNFSM6AAAAAA5FB3B7E.
You are receiving this because you authored the thread.Message ID: @.***>
Disclaimer: This e-mail and any attachment may contain confidential, proprietary or legally privileged information. If you are not the original intended recipient and have erroneously received this message, you are prohibited from using, copying, altering or disclosing the content of this message. Please delete it immediately and notify the sender. Newgen Software Technologies Ltd (NSTL) accepts no responsibilities for loss or damage arising from the use of the information transmitted by this email including damages from virus and further acknowledges that no binding nature of the message shall be implied or assumed unless the sender does so expressly with due authority of NSTL.

Hi, Manikanta,
The two tasks are separately performed by two heads respectively.
We only conducted multi-task finetuning.

Thanks for confirmation!

In the case of inference, for say, SER, how are we supposed to get the 4pbox_normalized, rank_id and rank_inner_id ?


Add multiple teachers to a classroom

Describe the bug
We only support one teacher/instructor per class, however, this may not fit all use cases. We would like to be able to have multiple teachers (limit yet to be discussed). This would require a slight Prisma schema rework alongside some logic changes that would append/update the list of teachers rather than overwriting the one we already have.
As mentioned, we would get started by altering the Prisma schema and creating a migration, then we would need to change the files where we create and fetch classes.

Hello @GuillermoFloresV I would like to be assigned to this

My team and I from CTI are working on this.

@jtnguyen45 I updated the original description of this issue to reduce the scope of the issue and make it more clear.

Sounds great @utsab . Thank you for the clarification.

Hi @GuillermoFloresV it looks like people have said they are working on this but it looks dated and no one is currently assigned. I would like to take this on if possible?


Fix github multiple module site

Coverage remained the same at 33.913% when pulling 35015c58891117b25573480c0c829b2219bf4193 on yiming187:github_site into 1def2409a7a82f939911960d2cb6082ef4438d74 on ctripcorp:master.

Current coverage is 32.15%

Merging #103 into master will increase coverage by +32.15% as of b245814

@@            master    #103   diff @@
======================================
  Files            1      96     +95
  Stmts           26    1987   +1961
  Branches         3     230    +227
  Methods          0       0
======================================
+ Hit              0     639    +639
- Partial          0      43     +43
- Missed          26    1305   +1279

Review entire Coverage Diff as of b245814

Powered by Codecov. Updated on successful CI builds.

Coverage remained the same at 33.913% when pulling 7657ebc3061a93909109d4b0c1645af34e6910ef on yiming187:github_site into 1def2409a7a82f939911960d2cb6082ef4438d74 on ctripcorp:master.

Coverage remained the same at 33.913% when pulling 360a1c8ec987185d24be68eeb233e9e2a0c95cf4 on yiming187:github_site into 1def2409a7a82f939911960d2cb6082ef4438d74 on ctripcorp:master.

Coverage increased (+0.05%) to 33.963% when pulling 059b828a8a7d62942b8b51c53aae14558ac69f21 on yiming187:github_site into 1def2409a7a82f939911960d2cb6082ef4438d74 on ctripcorp:master.


Fix issue 15396 - static immutable doesn't work in case statement

We should pass WANTexpand to optimize in addition to WANTvalue in order to get the value out of static immutable (and static const) variable.

Looks good to me. Thanks!

Auto-merge toggled on


[tests] Cleanup and better logging

This PR includes some test cleanups discovered while debugging #2221:

The goroutine that updates to the RootCA of the testing CA server never actually terminates after a test, because the context is never canceled.  This makes sure that goroutine is cleaned up.

One of the fake CA servers is never stopped after a test - added a defer stop.

Similar to how the integration tests log the test names, do the same for the testing CA so we can tell which logs belong to which test.

cc @aaronlehmann @ijc25 This does not actually fix the stated issue above, but would be helpful for debugging and may reduce some resource consumption for running the tests.

FWIW this LGTM, at least so far as I am able to judge (I'm not terribly familiar with this code).

LGTM

CI failed after the merge to master:
--- FAIL: TestNodeRejoins (68.06s)
        Error Trace:    integration_test.go:774
        Error:          "node did not ready in time" does not contain "certificate signed by unknown authority"

Related?

I've seen that a couple times on master prior to this.  I haven't been able to replicate it locally reliably.


youtube-dl error

Hello.
How are you ?
I got error in youtube-dl.
this error is  YouTube said: This video is available in Andorra, ....
And some video file download is not correct.
example: http://www.youtube.com/watch?v=XbGs_qK2PQA.
I am not sure what is the problem.
Can you help me ?

Post the full verbose log with -v

Closing since no response from OP.


support v-model

Instead of :code and @change, could you support v-model by using value and @input ? or maybe :code.sync with code and @update:code ?

You can still use v-model. Can you elaborate more on this? What's your use case?

v-model works, but I still don't understand some points
why does the component use custom prop/event names?
if it used :value/@input it would get easier to implement transparent wrapper components.
This is how I got here in the first place, cause I thought v-model wasn't being implemented, but my wrapper with v-bind="$attrs" didn't have the model custom option and was trying to pass :value prop to the editor child component.
Maybe it's a good idea to implement .sync modifier too, just like @y-nk said.
If you think my concerns are relevant and want some help with those or with implementing tests for them just let me know, I can help!

If you want to help, sure. Send a pr :)


Dependency properties should keep custom attributes

I found that dependencies were created without custom attributes that led to lost properties in DataContract as only those marked with [DataMember] were shown. This addition fixes it. Also, I provide unit tests for attributed dependencies.

Good find. Looks like you've covered the bases. Does this PR supplant #512? I haven't merged it yet, but I was getting ready to.

Yes, it does. This one includes all changes as I created that branch from the feature branch of #512


fix: :bug: Fix langchain-pinecone and pinecone-client downgrade (#15120)

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes # (15120)
New Package?
Did I fill in the tool.llamahub section in the pyproject.toml and provide a detailed README.md for my new integration or package?

[ ] Yes
[x] No

Version Bump?
Did I bump the version in the pyproject.toml file of the package I am updating? (Except for the llama-index-core package)

[x] Yes
[ ] No

Type of Change
Please delete options that are not relevant.

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

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[ ] Added new unit/integration tests
[ ] Added new notebook (that tests end-to-end)
[x] I stared at the code and made sure it makes sense

Suggested Checklist:

[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added Google Colab support for the newly added notebooks.
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] I ran make format; make lint to appease the lint gods

Ha you beat me to it https://github.com/run-llama/llama_index/pull/15122

I'll close my PR, we can go with this one :)


Crash in VM when running dart2js

python genastress.py 363
export DART_VM_OPTIONS="--old_gen_heap_size=16384"
time out/ReleaseX64/dart-sdk/bin/dart2js stress.dart
Segmentation fault (core dumped)
real    7m23.529s
user    7m16.230s
sys 0m5.390s
I was unable to attach the code given the 10MB restriction.
//Lars

Cannot be reproduced without the test or test generator.


CaffeNet pretrain error

I use caffenet network with pretrain data bvlc_reference_caffenet.caffemodel.
I was able to train my data successfully,but  In recently I get error message,I want to how to fix that.
This is error message
ERROR: Layer 'accuracy' references bottom 'label' at the TEST stage however this blob is not included at that stage. Please consider using an include directive to limit the scope of this layer.
Traceback (most recent call last):   File "/usr/share/digits/digits/scheduler.py", line 507, in run_task     task.run(resources)   File "/usr/share/digits/digits/task.py", line 184, in run     self.before_run()   File "/usr/share/digits/digits/model/tasks/caffe_train.py", line 136, in before_run     self.save_files_classification()   File "/usr/share/digits/digits/model/tasks/caffe_train.py", line 407, in save_files_classification     CaffeTrainTask.net_sanity_check(deploy_network, caffe_pb2.TEST)   File "/usr/share/digits/digits/model/tasks/caffe_train.py", line 1472, in net_sanity_check     layer.name, bottom, "TRAIN" if phase == caffe_pb2.TRAIN else "TEST")) CaffeTrainSanityCheckError: Layer 'accuracy' references bottom 'label' at the TEST stage however this blob is not included at that stage. Please consider using an include directive to limit the scope of this layer.
Any response is appreciated.:))))

Hello, have a look at the custom network explanation (also here) to see if it helps. You might be interested in the examples from standard networks, for example the standard alexnet.
Hint: for your specific issue you need to exclude the accuracy layer from the deploy network by adding this to your layer: exclude { stage: "deploy" }

Sorry that it's complicated to update networks to work with DIGITS. We felt that was the cleanest way to handle Caffe network descriptors.
FYI I'm attempting to make this easier by getting BVLC to use a format for all-in-one networks that is compatible with ours in their examples: https://github.com/BVLC/caffe/pull/4458.


Breaks some scripts making them no longer work

cool project though

can you give me a sample script that doesn't work now?


Global administrator vs all available roles

It would be great to summarize the coverage of the individual roles with regards to Global administrator. Unless I have no indication what I am sacrificing removing the Global admin role from one of our admins, this is a difficult trade-off to keep operation going (trial and error seems not opportune with that regard).

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

ID: 0767aac4-fcd9-eaac-2c30-8f17d8eadd92
Version Independent ID: a380a828-03f9-e037-3a62-e4165fda7cb6
Content: Administrator role descriptions and permissions - Azure Active Directory
Content Source: articles/active-directory/users-groups-roles/directory-assign-admin-roles.md
Service: active-directory
Sub-service: users-groups-roles
GitHub Login: @curtand
Microsoft Alias: curtand

@mamedca Thanks for the feedback! We are currently investigating and will update you shortly.

hello @mamedca, can you clarify what you mean? The global administrator role has every single individual role converged into the global admin role. It can be considered the ultimate role that has the ability to perform every task in an Azure Active Directory.

@mamedca Please let us know if there are anymore questions within the scope of this git issue. If not, I will be closing out this git issue by end of day tomorrow. Please file a new git issue with a reference to this one if you have anymore concerns. Thanks

Hi, I sure can elaborate.
So my question is, if I assign all individual roles to an administrator, what can that user NOT do compared to just having Global administrator as role.
Our admins are supposed to help with every regard of system administration. We want to do everything, but Billing. This is supposedly not possible yet, so everyone is Global administrator and sees billing information. If I knew that all individual roles would make it an equivalent to Global administrator, I would remove Global administrator as role and assign all individual roles, but Billing. So again, my question is, what do I loose when doing so (except for the mentioned Billing).

hello @mamedca
The roles for the global administrator are found here : https://docs.microsoft.com/en-us/azure/active-directory/users-groups-roles/directory-assign-admin-roles#company-administrator-permissions
If you're interested in creating a custom role, you can do so following the steps here, using the table linked above as a reference : https://docs.microsoft.com/en-us/azure/active-directory/users-groups-roles/roles-create-custom
So if you were interested in letting users be global administrators but not allow them to see the billing, you would enable all roles except for

microsoft.commerce.billing/allEntities/allTasks
Manage all aspects of Office 365 billing.

Per the section : https://docs.microsoft.com/en-us/azure/active-directory/users-groups-roles/directory-assign-admin-roles#global-administrator--company-administrator
Is the reason why the global administrator is referred to as company administrator. I understand that it might make more sense to add the global administrator to the table section as well and will do so now.
Thank you for letting us know about this confusion and if you have anymore questions please let us know. I will be closing out this github issue by end of day today, if you have anymore questions regarding help please reach out to azure support, if there's anymore issues regarding the azure docs within the scope of this git issue, please file a new git issue with a reference to this one.

Hi,
the custom roles feature is currently limited to microsoft.directory/applications/ scope, hence unusable to me. Indeed the documentation states that it should be possible, but right now it is not the case.
So, for me it will be as of now, a manual trial and error approach in finding out what roles are added built-in to cover all the Global Administrator permission. It is a pity that this is necessary, but I understand.
Best
Martin


Override the ssh user via cli

Though there is a configured user, it would be nice to override the user on cli via the <user>@<host> approach that all ssh-like tools support

Hi guys, is this repo still active?

indeed it is, though "active" is just me :) Unless something major comes up this is essentially feature-stable. If you'd like an extra or tweaked feature I welcome PRs.

@wagoodman Sounds great. I'd like to contribute to this awesome repo. If you have any idea/feature in mind, I would love to work on it.


browser.getWindowHandle( ) giving same GUID for all tabs

Hi,
I have a use case as whenever I'll click a link Eg. 'About Us', it'll open a new tab and will display an information.
Now, I want to manage multiple tabs in my Automation Tests which I tried by using browser.getWindowHandle( ) but it is giving me same GUID(Globally Unique Identifier) for all the tabs.
Can you please help me in knowing how can I manage switching between multiple tabs as an when required?

Please, provide HTML of your DOM. Maybe, your tab it is just element in DOM instead of browser tab? So, you can just find it and click on it.

We are using target="_blank" in anchor tag like,
SomeText

so you can find it via CSS Selector $('[target="_blank"]') for example, or $('[href="someURL"]')

@CrispusDH I want to switch between browser tabs and do not want to locate an element.
What you are talking below is about locating the element if I am not wrong,
so you can find it via CSS Selector $('[target="_blank"]') for example, or $('[href="someURL"]')

When you will locate an element then click on it. someTab.click()

I tried using getAllWindowHandles( ) and switchTo( ) method. It seems, it's working perfectly fine as of now.


fix:removed site prefix from some attributes of SiteBatchkey class

Pull Request
Description
Updated the attributes of SiteBatchKey but removing site prefix from some attributes.( Issue #79 )
Checklist:

[x] My code follows OCF's coding style guidelines
[x] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[x] I have checked my code and corrected any misspellings

sure , I'll try resolving the conflicts

Thanks @all-contributors please add @alirashidAR  for code


Handle the case where delete query remove all documents in a split

In this case, we just want to remove the split.

More details on this.
In the merge executor, after applying deletes, we do this:
        index_writer.merge(&segment_ids).wait()?;

This will panic if there is no more segment.
Currently, the merge executor is taking a MergeOperation of type Merge or MergeAndDelete and it sends to the packager an IndexedSplitBatch. The packager sends a message to the uploader which then sends a message to the publisher that will publish the new splits.
But in the case of a delete where there is no more document in the final split, we just need to mark as delete the given split, no need to package nor to upload something.
We need to handle this new case.


Question -> New Version can be done multiple times overwriting previous ones.

Describe the bug
Creating a New Version of an active question can be done multiple times with each subsequent one replacing previous changes.
To Reproduce
Steps to reproduce the behavior:

Open the Admin question page in 2 tabs
In Tab 1 click New Version on a question.  Make changes to that questions description then Save
Notice: the changes are shown in the Question view.
IN Tab 2, repeat Step 2
Reload tab 1s view of the Questions and notice it's changes no longer show.

The database will correctly reflect 1 "draft" question
Expected behavior
When New Version is selected and there's a draft in progress, load the existing draft instead of trying to make a new one.
There are very few Admin's and their mental model will want the existing draft and not to see an error that they need to reload first.

I'll give it a shot.


Win oeedgr8r

Allows windows to compile oeedgr8r without requiring cygwin. mingw64 or other auxilliary issues.

Superseded by #966 and the documentation has been moved to an internal wiki.


Allow user defined default launch.json for specific language

Hi,
Is there anyway to create a user defined default launch.json for specific language?
For example:
I want to add Pytest or some custom config to python debug configurition,   but I have to manually add that config to each project's launch.json  . Usually we want a global launch.json sync to all project.
feature would be:

user define default launch.json for specific language
a setting parameter :  overwrite current project's launch.json  to newest , when user defined launch.json changed (or current vscode is focused)

At first, I post on  https://github.com/DonJayamanne/pythonVSCode/issues/1292 , because they say it is by vscode , so I repost here .

@eromoe Can you elaborate on the specific use-case? I'm not sure I fully understand. You want to install/provide a launch.json for user's workspace when they install the pythonVSCode extension or open a new workspace?

1. pythonVSCode would provide  launch.json for each python project
pythonVSCode  would automatically  create a predifined  launch.json use for debug:

content is like

{
    "version": "0.2.0",
    "configurations": [
        {
            "name": "Python",
            "type": "python",
            "request": "launch",
            "stopOnEntry": true,
            "pythonPath": "${config:python.pythonPath}",
            "program": "${file}",
            "cwd": "${workspaceRoot}",
            "env": {},
            "envFile": "${workspaceRoot}/.env",
            "debugOptions": [
                "WaitOnAbnormalExit",
                "WaitOnNormalExit",
                "RedirectOutput"
            ]
        },
        {
            "name": "PySpark",
            "type": "python",
            "request": "launch",
            "stopOnEntry": true,
            "osx": {
                "pythonPath": "${env:SPARK_HOME}/bin/spark-submit"
            },
            "windows": {
                "pythonPath": "${env:SPARK_HOME}/bin/spark-submit.cmd"
            },
            "linux": {
                "pythonPath": "${env:SPARK_HOME}/bin/spark-submit"
            },
            "program": "${file}",
            "cwd": "${workspaceRoot}",
            "env": {},
            "envFile": "${workspaceRoot}/.env",
            "debugOptions": [
                "WaitOnAbnormalExit",
                "WaitOnNormalExit",
                "RedirectOutput"
            ]
        },
.....
}

2. I want to add a debug config named PyTest  , which can use in every existed project,  or new create project .
3. But now,  launch.json is create by pythonVSCode  with its default launch.json . Except the project I modified myself,  neither new project  nor other old projects  would have  PyTest .
4. So a global user defined launch.json ,  automatically sync with each project would be good.

We currently support having a global launch.json that a user can specify in his configuration file. More about this https://github.com/Microsoft/vscode/issues/18401
Apart from that any debugger can customize the launch.json any way he wants, more about that can be found in https://code.visualstudio.com/updates/v1_16#_debug-api-updates

I have clarified that if a launch.json is present in the workspace the global launch confiugration is ignored.
I  think this global is useless in most situation.

At least,

If I add custom python and js gloabl config ,  I wouldn't like to see the js config show on python project.
Please provide a parameter,  to make global config auto merge into  workspace's config or not.

I have describe the details, I think this feature is not only a global config. Sync/merge config is the big point.
Please consider to reopen this.

This feature request will not be considered in the next 6-12 months roadmap and as such will be closed to keep the number of issues we have to maintain actionable. Thanks for understanding and happy coding!


Llm params

Description: Fixed the OpenLLM python client to be able to edit LLM config parameters when sending requests.
Issue: Currently to use the client with necessary user configuration, the openllm bento needs to be edited. For testing purposes this can become quite tedious, to keep editing the configuration in the bentos and query the LLM server.
Dependency: No additional dependencies.
Tests: Sample codes have been included in the main directory under sample-tests folder.
@aarnphm

Sorry what does this PR add?
query and generate should already accept llm parameters on each request. see llm_config.
This seems to me like a code golf PR. Is there anything specific difference about this other than style change?

I will close this since I didn't see anything difference. I will check your other PR. Thanks

Hi, thank you for checking in. I noticed that the query method, always uses the default configurations for the Model. For example, if the model is deployed with max_tokens 128, then the same is used for subsequent queries. The other LLM_config params behave the same way. Therefore, I added an update feature to be able to change the configuration when the user uses the query method.
As for your concern over using LLM_config, the LLM config is uses as a whole new dictionary key in the final configuration.
Example;
client.query(llm_config  = {max_new_tokens:2000}), the final configuration is reflected as
{llm_config ; {max_new_tokens:2000}, generation_config : {default_llm_config}
This is the above problem that i am trying to fix since at the end of the day the model uses generation config params and not the LLM_config.
Do take a look at the changes once again @aarnphm.
Thanks

This is actually fixed to since all items are dumped flatten, so it will correspond correctly now. I have tested this with 0.4.0

That request is proccessed with
```python
client.generate("What is the definition of time?", max_new_tokens=12)
```
If you don't pass any generation config
client.generate("What is the definition of time?")

If you see the default value is used here

You can clearly sees the difference here.


Args inverse

https://github.com/buildingSMART/ifcJSON/blob/7ead4a168a757fca1922584e127a70cf8364b864/file_converters/ifc2json.py#L44
Traceback (most recent call last):
File "ifc2json.py", line 80, in
NO_INVERSE=args.inverse,
AttributeError: 'Namespace' object has no attribute 'inverse'
Change line 80 : inverse to no_inverse

still gives error : line 80, in
NO_INVERSE=args.inverse,
AttributeError: 'Namespace' object has no attribute 'inverse'

Thanks for reporting this should be fixed now


RESPONSE!???

How do i get response, i need to return file name cause i'm generating name on backend, so how do i return it in response???

please check the readme.
On 2 December 2017 at 19:27:32, Kerim Dragolj<EMAIL_ADDRESS>wrote:

How do i get response, i need to return file name cause i'm generating
name on backend, so how do i return it in response???
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/bleenco/ngx-uploader/issues/396, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABtntvGHA7q9-sv0Akil5DpMx_IjD1SPks5s8ZaUgaJpZM4QzaLB
.

i did, like 10x just today, still dont get it.. i mean im passing it in response, but how do i track uploading progress? im sending it in response so when percentage is 100%, but how to track it?

this question is not related to module itself but you need a understanding of how to handle events with the web applications. closing.

ty for help

clone the repo you have everything setted up.


UTF-8 support

Can you please save the text files as UTF-8 fromat ? ( I do it manullay every time I install the package )

Hi @mfmsaadi did you manage to get the package working? As you see my issue, I haven't maybe you can help me out?

Hi @mfmsaadi ,
Can you explain more, do you use a tool for conversion or just update file encoding using a text editor?

Hi @moharrum,
Yes just update the file encoding using a rext editor :)

Hi @pimduijst,
Sorry that I didn't see your comment before !
I think @pixxet helped you in your issue thread :)

Yup, thanks @pimduijst for the quick fix.
This is now fixed in 2.3.1


"MarqueeLabel: Failed to find t for Y input!"

I am working with tvOS FYI
Context: I have a collectionviewcell that has a marquee label on it...On awakeFromNib of that cell I pause the marquee...When the collectionviewcell becomes focused i call these methods:
titleLabel.restartLabel()
titleLabel.unpauseLabel()
Then when the focus leaves that cell i call:
titleLabel.pauseLabel()
titleLabel.resetLabel()
The bug is when I come back to the cell a second time. When I call:
titleLabel.restartLabel()
titleLabel.unpauseLabel()
I get the  "MarqueeLabel: Failed to find t for Y input!" message.

The label doesn't marquee when i come back to it a second time FYI...it just sits there

Hmm, this might be a result of the pausing/unpausing of the label? That error occurs when MarqueeLabel can't figure out how to time the scroll animation, and can be a real pain to debug.
You could try using the holdScrolling property instead, and see if that avoids the problem? I'd say using holdScrolling is more appropriate for your use case anyway. To replicate your flow:
In awakeFromNib:
// do setup, and also set...
titleLabel.holdScrolling = true
// this way the labels won't scroll until you change holdScrolling OR
// manually trigger a scroll with triggerScrollStart()

When the cell becomes focused, and you want to start the scroll:
titleLabel.holdScrolling = false  // if scroll is appropriate, it should start automatically

When the focus leaves the cell, and you want to stop the scroll:
titleLabel.holdScrolling = true  // prevent auto scrolling
titleLabel.shutdownLabel()  // this will return the label to the home position

In looking at this I realized the restartLabel() function doesn't do what I intended it to do, which is return the label to the home position and subsequently start the scroll if the appropriate conditions are met. The current implementation won't stop an in-flight animation unless the conditions for scroll are met. I'll push an update to correct that, although I don't think that error is directly related to your issue.

Pushed the change described above in release 2.4.0.

that worked great!...thank you

Awesome, glad that fixed it!


Improving of the generated Web3 contracts

It's not required to wrap these methods.
You could do something like this:
import {Contract} from 'web3-eth-contract';

class MyContract extends Contract {
// pre inject abi
}

I think you're referring to legacy target (web3 0.20.x), right? It's done like this to add promises supports and (subjectively) improve the API.
We don't do it anymore for any other targets. Reopen please if I am mistaken.


[Always] append internal service port to Service Name

Hi all,
we have a use case who needs always expose the Internal Service Port in the Service Name.
I think it could be useful, because services always will have the same format and can be easily parsed with ^\w+-\d+$.
This behavior could be setted with a flag appendInternalPort by default false to maintain backward compatible.

@ch3lo Maybe PR title would better reflect the code changes if it was something like "Flag/Option to unconditionally append internal service port number to Service Name"