Feature request

Is your feature request related to a problem? Please describe.
when saving the output to a txt file, it is being saved with generic name, therefore if i scan 2 wpengine sites (ex: sxq.wpengine.com and 123.wpengine.com), the file name that is being saved is the same (wpengine_header_date). So if i run scan on 5 wpenging sites, it keeps on overwriting the name of the file since its the same as prior scan.
Describe the solution you'd like
Able to include a txt file that humble can use to scan a list of sites and save the output as txt/pdf/html file. Also, save the output as the full URL (123.wpengine.com_date) instead of generic (ex:wpendine_header_date)
Describe alternatives you've considered
manually doing it
Additional context
Add any other context or screenshots about the feature request here.

Hello,
Thanks for your suggestions!.
Take a look at this recent commit: https://github.com/rfc-st/humble/commit/f42344c6fdda9e910adb4753cbcd34b46b9b8891; the export file name now includes both the subdomain and the TLD.
About the other points, maybe in the future I will add to humble functionality to scan URLs taking as source a text file, in which there are several of them. For now I leave it in backlog.
An alternative option to what you propose would be (tested on Linux console), something similar to:
datasets=('https://google.com' 'https://tesla.com'); for dataset in "${datasets[@]}"; do python humble.py -u "$dataset" -o pdf; done

Hi again!
The example:
datasets=('https://google.com' 'https://tesla.com'); for dataset in "${datasets[@]}"; do python humble.py -u "$dataset" -o pdf; done
must be on a single line in the console, and must be executed inside the directory where "humble.py" resides. This example:
1.- Defines 'datasets' with a list of the URLs to be analyzed.
2.- Iterates through that list, executing "humble.py" (with whatever parameters you want) against each of the above URLs.
3.- In this example, '$dataset' is the URL.
This is the output that i get on Kali Linux (with this same example, changing only "python" to "phyton3"):


What happens when machines are off pre and post replication

Just skimming through this document, it does not mention what happens when a machine is off pre replication and post-replication. May you please cover this.

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

ID: c714cc40-47f5-fb3c-51e0-f504c38ec937
Version Independent ID: 15da2499-61eb-ec11-53be-7d006cdd4b38
Content: Set up disaster recovery for Azure VMs to a secondary Azure region with Azure Site Recovery
Content Source: articles/site-recovery/azure-to-azure-tutorial-enable-replication.md
Service: site-recovery
GitHub Login: @rayne-wiselman
Microsoft Alias: raynew

Thanks!
VMs must be on in order to replicate. We use an extension to the VM agent for replication, so the agent must be running. thx.
#please-close


Comment on a question is having some issues

After doing a comment on a question, if I immediately try to comment again on the same quesiton the the page reloads and shows on button to either comment or answer. Have a look in the pic below.

it's not an issue, it's normal Q2A behavior.
thirst time answer or comment is submitted, it will be posted using ajax and without page load, but next time Q2A will act as if JS is disabled and reload page to sent new request.


Update x-status as per the ModelGuide

x-status is changed from enum to an object. so the models need an update

@ANISH-GOTTAPU FYI

@ANISH-GOTTAPU @ashutshkumr is this done?


Calico CNI timed out deleting device in netns

Expected Behavior
pod should be deleted successfully， should not terminating for longtime
container veth should be deleted successfully， should not hang too long
Current Behavior
E0706 02:09:22.341833   96334 cni.go:352] Error deleting $pod from network calico/k8s-pod-network: Calico CNI timed out deleting device in netns /proc/110432/ns/net
W0706 02:09:22.349142   96334 docker_sandbox.go:384] failed to read pod IP from plugin/docker: NetworkPlugin cni failed on the status hook for pod "$pod": Unexpected command output Device "eth0" does not exist.
I0706 02:09:22.354099   96334 kubelet.go:1930] SyncLoop (PLEG): "$pod)", event: &pleg.PodLifecycleEvent{ID:"ed73ef2b-dd65-11eb-9b19-20677cd40678", Type:"ContainerDied", Data:"e44a342accf338ad93571dff08d35089ccf29f75e6e9e7e64e350320acad23fa"}
W0706 02:09:23.376623   96334 docker_sandbox.go:384] failed to read pod IP from plugin/docker: NetworkPlugin cni failed on the status hook for pod "$pod": Unexpected command output nsenter: cannot open /proc/110432/ns/net: No such file or directory
E0706 02:09:41.014426   96334 remote_runtime.go:132] StopPodSandbox "2898b9b55bbed12ffbcb3798371750c4d13f5ce534e110a713b16ee68f68c7e3" from runtime service failed: rpc error: code = Unknown desc = NetworkPlugin cni failed to teardown pod "$pod" network: Calico CNI timed out deleting device in netns /proc/110432/ns/net
E0706 02:09:41.014547   96334 kubelet.go:1553] error killing pod: failed to "KillPodSandbox" for "ed73ef2b-dd65-11eb-9b19-20677cd40678" with KillPodSandboxError: "rpc error: code = Unknown desc = NetworkPlugin cni failed to teardown pod \"$pod\" network: Calico CNI timed out deleting device in netns /proc/110432/ns/net"
E0706 02:09:41.014576   96334 pod_workers.go:190] Error syncing pod ed73ef2b-dd65-11eb-9b19-20677cd40678 ("$pod)"), skipping: error killing pod: failed to "KillPodSandbox" for "ed73ef2b-dd65-11eb-9b19-20677cd40678" with KillPodSandboxError: "rpc error: code = Unknown desc = NetworkPlugin cni failed to teardown pod \"$pod\" network: Calico CNI timed out deleting device in netns /proc/110432/ns/net"
W0706 02:09:41.669790   96334 docker_sandbox.go:384] failed to read pod IP from plugin/docker: NetworkPlugin cni failed on the status hook for pod "$pod": CNI failed to retrieve network namespace path: cannot find network namespace for the terminated container "2898b9b55bbed12ffbcb3798371750c4d13f5ce534e110a713b16ee68f68c7e3"

Possible Solution
// CleanUpNamespace deletes the devices in the network namespace.
func CleanUpNamespace(args *skel.CmdArgs, logger *logrus.Entry) error {
	// Only try to delete the device if a namespace was passed in.
        if args.Netns != "" {
                logger.WithFields(logrus.Fields{
                        "netns": args.Netns,
                        "iface": args.IfName,
		}).Debug("Checking namespace & device exist.")
                devErr := ns.WithNetNSPath(args.Netns, func(_ ns.NetNS) error {
                        _, err := netlink.LinkByName(args.IfName)
			return err
                })

                if devErr == nil {
                        logger.Infof("Calico CNI deleting device in netns %s", args.Netns)
                        // Deleting the veth has been seen to hang on some kernel version. Timeout the command if it takes too long.
                        ch := make(chan error, 1)

                        go func() {
                                err := ns.WithNetNSPath(args.Netns, func(_ ns.NetNS) error {
                                        return ip.DelLinkByName(args.IfName)
                                })

                                ch <- err
                        }()

			select {
                        case err := <-ch:
				if err != nil {
                                        return err
                                } else {
                                        logger.Infof("Calico CNI deleted device in netns %s", args.Netns)
                                }
                        case <-time.After(5 * time.Second):
                                return fmt.Errorf("Calico CNI timed out deleting device in netns %s", args.Netns)
                        }
                } else {
                        logger.WithField("ifName", args.IfName).Info("veth does not exist, no need to clean up.")
                }
        }

        return nil
}

the comment say 'Deleting the veth has been seen to hang on some kernel version'
Your Environment

Calico v3.6.5
kubernetes v1.14.3
Operating System and version: CentOS 4.9

@skyline09 Do you see this happen often, and do the pods get deleted eventually? Otherwise, if this is only happening occasionally, I think transient issues with deleting the interface are expected.
Also, it might be worth testing with a newer version of Calico and Kubernetes, those versions are quite old.


sendTextMessage is not working

I am implementing this in my hapi application. sendMessage is working fine, but sendTextMessage is not working.
var nexmo = require('easynexmo');
nexmo.initialize('KEY', 'SECRET');
function consolelog (err,messageResponse) {
if (err) {
console.log(err);
} else {
console.log(messageResponse);
}
}
nexmo.sendTextMessage(XXXXXXXXXXX,YYYYYYYYYY,'Test Meessage',consolelog); // This is not working
nexmo.sendMessage({from:XXXXXXXXXXX,to: YYYYYYYYYY, text:'Test Message'}, consolelog); // This is working fine

There is a 4th parameter in sendTextMessage method to send any additional options to the API. I made this parameter optional in the new version published today, please give this a try in the new version and let me know how it goes.
Thanks for raising this issue and helping in improving the library.

It's working perfectly. Thanks for quick fix.


[TEST] Run e2e test cases on Photon OS to see if any failures there

What's the test to develop? Please describe
Run e2e test cases https://vmware.github.io/photon/ to see if any failures there.
Describe the tasks for the test

Additional context

I tried to create a photon OS AWS AMI to start a ec2 instance based on it, but it's not possible because of insufficient permission for iam:CreateRole:
aws iam create-role --role-name vmimport --assume-role-policy-document trust-policy.json

An error occurred (AccessDenied) when calling the CreateRole operation: User: arn:aws:iam::641769369267:user/yang.chiu is not authorized to perform: iam:CreateRole on resource: arn:aws:iam::641769369267:role/vmimport because no identity-based policy allows the iam:CreateRole action

@innobead Is it possible for you to help add the permission? Thank you!

@yangchiu is it working now?

Test results based on Photon OS 5.0 GA:
amd64: 8 failures
arm64: 4 failures
Failed test cases are:

test_ha_backup_deletion_recovery:

Flaky test case on master pipeline as well

test_engine_image_not_fully_deployed_perform_auto_upgrade_engine:

Flaky test case on master pipeline as well

test_volume_metrics:

Known issue https://github.com/longhorn/longhorn/issues/7626#issuecomment-1984969057

test_engine_live_upgrade_while_replica_concurrent_rebuild:

Timeout issue. Passed after increased the timeout value: https://ci.longhorn.io/job/public/job/v1.4.x/job/v1.4.x-longhorn-tests-sles-amd64/487/

test_setting_concurrent_rebuild_limit:

Timeout issue. Passed after increased the timeout value: https://ci.longhorn.io/job/public/job/v1.4.x/job/v1.4.x-longhorn-tests-sles-amd64/484/

test_setting_concurrent_volume_backup_restore_limit:

Timeout issue. Passed after increased the timeout value: https://ci.longhorn.io/job/public/job/v1.4.x/job/v1.4.x-longhorn-tests-sles-amd64/485/

test_setting_concurrent_volume_backup_restore_limit_should_not_effect_dr_volumes:

Timeout issue. Passed after increased the timeout value: https://ci.longhorn.io/job/public/job/v1.4.x/job/v1.4.x-longhorn-tests-sles-amd64/486/

In conclusion, there is no outstanding issue on Photon OS 5.0 Longhorn regression tests.
cc @innobead

We need to check if the original test ran with Realtime kernel parameter enabled. If not, we need to rerun it if it's feasible on our test bed.
@chriscchien Can you help with this? @yangchiu is OOO this week.

From the GA page, if I understand correctly only ISO x86_64 Real-Time flavour support Realtime kernel parameter enabled (I tried use tdnf install linux-rt to update kernel in ec2 instance, but instance will not ready after reboot).
Currently I have some issue while setting up cluster with ISO file. I will continue work on this and may need more time, thank you.

Update for the regression test in my local environment on Photon5 RT OS:

test_support_bundle.py::test_support_bundle_should_not_timeout:
Need 11G Memory at least for worker nodes.
test_settings.py::test_instance_manager_cpu_reservation:
Need fixed 4 CPUs worker nodes.
Other test cases are almost mentioned as https://github.com/longhorn/longhorn/issues/8113#issuecomment-1985371533

Update for the regression test in my local environment on Photon5 RT OS:

test_support_bundle.py::test_support_bundle_should_not_timeout:
Need 11G Memory at least for worker nodes.
test_settings.py::test_instance_manager_cpu_reservation:
Need fixed 4 CPUs worker nodes.
Other test cases are almost mentioned as [TEST] Run e2e test cases on Photon OS to see if any failures there #8113 (comment)

@mantissahz these test cases are intended. Did we encounter any issues with other cases?

these test cases are intended. Did we encounter any issues with other cases?

No, no other cases are outstanding.

Ref: https://github.com/longhorn/longhorn/issues/8253

I've tried different approaches but still unable to create a valid AMI to run an EC2 instances. Need more time to investigate.

No worries. No need to spend extra time on this image build (rt). We already spent enough time on this. Revisit this if needed.


Refactor observation reports.

Add mycoflora format.
Move species_list report here.
Get it all to pass rubocop (as much as possible).
Add unit tests.

Coverage decreased (-10.2%) to 78.024% when pulling 6de5a20bd52c598664c8fcc756125fc7752d4c02 on pellaea:mycoflora_report into 7ff5a270ef0943a0100c2761d5af0a125a685111 on MushroomObserver:master.

Coverage increased (+0.02%) to 88.29% when pulling 9db486874f67aec1a9edc3ffb09b96e41b977754 on pellaea:mycoflora_report into 7ff5a270ef0943a0100c2761d5af0a125a685111 on MushroomObserver:master.

Coverage increased (+0.02%) to 88.29% when pulling 66df8709e7121791eadea11e9460e0d8ff0f1f5a on pellaea:mycoflora_report into 7ff5a270ef0943a0100c2761d5af0a125a685111 on MushroomObserver:master.

IMO: Just fix the easy offenses, mark the others Invalid or Wontfix.

Oooh!  There is a way to mark them "invalid" or "wontfix"???  Yes!  We need
to in this case, I think.  I tried valiently, but could not find any sane
way to reduce a method which is comprised of a single array of labels(!)
Even when I moved every value in the array into a method, it still
complains about too many lines and ABC overcomplexity.  But I don't see any
way to avoid creating a big array.    How does one break that up??
On Thu, Jun 8, 2017 at 5:00 PM, Joseph D. Cohen<EMAIL_ADDRESS>wrote:

IMO: Just fix the easy offenses, mark the others Invalid or Wontfix.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/MushroomObserver/mushroom-observer/pull/319#issuecomment-307226107,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADE2te3Ziy7RREJszP9O6ELSwL50jFrXks5sCGD_gaJpZM4NzRGv
.

Yes, you can do that.
What do you see when you click on Details on the codeclimate line in the report above?

If you see lines like this:

then hover over New.

Thanks, Joe.  Yes, I see four options:
New
Confirmed
Invalid
Wontfix
Would you mind telling me what the lower three mean?
On Thu, Jun 8, 2017 at 5:25 PM, Joseph D. Cohen<EMAIL_ADDRESS>wrote:

If you see lines like this:
[image: screen shot 2017-06-08 at 14 24 59]
https://user-images.githubusercontent.com/45460/26951531-52160098-4c56-11e7-845b-ea4ad5f62953.JPG
then hover over New.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/MushroomObserver/mushroom-observer/pull/319#issuecomment-307232000,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADE2tSfMeDn22sAbWPzSl9pu-2Dk2e--ks5sCGbigaJpZM4NzRGv
.

See https://docs.codeclimate.com/docs/issues under Issue Statuses.

Thanks!
On Thu, Jun 8, 2017 at 5:34 PM, Joseph D. Cohen<EMAIL_ADDRESS>wrote:

See https://docs.codeclimate.com/docs/issues under Issue Statuses.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/MushroomObserver/mushroom-observer/pull/319#issuecomment-307234054,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADE2tfj_YEisoByGeFQyDiKDDtrS1OjOks5sCGjsgaJpZM4NzRGv
.

Another approach is disabling individual cops for part of a file by magic comments.
The relevant cops here are Metrics/AbcSize and Metrics/MethodLength.

Thanks.  Depends if it recomplains each time I touch one of these too-long
methods.
On Thu, Jun 8, 2017 at 5:44 PM, Joseph D. Cohen<EMAIL_ADDRESS>wrote:

Another approach is disabling individual cops for part of a file
https://github.com/bbatsov/rubocop/blob/master/manual/configuration.md#disabling-cops-within-source-code
by magic comments.
The relevant cops here are Metrics/AbcSize and Metrics/MethodLength.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/MushroomObserver/mushroom-observer/pull/319#issuecomment-307236173,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADE2tbVrRoNZiOtvvJOu5FHALhZZUpKyks5sCGs4gaJpZM4NzRGv
.

I think it will do that; it hates you.

On Thu, Jun 8, 2017 at 5:50 PM, Joseph D. Cohen<EMAIL_ADDRESS>wrote:

I think it will do that; it hates you.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/MushroomObserver/mushroom-observer/pull/319#issuecomment-307237344,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADE2tbGAH11ZVl6vDVOi3B6leKz68GmTks5sCGyIgaJpZM4NzRGv
.

Weird, it did it to me again.  There were 6 temporary complaints.  Started looking them up, then discovered the file in question passes rubocop locally.  Looked back and the 6 errors were gone.  Bizarre.
Anyway, rubocop seems content for the moment.  Still want to add a unit test for the content of at least one line of each report, but otherwise done.

Coverage increased (+0.02%) to 88.29% when pulling 8482e90ec1f4264158a6515148a27033e9d71ed5 on pellaea:mycoflora_report into 7ff5a270ef0943a0100c2761d5af0a125a685111 on MushroomObserver:master.

Yes that's weird.  I didn't see it, but I wasn't watching the report all the time.  The only thing I saw was that it passed.

Let's go with "it hates me".
On Thu, Jun 8, 2017 at 5:58 PM, Joseph D. Cohen<EMAIL_ADDRESS>wrote:

Yes that's weird. I didn't see it, but I wasn't watching the report all
the time. The only thing I saw was that it passed.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/MushroomObserver/mushroom-observer/pull/319#issuecomment-307238967,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADE2tYsDWfhRr9HTVRNqxF4Ix-m642Giks5sCG5ygaJpZM4NzRGv
.

Coverage increased (+0.03%) to 88.292% when pulling 424342024acbb5b402e919eebbbaaa02cddf6acc on pellaea:mycoflora_report into 7ff5a270ef0943a0100c2761d5af0a125a685111 on MushroomObserver:master.

Coverage increased (+0.03%) to 88.292% when pulling f964b0bb2d39102a5f0655c175f437f2b7f12f92 on pellaea:mycoflora_report into 7ff5a270ef0943a0100c2761d5af0a125a685111 on MushroomObserver:master.

Coverage increased (+0.09%) to 88.359% when pulling 6e17a653444f08f3a8a64e60479d30fa3443fdc6 on pellaea:mycoflora_report into 7ff5a270ef0943a0100c2761d5af0a125a685111 on MushroomObserver:master.

Coverage increased (+0.1%) to 88.362% when pulling 174f481cdccc537db0577062212d6e5341db072a on pellaea:mycoflora_report into 7ff5a270ef0943a0100c2761d5af0a125a685111 on MushroomObserver:master.

I looked at this, but can't really help.  Nothing jumped out at me from the code.
But we should ask Steve about lat/lon.  My understanding of the code is that for decimalLatitude and decimalLongitude we export the "best" figures:

If the user has supplied exact lat/lon we export those,
else we export the lat/lon of the center of the Observation location.

The report doesn't distinguish between those two.  So there's no way to know the precision of decimalLatitude anddecimalLongitude -- could be up to 1/2 the diameter of the planet.
Steve might prefer something else. Maybe the observation.lat and .long, plus the Location bounding points. Maybe just leave decimalLatitude anddecimalLongitude``nil if there's no observation.lat and .long.

Good point, I'll ask.  Thanks!
On Tue, Jun 13, 2017 at 7:30 PM, Joseph D. Cohen<EMAIL_ADDRESS>wrote:

I looked at this, but can't really help. Nothing jumped out at me from the
code.
But we should ask Steve about lat/lon. My understanding of the code is
that for decimalLatitude and decimalLongitude we export the "best"
figures:

If the user has supplied exact lat/lon we export those,
else we export the lat/lon of the center of the Observation location.

The report doesn't distinguish between those two. So there's no way to
know the precision of decimalLatitude anddecimalLongitude -- could be up
to 1/2 the diameter of the planet.
Steve might prefer something else. Maybe the observation.lat and .long,
plus the Location bounding points. Maybe just leave decimalLatitude and
decimalLongitude``nil if there's no observation.lat and .long.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/MushroomObserver/mushroom-observer/pull/319#issuecomment-308277493,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADE2tVcT7LY6t3B0Pl43EWvGPCEqXdusks5sDxuwgaJpZM4NzRGv
.

Coverage increased (+0.1%) to 88.362% when pulling 2098f2f8235b5b78daae32d3c92bdc8926aa27c2 on pellaea:mycoflora_report into 7ff5a270ef0943a0100c2761d5af0a125a685111 on MushroomObserver:master.


Session saved in the DB but req.session result is {"cookie":{"originalMaxAge":172800000,"expires":172800000,"secure":false,"httpOnly":true,"path":"/"}}

var store = new MongoStore(
{
uri: process.env.ATLAS_URI,
collection: 'sessions'
},
function(error) {
// Should have gotten an error
});
store.on('error', function(error) {
assert.ifError(error);
assert.ok(false);
});
app.use(session({
secret: 'abcdefghijklm',
saveUninitialized: true,
resave: true,
cookie: {
secure: false,
expires: 1000 * 60 * 60 * 24 * 2
},
store: store
}));
app.post('/cart', function(req, res){
var cart = req.body;
req.session.cart = cart;
req.session.cart = cart;
req.session.save(function(err){
if(err){
throw err;
}
res.json(req.session.cart);
});
})
app.get('/', function(req, res) {
res.send(req.session);
});

Same issue here.
// api/auth:login
router.post(apiRoutesBack.AUTH.LOGIN, async (req, res) => {
        req.session.user = {
            id: 'someId'
            email: 'someEmail',
            username: 'someUsername',
        };

        res.json({ user: req.session.user });
});

And on app load, in my initmethod,
// api/auth:load
router.get(apiRoutesBack.AUTH.LOAD, async (req, res) => {
        console.log('req.session', req.session);
        res.json(req.session);
});

console.log('req.session', req.session); always prints:
req.session Session {
        cookie: {
            path: '/',
            _expires: 2020-09-18T07:13:43.733Z,
            originalMaxAge: 604800000,
            httpOnly: true,
            secure: null,
            domain: null,
            sameSite: null
      }
}

My setup in server.ts:
import express from 'express';
import next from 'next';
import { authRouter } from './routes';
import { connectDb } from './utils/connectDb';
import session from 'express-session';
import { v4 } from 'uuid';
import MongoDBStore from 'connect-mongodb-session';
import config from 'config';

const sessionStore = MongoDBStore(session);
const sessionInit = {
    secret: 'This is a secret',
    cookie: {
        maxAge: 1000 * 60 * 60 * 24 * 7,
    },
    resave: true,
    saveUninitialized: true,
    genid: () => v4(),
};

const PORT = process.env.PORT || 3000;

const dev = process.env.NODE_ENV !== 'production';
const nextApp = next({ dev });
const handle = nextApp.getRequestHandler();

const applyRoutes = (app: Application) => {
    app.use('/api/auth', authRouter);
};

const startServer = async () => {
    await nextApp.prepare();
    await connectDb();

    const app = express();

    if (!dev) {
        app.set('trust proxy', 1); // trust first proxy
        sessionInit.cookie.secure = true; // serve secure cookies
    }

    sessionInit.store = new sessionStore({
        uri: config.get('mongoURI'),
        collection: 'sessions',
    });

    app.use(session(sessionInit));

    app.get('*', (req, res) => handle(req, res));
    app.listen(PORT, () => console.log(`App listening on port ${PORT}`));
};

startServer();

The sessions collecting gets created on MongoDB Atlas, but the cookie seems to not persist through page reloads and page navigations.

Same for me. The session gets stored in the database, but not in req.session.
Is there a special way to update the session for req.session?

Ended up using pure cookies (stateless, server less, storeless). [next-iron-session](https://www.npmjs.com/package/next-iron-session)

I was having this same issue and spent so long trying to figure this out! But here's what fixed it:
add this before session configuration:
app.set(("trust proxy", 1);
Add proxy option to session:
app.use(session({     secret: 'secret',     resave: false,     saveUninitialized: false,     cookie: { secure: true, httpOnly: true, sameSite: 'none' },     store: store,     proxy: true }))

I was having this same issue and spent so long trying to figure this out! But here's what fixed it:
add this before session configuration: app.set(("trust proxy", 1);
Add proxy option to session: app.use(session({ secret: 'secret', resave: false, saveUninitialized: false, cookie: { secure: true, httpOnly: true, sameSite: 'none' }, store: store, proxy: true }))

hey, it still not working, I use the Redis store for sessions like that:
app.set("trust proxy", 1);

app.use(bodyParser.json());
app.use(bodyParser.urlencoded({ extended: true }));
app.use(cookieParser());
app.use(
  cors({
    origin: "http://localhost:3000",
    credentials: true,
  })
);

app.use(
  session({
    store: new RedisStore({
      client: redisClient,
      ttl(sess) {
        return 1000 * 60 * 60 * 24 * 7;
      },
    }),
    proxy: true,
    secret: "secretmeforever",
    saveUninitialized: false,
    resave: false,
    cookie: {
      maxAge: 1000 * 60 * 60 * 24 * 7,
      httpOnly: false,
      secure: false,
      sameSite: "none",
    },
  })
);

but result still:
Session {
  cookie: {
    path: '/',
    _expires: 2023-12-17T06:57:16.209Z,
    originalMaxAge: 604800000,
    httpOnly: false,
    secure: false,
    sameSite: 'none'
  }
}

I lost a few days... I'm already tired

@DavidForDev  I feel you. I tried for 3 days before getting it to work. There were a few other things that I did along the way that probably made a difference.

If you are using passport like me, confirm that your deserialize function is being called and working properly. Add console.log() statements. If you don't see the deserialize function being called, it probably means the cookie wasn't sent with the request.
Confirm if the session cookie is actually being sent with your request. On Chrome you can use the dev tools > application >storage > cookies > localhost:3000 to confirm it even exists. Then check the network request and see if the cookie was sent with in the headers. Things can get a bit wonky here because your dev environment might not register the cookie as the same origin as your server. To combat this you can:

unblock 3rd party cookies in your browser settings
in your cors setup: corsOptions = { origin: true, credentials: true }
in your frontend where you're making the request, set credentials to 'include'.

    try {
        const response = await fetch(
            baseUrl + path, {
                method: 'POST',
                credentials: 'include',
                headers: {
                    'Content-Type': 'application/json',
                },
                body: JSON.stringify(body)
            });
            const data = await response.json();
            return data;
    } catch (err) {
        return err
    }```

- I haven't yet figured out how to get the cookie to send with GET requests. I suspect it's something to do with my CORS setup, but you might have better luck testing this with post/put requests first.

If you want you can check out my auth code [here](https://github.com/effieguenther/grandma-server/blob/main/authenticate.js) and my session code [here](https://github.com/effieguenther/grandma-server/blob/main/connect.js)

@DavidForDev I feel you. I tried for 3 days before getting it to work. There were a few other things that I did along the way that probably made a difference.

If you are using passport like me, confirm that your deserialize function is being called and working properly. Add console.log() statements. If you don't see the deserialize function being called, it probably means the cookie wasn't sent with the request.
Confirm if the session cookie is actually being sent with your request. On Chrome you can use the dev tools > application >storage > cookies > localhost:3000 to confirm it even exists. Then check the network request and see if the cookie was sent with in the headers. Things can get a bit wonky here because your dev environment might not register the cookie as the same origin as your server. To combat this you can:

unblock 3rd party cookies in your browser settings
in your cors setup: corsOptions = { origin: true, credentials: true }
in your frontend where you're making the request, set credentials to 'include'.
export const post = async (path, body) => { try { const response = await fetch( baseUrl + path, { method: 'POST', credentials: 'include', headers: { 'Content-Type': 'application/json', }, body: JSON.stringify(body) }); const data = await response.json(); return data; } catch (err) { return err }
I haven't yet figured out how to get the cookie to send with GET requests. I suspect it's something to do with my CORS setup, but you might have better luck testing this with post/put requests first.

If you want you can check out my auth code here and my session code here

thank you for your answer. I use different tools, so for more detail, you can see my post on Stackoverflow: https://stackoverflow.com/questions/77634864/trouble-accessing-session-data-in-express-redis-after-setting-from-apollo-client
thanks


[BUG] iOS: authenticationPresentingViewController error when confirming payment sheet with 3DS

Describe the bug
When attempting to confirmPaymentSheetPayment, I encounter the following error on iOS with a 3DS payment and on a  modal presentation view
To Reproduce

Open a modal presentation view

init payment
attempt to confirm the payment with confirmPaymentSheetPayment
-> the 3DS screen is not displayed and i get the following error:

authenticationPresentingViewController is already presenting. You should probably dismiss the presented view controller in prepareAuthenticationContextForPresentation

Works correctly with a classic payment (no 3DS)
Works correctly on Android
Works correctly on containedModal presentation view

Expected behavior
The 3DS screen should be displayed on top of the stack.
Smartphone:

Device: iPhone 15 Plus Simulator
OS: iOS 17.5

Additional context

Expo version: 51.0.34
@stripe/stripe-react-native version: 0.38.6
React Native version: 0.74.5
@react-navigation

The 3DS screen opening normaly in a modal view seems conflict with my current modal view
Did you note some issues with a "modal" presentation view and 3DS screen?

I opened a bug on react native because the Modal behaviour changed a lot and was not addressed in changelog.
I think that Stripe problem with 3d secure depend on this: https://github.com/facebook/react-native/issues/47694
I also opened a bug here to fix this: https://github.com/stripe/stripe-react-native/issues/1772


Support environment-based parameter value

Can you integrate an environment based parameter value?
For example google analytics id for production environment should differ from local or test environment.

That seems like a very useful feature, thank you for the suggestion, I think I'll look into that very soon.

This is now available in v1.0.0 via:
php katana build --env=production

You may create a config-production.php file with configurations that'll overwrite the ones in config.php.


jest-mock: Correctly handle export ... from ... in babel-style ES2015 modules

The logic in getSlots() in jest-mock currently doesn't correctly handle the code generated by babel for export { ... } from '...' statements.
This code:
export { Thing } from "./module-c";
Generates:
"use strict";

Object.defineProperty(exports, "__esModule", {
  value: true
});

var _moduleC = require("./module-c");

Object.defineProperty(exports, "Thing", {
  enumerable: true,
  get: function get() {
    return _moduleC.Thing;
  }
});

But getSlots() ignores the Thing field because it ignores getters. This means that when generating an automatic mock for this module, trying to import { Thing } from "module-that-uses-export-from" will result in Thing being undefined. A reproduction of the issue is here: https://github.com/benweissmann/jest-issue-repro
This PR detects the __esModule field, and correctly handles the module. I've added a unit test to verify the new behavior, and confirmed that this fixes the test in https://github.com/benweissmann/jest-issue-repro
An alternative approach to the special-case i used here would be to include enumerable getters in getSlots(). However, when I modified the conditional in this PR to be if (!propDesc.get || propDesc.enumerable), another test broke, so I think that's a riskier approach that more severly break backwards-compatiblity. You can see the full test output (https://gist.github.com/benweissmann/d2cedab28fb31fd6eb4a31c5b5e01474) or take a look at my other branch (https://github.com/benweissmann/jest/tree/enumerable-getters)

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please sign up at https://code.facebook.com/cla - and if you have received this in error or have any questions, please drop us a line at<EMAIL_ADDRESS>Thanks!

Thank you for signing our Contributor License Agreement. We can now accept your code for this (and any) Facebook open source project. Thanks!

Current coverage is 90.31% (diff: 100%)

Merging #1617 into master will not change coverage

@@             master      #1617   diff @@
==========================================
  Files            28         28
  Lines          1125       1125
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           1016       1016
  Misses          109        109
  Partials          0          0

Powered by Codecov. Last update 046e665...956ed28

Thank you!


TS incorrectly flags a side effect-only import as TS5097 (An import path can only end with a '.ts' extension...) in --watch mode

🔎 Search Terms
"TS5097", "allowImportingTsExtensions", "noEmit", "watch", "side effect", "import"
🕗 Version & Regression Information

This changed in commit or PR 89e928e8b41368bb0888c6d399854c53a2dbf3d4

⏯ Playground Link
No response
💻 Code
When importing a .ts file strictly for side effects,
import './a.ts';

I expect that this will be accepted by the typechecker. In fact, even if a.ts didn't exist, this would still be valid TS, see https://github.com/microsoft/TypeScript/issues/50394#issuecomment-1381014487 for a motivating example.
Typechecking with --noEmit validates this without error.
Typechecking with --watch gives me error TS5097: An import path can only end with a '.ts' extension when 'allowImportingTsExtensions' is enabled.
a.ts
console.log("I'm a side effect!");

b.ts
import './a.ts';

tsconfig.json
{}

🙁 Actual behavior
% npx tsc --noEmit # no output. Valid!
% npx tsc --watch
[5:54:03 PM] Starting compilation in watch mode...

b.ts:1:8 - error TS5097: An import path can only end with a '.ts' extension when 'allowImportingTsExtensions' is enabled.

1 import './a.ts';
         ~~~~~~~~

[5:54:04 PM] Found 1 error. Watching for file changes.

I shouldn't see a new error only in --watch mode.
🙂 Expected behavior
% npx tsc --noEmit # no output. Valid.
% npx tsc --watch
[5:54:37 PM] File change detected. Starting incremental compilation...

[5:54:37 PM] Found 0 errors. Watching for file changes.

Additional information about the issue
Workaround: // @ts-ignore before the import.

accidentally-ish one of my recent PRs fixes this: https://github.com/microsoft/TypeScript/pull/58668

@Andarist And you wanted to make this an error across the board... tsk tsk :wink:

I didn't say I'm happy I fixed this, did I? :trollface:

I really wonder why @Aarononomous is doing this in his code though. What's the reason behind importing a side-effectful .ts without enabling allowImportingTsExtensions? 🤔

I really wonder why @Aarononomous is doing this in his code though. What's the reason behind importing a side-effectful .ts without enabling allowImportingTsExtensions? 🤔

I'm working in a multitenant codebase. We put platform-specific code in *.platformName.ts files, adding the platform name as an extension. Then we have a webpack resolver configured to look for platform files preferentially,
resolve: {
  extensions: [
    ...
    `.${CONFIG.platform}.ts`,
    `.${CONFIG.platform}.tsx`,
    '.ts',
    '.tsx',
    ...

so that when we build for, say, platformX and see import './a', we first look for a file at ./a.platformX.ts, then ./a.ts. This allows us to write all of our platform-specific code without runtime tests or compiler macros or whatever, neatly encapsulated in its own file.
So not only do we not want to allow ts extensions, we can't! Otherwise our whole multitenant resolver behavior falls apart.
The exception to this is in the multitenant files themselves, where we do sometimes want to import common code from the extensionless base files, e.g.
a.platformX.ts
// @ts-expect-error
// We need the full filename with extension here: otherwise there's a circular import
import { module as baseModule } from './a.ts';

// do something with baseModule...

export const module;

The 'allowImportingTsExtensions' error is expected when we import with a full filename. It's just when we only import for side effects from the base file that we see this inconsistency in --watch mode.
So the code example above is really more like
a.ts
window.__CONFIG__ = { ... };

a.platformX.ts
import './a.ts';

window.__CONFIG__.platformX = { ... };

The use case makes sense but I think you could introduce some other convention for those base modules to avoid this problem altogether.

@Andarist, what do you suggest?
If I put my side-effectful code in, say, a.base.ts, I'd still be calling it in a.ts, a.platformX.ts, etc. with
import 'a.base.ts';


Font size is way too big until I resize the window

As the title says, the font size is way too big until I resize the window.
<EMAIL_ADDRESS><EMAIL_ADDRESS>

I am also facing same issue.
<EMAIL_ADDRESS><EMAIL_ADDRESS>Is there any workaround atleast till final fix for this issue?

It could be that some of your other components or styles are delayed and are not applied at the moment we mount the TextFit component, thus we compute the values incorrectly. Notice that the demo works correctly without resizing the window.
You can try and show a minimal reproduction case with the help of jsfiddle.net, or you can try and find what delays correct styling in your app, and trigger a resize event on the window yourself, once the app settles in and we can compute the correct dimensions.
By all means, do tell in this issue if you have new information.

trigger a resize event on the window yourself, once the app settles in and we can compute the correct dimensions.

Triggering a resize event on the window can have side-effects we don't want - for instance, I could have several components with Textfit and want to resize only one of them.
Would you consider adding a way to manually force the recomputation of the text size?

Textfit is idempotent, so it does not matter if other component resize as well.
If there’s a use case for it, we may add the feature. However, I would rather identify the use cases and solve them so that it works out of the box, without manual method calling.

Sorry, what I meant is that there can be performance hits if other components also act on the window resize event.

If you have a performance problem with resize, I think that’s what you might focus on. Consider what happens when user resizes their window. I don’t think this use case warrants our further attention, I’m sorry. :-(


Support Local Network Routing Through Warp Client

Thanks for creating a great add on!
Is your feature request related to a problem? Please describe.
I’m unable to use the Home Assistant apps through the tunnel when authentication is turned on. I’m able to login through the Cloudflare one time password wall, but the apps doesn’t seem to accept the resulting cookie.
Describe the solution you'd like
A workaround would be to support the Cloudflare Zero Trust companion app, that effectively creates a VPN to your local network from your device. I have this up an running on a stand-alone device in my LAN, and it works great. It would involve adding this lines to the Cloudflare config
warp-routing:
  enabled: true

Adding a route to your tunnel like this:
cloudflared tunnel route ip add 192.168.XX.0/24 tunnelname
And finally remove your LAN IP subnet in Cloudflare’s Split Tunnels settings.
See https://digaround.cloud/homeassistant-secure-access-with-cloudflare-warp/ for full context.

The problem with Cloudflare Access when logging in with Home Assistant in the companion apps is known. As a workaround, various forums recommend deactivating the Access rule for a short time and enabling again after successful login. After that Cloudflare Access should work with Home Assistant.
However, I have looked at the warp function and think that it can be implemented without much effort. I will create a first PR later to provide the function.

@ketilmo Looking into this, there are three things that come to my mind.

For me, using GitHub as an authentication provider is working, using Google is not. So try using GitHub, if this is a viable solution for you.
To clarify: Warp routing is just another way of routing traffic to complete subnets and not only single applications. Therefor, it is not really addressing your problem on its own. The set-up was just done in a way to also use WARP auth in your tutorial. Nevertheless, @elcajon is looking into enabling to use Warp routing, since it is a valid option anyways.
Having that said, you can also enable the WARP auth for singe applications. I suggest you do that if you do not want to go with GitHub as auth provider. In the policies section of your home assistant application, you can define that the user has to be authenticated using WARP. You could also create a group for that in the Groups section and then just add it.

Let me know if anything in that comment was not clear or if you have any follow-up questions.

@brenner-tobias @elcajon,
Thank you so much for your feedback and for implementing this feature in no time. I tried the GitHub integration, but it gives me the same error message in the companion app. But the WARP integration works great. Thanks a lot for implementing it, I highly appreciate it. Keep up the great work.

I fully understand that this is not something you can fix, as the problem is probably related to the HA app implementation. But hopefully, it will work more flawlessly in the app someday. Thanks again for creating a great add-on. It's appreciated!


How to add comments to subdirectories

For example, If have e structure like:
-common
--models
how do I add an info to a models directory?
Adding something like "common.models": "Info text" to comments section of store.js
Sorry if I missed some documentation, nice work!

Hi you can just use "common/models": 'Info text'

Hi @ArsSirek!
The comment above by @debs-obrien is correct :) For reference you can always check tree.js to see how the path is defined. I'll update the docs to be more explicit.


Omit body for declaration of computed properties

For example:

This is actually quite a bit more involved than I'd originally anticipated. As far as I can tell, the fix involves updating SwiftSemantics to either change the behavior of the initializer or to make declarations mutable so that SwiftDoc can trim information from its definition.


asynchronously implement the training and prediction process？

Very nice project, but how to asynchronously implement the training and prediction process

Hi @leepand,
Thank you for your interest in using MABWiser! Could you provide more detail about your request?

@erstrong  Thank you for that quick response. In a production environment, we often need to separate model updates from predictions for system  performance, i.e. one api gets user feedback and updates the model, while the other api uses the latest model information for predictions.

Hi @leepand,
The library is designed for prototyping and simulations. You can of course use it in any way you want under the Apache 2.0 license, but we have not tested it for production use or asynchronous programming.
Regarding your specific question, predict would be blocked by the fit method as it requires that the fit method must have been previously called on at least one row of data. After that initial fit, there is nothing in the API to prevent partial fit and predict from running concurrently. However, some of the learning policies both update stored attributes in fit/partial fit and perform calculations with those attributes in predict, which could create problems.
If you want to use the library in a live environment, I would suggest fitting the model offline or scheduling partial fitting to occur during recurring downtime. The MAB objects can be serialized with the pickle protocol. You might also want to look at StreamingBandit which is designed for live experiments.

Hi @erstrong,
Thank you so much for your response, I'll try the pickle method.


rfc: add FromId and ToId to FluentSortableList

💬 RFC

The current implementation of FluentSortableListEventArgs does not allow one to determine from which list onto which list the item was dragged and then dropped. By providing two additional properties, FromId and ToId the developer would be able to determine which lists were involved.
🔦 Context

I am building a todo application and sort the items into buckets (think of kanban columns). I want a view with all the buckets and the ability to drag and drop one item from one list to the next. I have considered using the FluentDragContainer. I decided against it because I would have to re-implement the nice visuals of FluentSortableList on my own and because the underlying SortableJS library provides the means to solve my problem.
💻 Examples

I have forked the fluentui-blazor project and implemented changes that accomplishes what I want. You can view it here. I also changed the code in the demo page for the FluentSortableList-Component to test my changes and provide a demonstration.
https://github.com/user-attachments/assets/c893d1e0-6f93-4a12-ae12-8e558949fd46
I can create a pull request but I don't know if it would be okay to do that out of the blue.

Interesting :-)
Yes, I think this could be an interesting addition for developers.

I would name the properties FromListId and ToListId to be clearer.
In the Demo example, I wouldn't modify the whole example, which seems simple enough at the moment, but I would only add traces in the right console to display the Indexes and these new IDs.

A unit test will also be needed to validate this new functionality (although the current tests are not yet complete 😊 ).

^^
I can modify my branch accordingly. I have a question regarding the unit test, what exactly does the test have to test for? As I understand it I need to test that the Id gets added to the generated html.
After I modified it do I create a pull request?

I can modify my branch accordingly. I have a question regarding the unit test, what exactly does the test have to test for?
You can create a Unit Test to (1) verify that the IDs are added and (2) calling OnUpdateJS (simulating the JS call) the event will be fired.

And yes, you can create a PR (set a short title and a complete description including a screenshot or a GIF to explain correctly the new features).


Enable to gap detection when ordinals-api is streaming blocks from ordhook

Describe the bug
This inscription is not displayed in the ordinals explorer

https://ordinals.hiro.so/address/bc1ph03a7qkx3d2x8lslsukxxefmgjjtzsej3fvmdzyy5u8hcn4uyafql979l6

When I check it here I do see it associated with their wallet so it is definitely in there https://ordiscan.com/inscription/67758714
https://www.ord.io/67758714
Related transaction https://mempool.space/tx/9e3d1d1a2d5f4052190bc2e1658b9982d130c9f6cd0a57fbd5c7af8e7c4d958d
2024-04-07 22:08 (15 hours ago)

@314159265359879 Thanks for the nudge, this inscription should now be visible.
I'll re-purpose this issue, we need to re-enable to gap detection when ordinals-api is streaming its content from ordhook.

:tada: This issue has been resolved in version 4.0.0-beta.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 4.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


'st2 login' exits with "success" 0 on failure

Auth failure for st2 login command results in "success" exit code 0.
Example:
# st2 login 123 -p 123
Failed to log in as 123: 401 Client Error: Unauthorized
MESSAGE: Invalid or missing credentials for url: http://quoting-rabbit-st2auth-enterprise:9100/tokens

root@quoting-rabbit-st2client-enterprise-7ccdb7d457-xcp87:/opt/stackstorm# echo $?
0

Same applies if for example st2 login can't reach the st2api server, - the command fails with error message, but exit status is 0.
There could be some other similar issues with st2 login as well that needs a closer look and overhaul.
I however see the correct rc behavior from st2 auth CLI.

I think the issue is here
    def run_and_print(self, args, **kwargs):
        try:
            self.run(args, **kwargs)
        except Exception as e:
            print('Failed to log in as %s: %s' % (args.username, str(e)))
            if self.app.client.debug:
                raise

            return

If you run the command with --debug, it raises an Exception, and returns with rc=1.
extreme@EWC:~$ st2 --debug login -p foobar st2admin
# -------- begin<PHONE_NUMBER>95760 request ----------
curl -X POST -H  'Connection: keep-alive' -H  'Accept-Encoding: gzip, deflate' -H  'Accept: */*' -H  'User-Agent: python-requests/2.14.2' -H  'content-type: application/json' -H  'Content-Length: 2' -H  'Authorization: Basic c3QyYWRtaW46Zm9vYmFy' --data-binary '{}' http://<IP_ADDRESS>:9100/tokens
# -------- begin<PHONE_NUMBER>95760 response ----------
{
    "faultstring": "Invalid or missing credentials"
}
# -------- end<PHONE_NUMBER>95760 response ------------

Failed to log in as st2admin: 401 Client Error: Unauthorized
MESSAGE: Invalid or missing credentials for url: http://<IP_ADDRESS>:9100/tokens
ERROR: 401 Client Error: Unauthorized
MESSAGE: Invalid or missing credentials for url: http://<IP_ADDRESS>:9100/tokens

CLI settings:
----------------
Config file path: /home/extreme/.st2/config
Client settings:
----------------
ST2_BASE_URL: http://<IP_ADDRESS>
ST2_AUTH_URL: http://<IP_ADDRESS>:9100
ST2_API_URL: http://<IP_ADDRESS>:9101/v1
ST2_STREAM_URL: http://<IP_ADDRESS>:9102/v1
ST2_AUTH_TOKEN: None

Proxy settings:
---------------
HTTP_PROXY:
HTTPS_PROXY:

Traceback (most recent call last):
  File "/opt/stackstorm/st2/local/lib/python2.7/site-packages/st2client/shell.py", line 402, in run
    func(args)
  File "/opt/stackstorm/st2/local/lib/python2.7/site-packages/st2client/commands/auth.py", line 153, in run_and_print
    self.run(args, **kwargs)
  File "/opt/stackstorm/st2/local/lib/python2.7/site-packages/st2client/commands/auth.py", line 124, in run
    manager = self.manager.create(instance, auth=(args.username, args.password), **kwargs)
  File "/opt/stackstorm/st2/local/lib/python2.7/site-packages/st2client/models/core.py", line 40, in decorate
    return func(*args, **kwargs)
  File "/opt/stackstorm/st2/local/lib/python2.7/site-packages/st2client/models/core.py", line 306, in create
    self.handle_error(response)
  File "/opt/stackstorm/st2/local/lib/python2.7/site-packages/st2client/models/core.py", line 166, in handle_error
    response.raise_for_status()
  File "/opt/stackstorm/st2/local/lib/python2.7/site-packages/requests/models.py", line 928, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
HTTPError: 401 Client Error: Unauthorized
MESSAGE: Invalid or missing credentials for url: http://<IP_ADDRESS>:9100/tokens
extreme@EWC:~$ echo $?
1
extreme@EWC:~$

But if you're not using debug, the RC is 0.
extreme@EWC:~$ st2 login -p foobar st2admin
Failed to log in as st2admin: 401 Client Error: Unauthorized
MESSAGE: Invalid or missing credentials for url: http://<IP_ADDRESS>:9100/tokens
extreme@EWC:~$ echo $?
0
extreme@EWC:~$


chore: main back to alpha

:tada: This PR is included in version 2.5.1-alpha.1 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


collect (data): running the script

📖 Describe what you want
code in src/models leads to a failed of the script to download dataset if some functions missing apparently

✔ Definition of done
Dataset download script is not depend of the src/models code. Thus when running:
.42AI/get_dataset_script1.sh

I needed to remove the parser corresponding to features and model to be able to download the data

I made a rebase and there is still a problem with an unrelated portion of code about the model !


[BUG] Handle division by 0

Hi!
When the container of the Waveform Component has no width, the noOfSamples passed to the native is 0, and then the app crashes due to division by 0 error. It would offer a much better DX when you catch this case by e.g. don't render anything, when the width is 0 anyways, give the container a minimum width or at least make a Math.Max(noOfSampels, 1), so the app doesn't crash.

came across the same issue, thx for raising @mbpictures

We have released a new version 2.1.2. Please use that, and if the issue still persists, reopen or create a new issue.


Cut off of bottom of modal problem

It's not an image but html elements from ajax api, also responsive design so I can not set the height.
Somehow problem does not happen to any modal on the page if it's not first time.
Once opened (any) model smoothly show up without cut off.
"ADD TO CART" button middle of the page.
https://tokuyama-us.soto-media.com/estelite-sigma-quick/

The FAQ entry I linked you to was specific to HTML content, just like you are using.  Your HTML content has  elements in it that need width and height attributes so that the modal can be sized accurately before the images finish loading.

If I set width/height to modal content through js dynamically in onComplete:function() before resize(), does it work, by any chance?
I have many modal with different sizes, also its size will change depend on screen size.
so specify size to each modal content is bit tough.

I would avoid calling resize if you can prevent it, and you can prevent it if you just add the width and height attributes to your <img> elements. Sorry, Colorbox is an old script that doesn't handle progressive loading or responsiveness that well.

Thank you. I'll try to set height/width to imgs somehow.
Does min-height/min-width & max-height/max-width of imgs affects to modal size?

Yes, any CSS style that impacts layout.  Like the width and height attributes, CSS styles are immediately available, allowing those impacts to layout to be calculated once your markup has been added to the DOM.  Since your <img> elements currently lack a width/height attr, or CSS width/height/min-width/min-height, they will initially have 0x0 size until the image has loaded.  Hope that makes sense.

Thank you very much.


Update deploy/vercel.mdx

Description (required)

Update deploy/vercel.mdx

This section is outdated, the current Astro version is 4.7.1 and I think we should update it
Related issues & labels (optional)

Closes #
Suggested label:

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale
File
Note

en
guides/deploy/vercel.mdx
Source changed, localizations will be marked as outdated.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.

netlify has the same issue


Add a replay system

If the current timer passes the current lap timer, we know there is a desync early. This needs to be implemented before merge.

If the current timer passes the current lap timer, we know there is a desync early. This needs to be implemented before merge.

On second thought, it's fine to delay this until 0.2.


support/rails-7

Story
Support for Rails 7.0

[x] Bump railties version
[x] create Gemfile for Rails 7
[x] Update travis pipeline to use other ruby versions.

https://app.travis-ci.com/github/jotolo/wysiwyg-rails/builds/245941086

merge it plz

Rebased with master

Conflicts were resolved with new version. Version 4.0.12 proposed


maybe if typst's version above 0.7,there is a bug about counter of affiliation_counter

in my pc,first affiliation's number is "-",not "a"; second affiliation's number is "a",not `"b".
so i moved counter's step award
affiliations [
      #affiliation_counter.step()
      #super(affiliation_counter.display("a"))#h(1pt)#emph(affiliation)

Thanks for catching that!
I've implemented less hacky solution by updating the counter to 1


Added option to make toast undimissible

Added example
Added unit tests

In what cases would you want such a behavior @GermainBergeron ?

In some cases the error should be visible until the user fix the problem.
In my specific use case a user can add many cards to a dashboard and go over the size limit. To fix his problem he can resize a card, remove a card, etc. Until he fixes the situation the toast will stay visible (and he won't be able to save)

Codecov Report

Merging #404 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #404      +/-   ##
==========================================
+ Coverage   99.81%   99.81%   +<.01%
==========================================
  Files         155      155
  Lines        3689     3690       +1
  Branches      885      886       +1
==========================================
+ Hits         3682     3683       +1
  Misses          7        7

Impacted Files
Coverage Δ

src/components/toast/Toast.tsx
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 4401293...a5dff5e. Read the comment docs.


feat: enable gosec and default linter set

fixes: #352
Signed-off-by: jpwhitemn<EMAIL_ADDRESS>

If your build fails due to your commit message not passing the build checks, please review the guidelines here: https://github.com/edgexfoundry/device-mqtt-go/blob/main/.github/Contributing.md
PR Checklist
Please check if your PR fulfills the following requirements:

[x] I am not introducing a breaking change (if you are, flag in conventional commit message with BREAKING CHANGE: describing the break)
[x] I am not introducing a new dependency (add notes below if you are)
[ ] I have added unit tests for the new feature or bug fix (if not, why?) no new features added to product; just linting
[x] I have fully tested (add details below) this the new feature or bug fix (if not, why?)
[ ] I have opened a PR for the related docs change (if not, why?) n/a

Testing Instructions

New Dependency Instructions (If applicable)

recheck


Problem with file name auto fix

In Check L29 default value of $fix is no and isn't identical when validate at Check L112 and filename always auto fix.

https://github.com/kenjis/codeigniter3-filename-checker/blob/master/controllers/Check.php#L112 is $this->fix, not $fix.


Select which Dockerfile to build from (i.e. support -f flag)

In my application, I have to separate different test environments into different Dockerfile,
and name each of them in different Dockerfile names. (e.g. Dockerfile-a, Dockerfile-b, ...)
Can I achieve this?
Since I didn't find any fileName support from the GenericContainer.fromDockerfile api.
Thanks a lot!

Hi @Johnny850807, I've created a PR https://github.com/testcontainers/testcontainers-node/pull/96.
I'll merge it unless you have any concerns?

@cristianrgreco It looks great, thank you!

Great! Thanks for raising this, it's available in 2.16.0


extension stopped displaying images during debug

Issue Type: Bug
I updated my vscode and no longer able to display images using the extension.
Extension version: 2.2.0
VS Code version: Code 1.64.0 (Universal) (5554b12acf27056905806867f251c859323ff7e9, 2022-02-03T04:20:17.224Z)
OS version: Darwin x64 21.2.0
Restricted Mode: No
Remote OS version: Linux x64 5.4.0-1064-aws

System Info

Item
Value

CPUs
Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz (16 x 2300)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
2, 2, 2

Memory (System)
16.00GB (0.02GB free)

Process Argv
--crash-reporter-id 5819c93b-df7c-44ab-9a77-d451747848ac

Screen Reader
no

VM
0%

Item
Value

Remote
SSH: [DEV] Dolev CPU

OS
Linux x64 5.4.0-1064-aws

CPUs
Intel(R) Xeon(R) CPU E5-2686 v4 @ 2.30GHz (4 x 2300)

Memory (System)
15.63GB (0.88GB free)

VM
0%

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
pythonvspyl392:30425749
pythontb:30283811
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
pythondataviewer:30285071
vscod805:30301674
pythonvspyt200:30340761
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593:30376534
vsc1dst:30433059
pythonvs932:30410667
wslgetstartedc:30433508
vs360:30404995
vsrem710:30416614
vscop841cf:30434590
vscscmwlcmc:30433760

@DolevBria
Can't reproduce (tested with Windows and Ubuntu).
Do you see an error notification?
Do this setting is checked?

If it checked, please delete /tmp/svifpd and see if it helps.

I tried it, also tried unchecking this setting, still doesn't work with version 2.2.0.

Do you see the lightbulb near variables?
Do the Image Watch window works? (e.g. showing any variable)
(make sure to add "images" to the image watch settings, if you don't have it already)

The lightbulb doesn't appear on version 2.2.0.
The variable (which is a PIL image and ndarray representing RGB Image) is on my watch window (it is not None).
That setting is checked

I mean in this window:

Also, can you share a minimal example that doesn't work for you?

The IMAGE WATCH menu appears empty on version 2.2.0 and has my images on 2.1.1

I've tried to reproduce the problem on a minimal example, however I'm not able to. I can only say that the image watch menu is turning out empty, and the image is around 1000 * 2000 pixels represented as PIL image.

Just to make sure:
You tried to reproduce with a minimal example, and everything seems to be working on that example?
If so, I have a couple of questions:

Is that a simple python debug, or you're using IPython cells?
Do you have a multi-threaded code?
Can you try run the Update Frame ID command in the Commands Palette while debugging, and see if it changes anything?


Resolve SetLoadBalancerAffinityTimeout not being effective

[ ] Make sure you have followed Kube-OVN Code Style.

What type of this PR
Examples of user facing changes:

Bug fixes

Which issue(s) this PR fixes:
Fixes #(2462)
WHAT

🤖 Generated by Copilot at 2fd08a6
Update load balancer options with affinity timeout. This change modifies the pkg/ovs/ovn-nb-load_balancer.go file to set and store the affinity timeout option for load balancers in the OVN network backend.

🤖 Generated by Copilot at 2fd08a6

options map set
load balancer updated
autumn leaves fall

HOW

🤖 Generated by Copilot at 2fd08a6

Set the affinity timeout option for load balancers in the OVN network backend (link,                             F

@gugulee Please review

can you write a unit test for this issue?

Thanks! @jiliangsui


deprecate rangeEnabled in non-pilosa repos

Description
Remove unnecessary references to RangeEnabled, and add deprecation warnings to clients when RangeEnabled is used. This will definitely include the clients, and PDK. Let's check through through other public and private repos as well.
Creating this issue in github.com/pilosa/pilosa to avoid creating 8 other issues in disparate repos.
Blocked on #1205 - once that's merged, this should proceed.
Success criteria (What criteria will consider this ticket closeable?)

Python and Java clients have this on their master.
Implemented for the Go client in PR: https://github.com/pilosa/go-pilosa/pull/129
We need to update PDK deps to use the latest master of Go client when the PR ^ is merged.

This is implemented in all clients. Closing.


Duplicated timeseries in CollectorRegistry

Code:
from prometheus_client import Gauge, Counter
a = Gauge('request', 'test')
b = Counter('request_total', 'test')

Output:
Traceback (most recent call last):
  File "main.py", line 4, in <module>
    c = Counter('request_total', 'test')
  File "lib/python3.10/site-packages/prometheus_client-0.12.0-py3.10.egg/prometheus_client/metrics.py", line 136, in __init__
    registry.register(self)
  File "lib/python3.10/site-packages/prometheus_client-0.12.0-py3.10.egg/prometheus_client/registry.py", line 30, in register
    raise ValueError(
 ValueError: Duplicated timeseries in CollectorRegistry: {'request'}

I understand that _total suffix would be removed and automatically added afterward, but this behavior breaks some metrics. Any workaround on this? --

... rename one of the metrics? :grin:


New NasmLexer for Pygments

(Original issue 337 created by delroth on 2008-06-13T10:31:30+00:00)
Since we already have a GasLexer, there is no reason why we should not have a NasmLexer.
The patch including the lexer and a typo fix in a comment is attached.

(Original issue was assigned to thatch)
(Original issue 337 last updated on 2008-07-26T16:46:21+00:00)
(Issue automaticaly closed due to status in Bitbucket: resolved)

(Original comment by gbrandl on 2008-07-26T13:36:52+00:00)
It's now in main. Thanks all!

(Original comment by thatch on 2008-07-25T00:31:04+00:00)
A couple of tests are included in [194b1a425cbd], not yet merged.

(Original comment by mitsuhiko on 2008-06-13T22:17:52+00:00)
Implemented in [1917da7239a2], but tests are missing.  I'll leave it open until I add some test files.

(Original comment by delroth on 2008-06-13T10:53:27+00:00)
Strange, why is the priority always switching from minor to blocker ?


[Fix](row cache) invalid row cache using key encoded without sequence…

… column
This PR change segment writer to vertical segment writer， so the regression-test case may be be passed
add vertical segment writer
Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


Update raw export to not remove headers

Description
fixes #4167
Checklist

[x] I have updated tests where applicable.
[ ] I have added an entry to the CHANGELOG.

Fantastic, thanks! 🍰
Would you mind adding a changelog entry? Other than that this LGTM.


fix tab dropped bug

After finishing dragging the Tab, it is possible that the Tab will be displayed very small;
I tried to fix it; thanks.

Thank you for your reply; if there is anything else that needs to be modified, you can tell me, thank you


evaluate hand for pair, 2 pair, {3,4} of a kind, full house

I suppose you'd also have to add a NoMatch (-1) ad InvalidHand (-100) struct to the enum.

I'll add "Nothing" to the PokerHandRank enum type, because that's what it's called in Poker. "I got Nothing." is the common colloquialism.
As for "InvalidHand", I'm not sure it's worth it to add that to the enum. It really shouldn't happen unless we introduce a program bug. Rust just complained because I didn't account for all cases in the match statement.
We could add a separate enum for custom error codes and strings? Or wrap the result in an io::Result and do something like this? https://github.com/theimpossibleastronaut/rmwrs/blob/fabcf801a65a7d86a380573cf60ef7dff6d85511/src/lib.rs#L139


[Feature Request]: Add animation to the ambtracker.html file

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
We can add a little animation where the marker drops down.
Please check out the video and accept my pull request if you liked the dropping animation.
Use Case
It would be a nice visual effect after the page loads
Benefits
https://github.com/user-attachments/assets/caa94a9f-db2f-49be-97e9-92e7b08ac0d0
Priority
Low
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC contributor
[X] I want to work on this issue
[X] I'm willing to provide further clarification or assistance if needed.

Please accept my pull request if you liked this animation.

Please accept my pull request if you liked this animation.

what changes you have made? it seems same

Please accept my pull request if you liked this animation.

what changes you have made? it seems same

in this page, there is issue for live location tracking for ambulance. for ex on google map

Earlier the marker was just appearing on the page, now it drops down like a pin. Its a very small change.. please watch the before and after.
https://github.com/user-attachments/assets/f66d649e-8f31-4cb4-abec-db6599ae9468
Uploading Screen Recording 2024-10-03 170353.mp4…

After:
Uploading Screen Recording 2024-10-03 170353.mp4…

ok i'll read up on google API and see if i can help


[3pt]  - Add comments counter - Student B

Even if the API gives you the number of comments you will create a specific function to calculate it.
Make sure that the correct number is displayed on the Comments pop up.

The PR for this task.


postinstall script fails

Sorry to be so fast off the bat on this one!  The commit you just made to fix the windows postinstall script does not work on my mac.
Here is an error message:
><EMAIL_ADDRESS>postinstall /Users/aphillipo/Sites/work/dave/static/react/node_modules/react-router
> node scripts/postinstall.js

module.js:338
    throw err;
    ^

Error: Cannot find module '/Users/aphillipo/Sites/work/dave/static/react/node_modules/react-router/scripts/postinstall.js'
    at Function.Module._resolveFilename (module.js:336:15)
    at Function.Module._load (module.js:286:25)
    at Function.Module.runMain (module.js:475:10)
    at startup (node.js:117:18)
    at node.js:951:3
npm ERR! Darwin 15.0.0
npm ERR! argv "/Users/aphillipo/.nvm/versions/node/v4.1.1/bin/node" "/Users/aphillipo/.nvm/versions/node/v4.1.1/bin/npm" "install" "--save"<EMAIL_ADDRESS>npm ERR! node v4.1.1
npm ERR! npm  v2.14.4
npm ERR! code ELIFECYCLE

npm ERR<EMAIL_ADDRESS>postinstall: `node scripts/postinstall.js`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>postinstall script 'node scripts/postinstall.js'.
npm ERR! This is most likely a problem with the react-router package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node scripts/postinstall.js
npm ERR! You can get their info via:
npm ERR!     npm owner ls react-router
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /Users/aphillipo/Sites/work/dave/static/react/npm-debug.log

Sounds like scripts/postinstall.js isn't there? What do you get when you npm run postinstall? Also, what does ls -al scripts give you?

It's because scripts is in .npmignore. Probably make an npm-scripts? Except I'm trying that and Babel is choking on the stage 0 stuff for some reason.

Oh, right. Let's just remove that line.

It's easy enough to just move the NPM-required hook scripts and not ship the build stuff though. I'm trying this, though, and the actual npm run build is failing for some reason that I can't figure out.

@taion If you can figure it out, feel free to push something else. I'm not too concerned about it though.

@mjackson
It's the second weirdest thing I've seen, after that NPM packing bug. For whatever reason, babel running for an installed dependency seems to completely ignore the .babelrc for that dependency (and any command line options you specify), and instead goes straight to the .babelrc of the package for which you're installing the dependency.
In other words, the auto-build on postinstall only works on projects that specify stage 0 in their .babelrc configs. ¯_(ツ)_/¯

I had the same issue, deleting the cached build from .npm seems to make it obey my babelrc again.


Markdown filter not working with handlebar syntax in  element

Hi there, I'm using the latest version of cactus for mac and I'm experiencing some issues with the markdown plugin. In my article I am trying to demonstrate an example of the templating language used in Cactus, of course this uses curly braces.
If I was to tab twice to enable code in markdown and then type {{ hello }} I would expect the following to be rendered in the front end:
<p>{{ hello }}</p>
Unfortunately, I get the following:
<p> </p>
Can anybody please point out where I'm going wrong? I know that this can be done because it's used on the Cactus documentation - http://cactusformac.com/docs/extras.html#variables

{{ hello }} will cause the template engine (Django) to attempt to output the contents of the hello variable, regardless of indentation or anything.
You should be able to disable the template engine for this part of your template with:
{% verbatim %}
    {{ hello }}
{% endverbatim %}

Cheers,

Hey, thanks very much. I'll be sure to consult the django docs first in
future.
Jamie
On 7 Jun 2015 06:43, "Thomas Orozco"<EMAIL_ADDRESS>wrote:

{{ hello }} will cause the template engine (Django) to attempt to output
the contents of the hello variable, regardless of indentation or anything.
You should be able to disable the template engine for this part of your
template with:
{% verbatim %}
{{ hello }}
{% endverbatim %}
Cheers,
—
Reply to this email directly or view it on GitHub
https://github.com/koenbok/Cactus/issues/151#issuecomment-109698094.

As you do that; note that Cactus currently uses Django 1.5; which is a bit old. This might change in the future (but not just yet).
Cheers;


contrib.data.Dataset - doc issue with Dataset.map / tf.py_func in 1.3.0rc0

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): no
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Windows 10
TensorFlow installed from (source or binary): 1.3.0rc0
Python version: 3.6
CUDA/cuDNN version: 8/6
GPU model and memory: GTX 1080
Exact command to reproduce:

The following sample is taken from here and works in TF 1.2.1
import tensorflow as tf
import numpy as np

def _read_py_function(filename, label):
  return np.zeros((100,100,1)), label

def _resize_function(image_decoded, label):
  image_decoded.set_shape([None, None, None])
  image_resized = tf.image.resize_images(image_decoded, [28, 28])
  return image_resized, label

filenames = np.array(["/var/data/image1.jpg", "/var/data/image2.jpg"])
labels = np.array([0, 37])

dataset = tf.contrib.data.Dataset.from_tensor_slices((filenames, labels))
dataset = dataset.map(
    lambda filename, label: tf.py_func(
        _read_py_function, [filename, label], [tf.uint8, label.dtype]))
dataset = dataset.map(_resize_function)

In 1.3.0rc0 the following error is produced
Cannot convert a list containing a tensor of dtype <dtype: 'int32'> to <dtype: 'uint8'> (Tensor is: <tf.Tensor 'PyFunc:1' shape=<unknown> dtype=int32>)

This is due to the breaking change mentioned in release notes. To fix, one now has to introduce an explicit tuple() like so
dataset = dataset.map(
    lambda filename, label: tuple(tf.py_func(
        _read_py_function, [filename, label], [tf.uint8, label.dtype])))

This should at least be mentioned in the API docs / programmer guide.

@mrry, could you update the documentation to reflect this change?

Thanks @cheind for reporting this.

You're welcome. While this is certainly a doc issue at this point, I want to raise the concern that assigning tuple and list  totally different semantics in Python is very uncommon (even in Tensorflow) and could lead to many suprising behaviours on user side.

I have a similar error message, but I'm not sure if I got the proposed solution correctly.
The code raising the error:
dataset = tf.contrib.data.TFRecordDataset(shard_files)
dataset = dataset.map(partial(decoder.decode, items=['label', 'image']))

where decoder is a tf.contrib.slim.tfexample_decoder.TFExampleDecoder()
So, if I got the issue right, since decoder.decode() returns a list [label, decoded_image_data] this is implicitly casted to a tensor (and thus the cast fails because label and image_data have different types).
However, if I wrap the partial in a tuple() this should fix the problem.
Is this really the intended way to use the API? Even when many other TF APIs return lists instead of tuples?

@GPhilo We've fixed this issue in the internal branch, and it should appear at HEAD soon. (Follow #12396 to see when the commit lands.) After the fix, the behavior of Dataset.map() will be the same if a function returns a tuple or a list containing the same elements, and the tuple(...) workaround will no longer be necessary.

@mrry sounds great!


Add disclaimer to app

Add a disclaimer that Wyvern is in a prototype/alpha stage and that things can and will change at any time and, in some cases, data may be lost as the technology evolves.

This should also be documented in the readme and on the main page

[ ] #8
[ ] #12

This is now on the home/landing page


Make EditorFileDialog be able to pick a folder when entering it

Currently, when in "Open folder" mode, you have to first click on a empty space to be able to select the current folder. Now the option is available from the start, like in the FileDialog node.
Also made the text change from "Select Current Folder" to "Select This Folder" when clicking in one, for clarity, and also to be like the FileDialog node.

Changed RTRs to TTRs.

Thanks!


BUG: defaults are not inherited in SS3

Affected Version
3.7.1
Description
Basically we have this:
ParentClass extends ModelAdmin
{
    private static $db = [
        'FieldA' => 'Boolean'
    ];
    private static $defaults = [
         'FieldA' = 1;
    ];

}

and

class OtherClass extends ParentClass
{

}

When we create an object of the  OtherClass type then FieldA is set to FALSE. We would expect this to be TRUE because that is the default.  We tried both "true" and "1" as default values.
From what I know about Silverstripe this is not correct AND worked differently before AND if correct, has a massive impact.

Even if this works in SS4 it would be good to add a unit test for it.

I tried using
private static $db = [
     'FieldA' => Boolean(1)
];

in the parent class, but even this method did not seem to solve it.
Having said that, maybe I am wrong.  I think the first thing is to get a confirmation of this bug.

I've tested this and things seem to work. Is it because you have extends ModelAdmin instead of extends DataObject?

SS3 test here: https://gist.github.com/7d3ff3845e6938a53388c4c9accc35e4
SS4 test here: https://gist.github.com/9962a05b11cec90896c42182e63da02d
Both pass

Closing issue for now - feel free to reopen if you can amend that unit test to make it fail?

ahhh no, it is extends DataObject for sure.  I will do some more tests.


Feature Request: Abort on request

As a user I want to be able to abort my request/message. For example give me a "stop" button after I sent my message (but not yet received a response back). Allowing to stop ollama to fully abort the request and not consume any resources anymore, until I sent again a (new) message.

Hmm, is this 'as simple' as using cancellation tokens with fetch, or am i underedtimating this?

Not sure to be honest, since I didn't use the Ollama API directly: https://github.com/jmorganca/ollama/blob/main/docs/api.md#request
I really hope the API allows you to cancel the request again. It's not just the fetch it's about aborting the generate part in ollama as well. Similar to hitting 'ctrl+c' when within the ollama CLI tool (during a ollama run <...>), which will stop the generate part and give me back the prompt (it will also stop using resources).

Fixed.
This is available on the main branch now, git pull to update your local version, or use the hosted version, which has also been updated.

Great! Thanks.

@HelgeSverre I tested it today. But it actually does not work as expected. It still consumes resources after I pressed the 'Stop' button.

While I do indeed do not get a response back, in the background it's still using the computer resources. You maybe have a fast PC with a dedicated GPU with Tensor Cores, I do not.


Change xml config Parser to accept negative values

FileExchangeStore has a default value of -1 for attribute maxDays.
The parser doesn't support this yet.

@xorpherion have a look

This has been fixed, but the config schema file on http://membrane-soa.org/schemas/proxies-1.xsd hasn't been updated yet.

Fixed.


[Bug Report] The "/crate" command makes lags.

Describe the bug
With Timings, I noticed that processing this command takes too much time and can cause short-term lags. On the test server, I also conducted a test and saw that on the completely empty server, when executing this command, TPS sags a bit (from 20 to 19.7 / 19.8).
Versions (please complete the following information):

Minecraft Server Version: PaperSpigot-1.12.2
Plugin Version: 1.10

*Do NOT say Latest use /version crazycrates in the console.
Pastebin any errors in console
Link: no errors
Issue
Lags from "/crate" command
Screenshots

Inventory clicks also take quite a while to process:


Set correct permissions for jenkins-schema.sh

Forgot that we had to chmod 755 jenkins-schema.sh, otherwise this won't work:
https://ci.dev.publishing.service.gov.uk/job/govuk_content_tagger_schema_tests/4/console

at least we're using git so it actually has first-class understanding of the permissions flags


feat(fastapi): add FastAPI Integration

See #829

Closes https://github.com/getsentry/sentry-docs/issues/5272


Set from email incorrect error message

Hello, I've found an issue, when I set incorrect email to FROM get an error  '$email must be valid and of type string.', but $email in the string should be a value that I try to set. I figured out why it happened, in this function SendGrid\Mail\Mail::setFrom you should replace single-quotes to double-quotes in the error message. Thank you.

Nice find. Working on a fix ...

Similar to issue #841.
Fixing the $email in the string should be a value that I try to set may be simple, but this only works if an invalid string is provided. Also mentioned during review.
I think a more durable solution is needed like already contributed #762.

@kampalex Good call. Slowly working through old PRs but Octobers get crazy. I'll probably fix up that PR to resolve this issue.